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

Move equinox.common/registry/app/preferences and dependents to Java-17 #406

Merged
merged 3 commits into from
Nov 30, 2023

Conversation

HannesWell
Copy link
Member

The migrated plugins are all projects in equinox that are currently on Java-11.

@tjwatson I hope none of these plugins is one of the ones you want to keep at Java-1.8.
Could you please list the projects that should stay at JavaSE-1.8?
I only recall org.eclipse.osgi for sure, but I think not all of those that are currently at 1.8 need to stay there and could be migrated to Java-17 as well?

Copy link

github-actions bot commented Nov 25, 2023

Test Results

     24 files  ±0       24 suites  ±0   11m 25s ⏱️ +2s
2 150 tests ±0  2 105 ✔️ ±0  45 💤 ±0  0 ±0 
2 194 runs  ±0  2 149 ✔️ ±0  45 💤 ±0  0 ±0 

Results for commit 0455bd9. ± Comparison against base commit 6f079d2.

♻️ This comment has been updated with latest results.

@tjwatson
Copy link
Contributor

These are the ones I need to stay at Java 8:

  • org.eclipse.equinox.console
  • org.eclipse.equinox.coordinator
  • org.eclipse.equinox.metatype
  • org.eclipse.equinox.region
  • org.eclipse.osgi

@HannesWell
Copy link
Member Author

These are the ones I need to stay at Java 8:

Thanks for the list. I left those bundles untouched plus the following ones

  • org.eclipse.equinox.supplement
    Considered this effectivly as part of org.eclipse.osgi
  • org.eclipse.osgi.compatibility.state
    Not developed anymore
  • org.eclipse.osgi.util and org.eclipse.osgi.services
    Only provides metadata already respectively will (hopefully) only provide metadata soon
  • org.eclipse.equinox.launcher
    Assuming that it is used to launch a most basic osgi application I assumed it should stay at 1.8 too

All other bundles, including all tests now require Java-17. For the tests I think a lower java requirement is not necessary since the tests run in the CI with Java-17 anyways.

@HannesWell HannesWell force-pushed the java17-migration branch 3 times, most recently from 7b77d96 to 8d52e25 Compare November 29, 2023 00:41
@akurtakov
Copy link
Member

manual rebase is needed

Even if the tested bundle still required Java-1.8 its test bundle can
require Java-17 since in the CI Java-17 is used anyways to run it.
@HannesWell HannesWell merged commit 3a83f10 into eclipse-equinox:master Nov 30, 2023
22 of 25 checks passed
@HannesWell HannesWell deleted the java17-migration branch November 30, 2023 23:12
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.

3 participants