Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

Adapt to newer Omnisharp-roslyn including direct support for LSP #155

Open
mickaelistria opened this issue Jan 10, 2018 · 5 comments
Open
Labels

Comments

@mickaelistria
Copy link

See OmniSharp/omnisharp-roslyn#968 (comment)
OmniSharp-roslyn now directly support LSP with ./run -stdio -lsp. It's been tried successfully on Eclipse IDE with aCute. Using it allows to get rid of a bunch of dependencies such as nodejs and of some indirections.

@bmicklea
Copy link

bmicklea commented Feb 5, 2018

@tsmaeder - is this a licensing issue? I didn't think the omnisharp LSP was truly FOSS...

@mickaelistria
Copy link
Author

I think it's truly OSS, at least OSS enough to be approved in an Eclipse.org project (Eclipse aCute got a CQ approved for Omnisharp-Roslyn). License is MIT, it uses mono (MIT) too and dotnet core to provide its features.

@mickaelistria
Copy link
Author

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=15334 is the Eclipse.org CQ that allowed Eclipse aCute to ship Omnisharp-Roslyn.

@ghost
Copy link

ghost commented Feb 6, 2018

@mickaelistria I will look at it. Thanks for bringing this up

@ghost
Copy link

ghost commented Apr 14, 2018

Tried it and failed. Somehow, Che client refuses to work: eclipse-che/che#9351

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

No branches or pull requests

2 participants