-
Notifications
You must be signed in to change notification settings - Fork 80
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
SourceTests.testBug206679_247198 fails #1419
Comments
org.eclipse.equinox.http.jetty Fixes #1419
IMHO this is eclipse-pde/eclipse.pde#779 and not touching features. |
It looks like tycho ignores changes on See maven log from https://download.eclipse.org/eclipse/downloads/drops4/I20231006-0120/buildlogs.php
|
A feature is always generated (except you do not a clean build), I also can't see that something is baseline replaced (the build even says it is not a baseline capable project). So either the build still sees the old versions or something else prevents the upload of the feature. |
Checking git history of the eclipse.platform.releng/features/org.eclipse.help-feature/forceQualifierUpdate.txt file I don't see anything else was needed before to "touch" & rebuild the feature. So something changed in tycho? |
As written above, nothing is replaced so you can touch this as much as you like without any effect:
So I can only assume that the outdated version are the ones that are actually replaced by an older version and probably require some kind of version bump but I have not et analyzed that in detail, at least you can see that for jetty 12 and http.servlet (I assume that something updated by @akurtakov recently) ar actually pulled in as "latest". |
So it could be in fact PDE issue not properly resolving latest versions because of eclipse-pde/eclipse.pde#779 . Let's wait for the fix then. |
Maybe #1430 already shows if this will be fixed with jetty 12.0.2. |
Side effect from #1384.
After merging of eclipse-equinox/equinox#317
SourceTests.testBug206679_247198
started to fail in I Builds.I can see it locally too. This seem to be different from eclipse-pde/eclipse.pde#779 because PDE complains about unsatisfied imports from
org.eclipse.jetty.*
packages in theorg.eclipse.help
feature, that should be there.Looking on the feature I see outdated bundles included:
I assume the only thing is missing is a "bump" in
org.eclipse.help
feature so it "includes" neworg.eclipse.equinox.http.jetty
version.I will push a PR.
The text was updated successfully, but these errors were encountered: