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

Update to the latest lsp4e (0.13.8) version #749

Closed

Conversation

jcompagner
Copy link
Contributor

to get the fix:
eclipse-lsp4e/lsp4e#3

@mickaelistria
Copy link
Contributor

Thanks!
Note that the requirement here is a version range, so even without a release, you can totally mix in your IDE/RCP the current release of Wild Web Developer with a newer release of LSP4E when it happens.

@mickaelistria
Copy link
Contributor

Actually, this change has no payload (except restricting decent compatibility with older LSP4E). I don't think we should restrain the ranges for bugfix, bugfixes come for free just by upgrading dependencies to their newer version in the installation, no need for all downstream consumers to take any action to allow that.

@jcompagner
Copy link
Contributor Author

yeah i am always a bit confused i kind of expected if it says:

org.eclipse.lsp4e;bundle-version="0.13.6"

that that is a "hard" version (because you have to play with [] or () to make ranges)
but it doesnt seem to be so, if you do that then OSGi also allows everything that is bigger?

i tried this in my installation and yes if i add this: https://download.eclipse.org/lsp4e/releases/latest/

and then upgrade the lib it works also, my problem is gone.

but the thing is:
https://download.eclipse.org/wildwebdeveloper/snapshots/plugins/
that already did contain the new lsp4e also as far as i can see
and i did update WWD, but that didn't help, i really had to do this manually

i guess because eclipse didn't find that as an upgrade? it did see the feature upgrade of WWD but then it still sees that it had the minimum requirements for WWD and didn't update lsp4e at the same time (i had todo that manually)

@mickaelistria
Copy link
Contributor

The bundle in download area is coming from the target-platform, which references the latest, so I guess just re-running the build would be sufficient for download area to include latest LSP4E. I've just retriggered a build.

@mickaelistria
Copy link
Contributor

Yes "0.13.6" is actually means "[0.13.6)", or "[0.13.6,0.0.0)" which means "at least 0.13.6".
Let's close as no change is needed.

@jcompagner
Copy link
Contributor Author

last night i think that https://download.eclipse.org/wildwebdeveloper/snapshots/plugins/ already had the latest lsp4e (0.13.8.xxx)

And i updated to that also already, but eclipse didn't get that 0.13.8 i still had an old one.
I guess because eclipse thought no need to update because its not needed.
That is a bit annoying for end users if you ask me.
Because how should they know that they are supposed to update just a plugin that is not really a feature directly but just something under the hood.

only when yesterday evening i really manually forced an update it worked. and it got the 0.13.8

@mickaelistria
Copy link
Contributor

Same issue can happen with any Eclipse plugin, it's not a specificity of Wild Web Developer. That's more an issue/limitation about p2's "Check for Updates" IMO. Currently, p2 does only look for newer versions of the "top-level" artifacts and doesn't look for newer version of dependencies.

@jcompagner
Copy link
Contributor Author

yes i know that it is not directly related to WWD
but this makes issues that are reported quite horrible. because you don't really know what version a user is running on if they report i am running the latest WWD version X

it then can be any version of lsp4j or lsp4e underneath it

And i guess the current way to fix this is to fix the version number in WWD to LSP4E, but yes thats also not what we always want....

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