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

Consume lemminx as a bundle from the OSGi runtime #1741

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Nov 26, 2024

Currently lemminx is embedded into WWD what has some disadvantages:

  • the jar gets bigger
  • the embedded jar needs to be extracted before it can be used
  • version bumps are required for lemminx updates
  • lemminx can't be updated independent from WWD

This now adds a require-bundle to the manifest (so we are getting a certain version we are compatible with) and uses the bundle wiring to get hold of the bundle and then transform it into a file path.

As discused here

@mickaelistria
Copy link
Contributor

Looks like build.properties needs some cleanup

@laeubi laeubi force-pushed the consume_lemminx_as_bundle branch from 1b57f8d to 7accb86 Compare November 27, 2024 05:17
@laeubi
Copy link
Contributor Author

laeubi commented Nov 27, 2024

Looks like build.properties needs some cleanup

Done!

@mickaelistria
Copy link
Contributor

and version needs a bump.

@laeubi laeubi force-pushed the consume_lemminx_as_bundle branch from 7accb86 to 867a101 Compare November 27, 2024 08:12
@laeubi
Copy link
Contributor Author

laeubi commented Nov 27, 2024

and version needs a bump.

Done!

@laeubi laeubi force-pushed the consume_lemminx_as_bundle branch from 867a101 to 26753a7 Compare November 27, 2024 08:12
@mickaelistria
Copy link
Contributor

Looks like the test now needs an import-package.

@laeubi
Copy link
Contributor Author

laeubi commented Nov 27, 2024

Looks like the test now needs an import-package.

I fear this is due to the core.runtime bundle required and its insane re-exports of al system packages...

@mickaelistria
Copy link
Contributor

I fear this is due to the core.runtime bundle required

If you can replace it with Import-Package, it'd be welcome.

@akurtakov akurtakov force-pushed the consume_lemminx_as_bundle branch from 26753a7 to 9df3359 Compare December 3, 2024 05:26
@laeubi laeubi force-pushed the consume_lemminx_as_bundle branch from 9df3359 to 8f8d7e1 Compare December 3, 2024 09:31
@laeubi
Copy link
Contributor Author

laeubi commented Dec 3, 2024

@laeubi laeubi force-pushed the consume_lemminx_as_bundle branch from 8f8d7e1 to b5287e6 Compare December 3, 2024 11:16
Currently lemminx is embedded into WWD what has some disadvantages:

- the jar gets bigger
- the embedded jar needs to be extracted before it can be used
- version bumps are required for lemminx updates
- lemminx can't be updated independent from WWD

This now adds a require-bundle to the manifest (so we are getting a
certain version we are compatible with) and uses the bundle wiring to
get hold of the bundle and then transform it into a file path.

Signed-off-by: Christoph Läubrich <[email protected]>
@akurtakov akurtakov force-pushed the consume_lemminx_as_bundle branch from b5287e6 to add237d Compare December 3, 2024 13:06
@laeubi
Copy link
Contributor Author

laeubi commented Dec 3, 2024

@mickaelistria build is finally green!

@mickaelistria
Copy link
Contributor

All that is extremely good! Please merge at your own convenience.

@laeubi
Copy link
Contributor Author

laeubi commented Dec 3, 2024

All that is extremely good! Please merge at your own convenience.

Github says

Only those with write access to this repository can merge pull requests.

and I'm not a committer on the project ;-)

@mickaelistria mickaelistria merged commit b2b109e into eclipse-wildwebdeveloper:master Dec 3, 2024
6 checks passed
@mickaelistria
Copy link
Contributor

Thank you!

@laeubi
Copy link
Contributor Author

laeubi commented Dec 15, 2024

Is there any planned release date that includes this fix?

@mickaelistria
Copy link
Contributor

@vrubezhny Do you think you can produce a new release soon?

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.

2 participants