eclipse-lsp4j / lsp4j

A Java implementation of the language server protocol intended to be consumed by tools and language servers implemented in Java.
https://eclipse.org/lsp4j
Other
613 stars 145 forks source link

Switch from osgi to bnd makes projects that don't work in IDE from source #678

Closed jonahgraham closed 1 year ago

jonahgraham commented 2 years ago

The steps in CONTRIBUTING.md#eclipse no longer create proper OSGi plug-ins in Eclipse anymore since 6a75250. While the steps do create LSP4J projects that can be built and tested in Eclipse, they no longer have their META-INF/MANIFEST.MF updated with all the OSGi headers so that Eclipse PDE recognizes it as a proper bundle.

What this means is I can't import both LSP4J and LSP4E as source and do anything as LSP4E can't see LSP4J.

@cdietrich I missed this when testing the gradle upgrade. Any ideas before I debug this more?

cdietrich commented 2 years ago

No I don’t Unfortunately I don’t know the bnd plugin very well and in Xtext we have manual maintained manifests

jonahgraham commented 2 years ago

OK - I will have a look at some point - my urgency has gone off to resolve it quickly because the issue I was looking at wasn't actually LSP4J related after all, so I didn't need LSP4J source to resolve this issue:

cc: @sebthom FYI for you since you are actively working in this area, you may trip over it too if you want source of LSP4J editable.