Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot update library index in 1.20.0-dev.beta3 #9791

Closed
casella opened this issue Nov 28, 2022 · 1 comment
Closed

Cannot update library index in 1.20.0-dev.beta3 #9791

casella opened this issue Nov 28, 2022 · 1 comment

Comments

@casella
Copy link
Contributor

casella commented Nov 28, 2022

After a fresh installation of 1.20.0-dev.beta3 on Windows 10 (I first deleted the contents of my .openmodelica directory), I tried to run File | Manage Libraries | Update library index. Unfortunately, I get the following output:

[3] 18:33:28 Error
Failed to rename file after downloading with curl C:/Users/Francesco Casella/AppData/Roaming/.openmodelica/libraries/index.json.tmp1 C:/Users/Francesco Casella/AppData/Roaming/.openmodelica/libraries/index.json: File exists

[4] 18:33:28 Scripting Notification
[C:/Users/Francesco Casella/AppData/Roaming/.openmodelica/libraries/index.json: 0:0-0:0]: Downloaded package index from URL https://libraries.openmodelica.org/index/v1/index.json.

First of all, the error shouldn't be there. Second, despite the claim that the package index has been downloaded, the Install library menu still shows the four built-in MSL libraries and nothing else.

This issue basically prevents using the package manager, so it must definitely be fixed before the 1.20.0 release.

@casella casella added this to the 1.20.0 milestone Nov 28, 2022
adrpo added a commit to adrpo/OpenModelica that referenced this issue Nov 28, 2022
adrpo added a commit to adrpo/OpenModelica that referenced this issue Nov 28, 2022
- move SystemImpl__rename code to omc_rename
- use omc_rename in om_curl and SystemImpl__rename
adrpo added a commit to adrpo/OpenModelica that referenced this issue Nov 28, 2022
- move SystemImpl__rename code to omc_rename in omc_file.c
- use omc_rename in om_curl.c and in SystemImpl__rename in systemimpl.c
- add omc_rename declaration in omc_file.h
@adrpo adrpo closed this as completed in 7c43b35 Nov 28, 2022
adrpo added a commit to adrpo/OpenModelica that referenced this issue Nov 28, 2022
- omc_rename returns zero on success
- SystemImpl__rename returns true on success
adrpo added a commit to adrpo/OpenModelica that referenced this issue Nov 28, 2022
- omc_rename returns zero on success
- SystemImpl__rename returns true on success
adrpo added a commit that referenced this issue Nov 28, 2022
- omc_rename returns zero on success
- SystemImpl__rename returns true on success
adrpo added a commit that referenced this issue Nov 28, 2022
- omc_rename returns zero on success
- SystemImpl__rename returns true on success
@casella
Copy link
Contributor Author

casella commented Nov 29, 2022

Good that you didn't have to fix the fix for the fix to the beta 😆

All's well that ends well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants