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

Only apply a better profile if allowed #4553

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Dec 22, 2024

Currently a configured profile is unconditionally replaced by a better one of the current JVM, but this is wrong if not explicitly allowed by the configuration.

This is now changed, to be only used if ignoring the BREE is actually enabled.

@laeubi laeubi added this to the 5.0 milestone Dec 22, 2024
@laeubi
Copy link
Member Author

laeubi commented Dec 22, 2024

Marking for Tycho 5 only as it is probably a breaking change, so lets see what all fails now.

@laeubi laeubi force-pushed the fixup_determination_of_profiles branch 2 times, most recently from 04c5719 to 2b70930 Compare December 22, 2024 06:23
Currently a configured profile is unconditionally replaced by a better
one of the current JVM, but this is wrong if not explicitly allowed by
the configuration.

This is now changed, to be only used if ignoring the BREE is actually
enabled.
@laeubi laeubi force-pushed the fixup_determination_of_profiles branch from 2b70930 to 0d5ceee Compare December 22, 2024 06:26
Copy link

github-actions bot commented Dec 22, 2024

Test Results

  603 files  ±0    603 suites  ±0   4h 6m 26s ⏱️ - 12m 35s
  431 tests  - 1    416 ✅  -  8   7 💤 ±0  0 ❌  - 1   8 🔥 + 8 
1 293 runs   - 3  1 247 ✅  - 26  22 💤 ±0  0 ❌  - 1  24 🔥 +24 

For more details on these errors, see this check.

Results for commit 250b3e0. ± Comparison against base commit 674ca89.

This pull request removes 2 and adds 1 tests. Note that renamed tests count towards both.
org.eclipse.tycho.test.eeProfile.Java7ResolutionTest ‑ testP2ResolutionWithLowerBREEThanRequiredBundle
org.eclipse.tycho.test.eeProfile.Java7ResolutionTest ‑ testRepositoryAggregationForJava7
org.eclipse.tycho.test.eeProfile.Java7ResolutionTest ‑ Unknown test

♻️ This comment has been updated with latest results.

@laeubi
Copy link
Member Author

laeubi commented Dec 22, 2024

I think the message is quite expected here:

[ERROR] org.eclipse.core.jobs 3.15.400.v20240619-0602 requires Execution Environment that matches (&(osgi.ee=JavaSE)(version=17)) but the current resolution context uses [a.jre.javase 1.8.0]

so we probably should update the tests here and then apply the change even to Tycho 4

@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant