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

OmniSharpInstall currently broken because latest release doesn't have downloads for the various platforms #766

Open
PMunch opened this issue Mar 21, 2022 · 8 comments

Comments

@PMunch
Copy link

PMunch commented Mar 21, 2022

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. Apparently curl 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.

@nickspoons
Copy link
Member

nickspoons commented Mar 22, 2022

@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 :OmniSharpInstall v1.38.1

Yes, some sensible error warnings during the install process would be nice, if anyone wants to do the work.

@nickspoons
Copy link
Member

The issue is resolved, the release assets are available now

@PMunch
Copy link
Author

PMunch commented Mar 22, 2022

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.

@justonia
Copy link

justonia commented Mar 7, 2023

Worth noting I had to use 1.38.1 to get this to work on WSL1 + Unity.

@cachandlerdev
Copy link

Has there been any update on this issue? It seems like we still have to install 1.38.1.

@nickspoons
Copy link
Member

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?

@cachandlerdev
Copy link

cachandlerdev commented Jun 22, 2023

Unfortunately, I've been having a bit of difficulty trying to correctly setup this plugin. I can successfully run :OmniSharpInstall, but something seems to be going wrong because I keep getting the message "The OmniSharp server does not appear to be installed. Would you like to install it?" every time I restart vim, and it never actually starts working after the installation.

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.

@nickspoons
Copy link
Member

This is not related to this issue, let's discuss it in #841

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

No branches or pull requests

4 participants