Update to Java 25, Newer tps, etc#3638
Conversation
Test Results 8 076 files ±0 8 076 suites ±0 3h 38m 46s ⏱️ - 1m 36s For more details on these failures, see this check. Results for commit 51b8399. ± Comparison against base commit fabc12e. ♻️ This comment has been updated with latest results. |
Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
@HannesWell @LorenzoBettini @szarnekow does this ring a bell, e.g with respect to tycho 5? |
|
attempt: 0d65523 |
This seems to be caused by Tycho 5. Regarding this change, for Java-25 I've already created a PR a while ago: And from my POV would be ready for some time. This would unlock multiple PRs from my side, which I consider ready besides that. |
|
@HannesWell meanwhile i have a ton of stuff on this branch. the question is rather what i overlooked
|
|
@szarnekow @LorenzoBettini @HannesWell rebased on changes by @HannesWell that are on main now |
|
In order to simplify the review and later tracing in the history I extracted your commits to move Xtext plugins to Java-21 and to drop support for Java-17, sorted them in to commits and created a separate PR with them: This also includes a few missing clean-ups in the tests I found during review. I've made sure to keep the diff to this PRs branch minimal and there are only a handful of merge conflicts (mainly in the Manifests due to closely located updated bundle requirements). With that the number of changes in this PR should be significantly reduced where the majority are mechanical changes. |
|
@HannesWell do you see further split-off potential? |
Yes, I think the TP updates can be further split off. I have just created a PR with that: Then only the changes for deployment changes and building the relocations should remain. Should that also be exported into a dedicated PR so this becomes 'empty' and can be closed (to have a dedicated PR for context)? |
|
we need to discuss the relocation stuff |
| <unit id="org.eclipse.pde.feature.group" version="0.0.0"/> | ||
| <unit id="org.eclipse.draw2d.feature.group" version="0.0.0"/> | ||
| <unit id="org.eclipse.emf.sdk.feature.group" version="0.0.0"/> | ||
| �IF config.javaVersion.isAtLeast(JavaVersion.JAVA25)� |
There was a problem hiding this comment.
@HannesWell why did you have this if/else?
afaik all simrels are still on java 21
There was a problem hiding this comment.
Good question.
I believe I did this because support for Java-25 is better with 2026-03, although it works already with 2025-03.
Should I remove it and always use 2025-03?
There was a problem hiding this comment.
i believe we can do the rest with this pr. i will undo the stuff to fullbuild.sh for now and then later on check with lorenzo
There was a problem hiding this comment.
so ok with the remaineder of this pr? if yes ill merge once build is in, and tomorrow morning update the refefrence projects one as well
There was a problem hiding this comment.
Fine for me.
Thanks.
I'll have a look at mwe in the next days and will try to land my open PRs (I assume they need some adjustments now) and apply similar changes there too.
Understand. I've also extracted that into a dedicated PR so a dedicated discussion could happen there: With that only the change of Maven-Central snapshot repositories remains as far as I can see. But from your comments I have the impression that it's a temporary change for testing and this PR is then effectively empty as everything has been extracted? If that's correct, I suggest to close this and create dedicated follow-up PRs, e.g. if #3638 (comment) requires a change. |
No description provided.