-
Notifications
You must be signed in to change notification settings - Fork 419
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
Build checkouted version fail #1363
Comments
@JulioJu Can you share the error? |
hi @rchande
Without forget #1202 (comment) (I'm on Arch Linux, I should modify two files).
If I create a new branch with
All work well. @rchande what do you think about that ? It it confirmed ? At least on MS Windows ? |
@JulioJu Yes, this is a known limitation with GitVersion (it crashes if you check out a commit with no tag/branch) and also happens on windows. I don't think we plan to fix this specifically, but it may go away if we upgrade to a later instance of GitVersion. cc @david-driscoll |
@rchande thanks for your answer :-) |
We moved to latest GitVersion which resolved the problem. |
It seems it's not possible to compile an old version of
omnisharp-rosly
without make a new branch.For instance:
git checkout 1dd917436661f250c4aa6305a4ec442262d76187 # without forget https://github.com/OmniSharp/omnisharp-roslyn/issues/1202#issuecomment-421543905 ./build.sh --target Quick --verbosity Verbose --use-global-dotnet-sdk
doesn't work !
but the following works
git checkout 1dd917436661f250c4aa6305a4ec442262d76187 # without forget https://github.com/OmniSharp/omnisharp-roslyn/issues/1202#issuecomment-421543905 git checkout -b 327 ./build.sh --target Quick --verbosity Verbose --use-global-dotnet-sdk
I've had this issue when I've tried to bisect to resolve OmniSharp/omnisharp-vim#427 with @nickspoons
Anybody could confirm ? If it's confirmed, maybe this tip could be added in the README.md ?
The text was updated successfully, but these errors were encountered: