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

Add an uber-bundle artifact #1701

Closed
wants to merge 1 commit into from
Closed

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Nov 10, 2024

Currently lemminx only provides an uber-jar that has no OSGi meta data at all, because of this for example wild-webdeveloper needs to embed this jar instead of use one installed in the OSGi framework.

This now adds a new uber-bundle artifact that is at the moment very similar to uber-jar but contains some OSGi data useful for writing extensions. This can then be used as a starting point to incrementally make it more OSGified, e.g. by export/import its dependencies that are already available as OSGi bundles (gson, lsp4j, ...)

FYI @mickaelistria

This currently requires tycho-snapshot repository therefore I marked the PR as a draft but wanted to share current progress already.

Currently lemminx only provides an uber-jar that has no OSGi meta data
at all, because of this for example wild-webdeveloper needs to embed
this jar instead of use one installed in the OSGi framework.

This now adds a new uber-bundle artifact that is at the moment very
similar to uber-jar but contains some OSGi data useful for writing
extensions. This can then be used as a starting point to incrementally
make it more OSGified, e.g. by export/import its dependencies that are
already available as OSGi bundles (gson, lsp4j, ...)

Signed-off-by: Christoph Läubrich <[email protected]>
@laeubi
Copy link
Contributor Author

laeubi commented Nov 10, 2024

As an alternative I'm currently try to make the lemminx bundle itself a bit more useful, has anyone actually using that inside OSGi environments and can tell about it more?

@laeubi
Copy link
Contributor Author

laeubi commented Nov 10, 2024

There is now

as a first starting point.

@laeubi
Copy link
Contributor Author

laeubi commented Nov 24, 2024

I'm closing this in favor of much simpler

@laeubi laeubi closed this Nov 24, 2024
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

Successfully merging this pull request may close these issues.

1 participant