-
Notifications
You must be signed in to change notification settings - Fork 168
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
OmniSharpInstall currently broken because latest release doesn't have downloads for the various platforms #766
Comments
@PMunch apparently there's an issue with a nuget key, which is causing the broken release. In the meantime, the previous version of the server can be installed with Yes, some sensible error warnings during the install process would be nice, if anyone wants to do the work. |
The issue is resolved, the release assets are available now |
I think this issue should be kept open (or another one created) for the part where the script isn't able to detect the failed download when using curl. The script has a "couldn't download" error block, but this is never entered as curl by default doesn't set an error return code. |
Worth noting I had to use 1.38.1 to get this to work on WSL1 + Unity. |
Has there been any update on this issue? It seems like we still have to install 1.38.1. |
What issue? The issue is that sometimes the build assets are missing on omnisharp-roslyn releases. That is not the case currently, the v1.39.7 assets are available. Why would you have to install 1.38.1? |
Unfortunately, I've been having a bit of difficulty trying to correctly setup this plugin. I can successfully run I also have an issue where the omnisharp log reports failing to open most of the files due to missing assembly references, but perhaps that falls under one of the other Github issue pages. |
This is not related to this issue, let's discuss it in #841 |
Tried to run
OmniSharpInstall
today and it didn't work, just complained that it couldn't unzip the file. Looking at the file it was obvious why, it simply contained a single line of normal text reading "Not Found". Reading through the helper script and trying to run the commands I quickly found out that the issue is that the latest release (v1.38.2) doesn't have assets following the naming conventions. Apparentlycurl
doesn't set an error code, it just writes an error to the file by default. The-f
flag should be added to the curl call to allow it to fail with an error code so that the appropriate error could be triggered in the script.Oh, and someone should probably fix the OmniSharp release to have all the assets, not sure if this is just a CI thing where they will be added once they are done, but for now at least it is broken.
The text was updated successfully, but these errors were encountered: