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 1 commit 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

Test Results

  603 files  ±0    603 suites  ±0   3h 53m 9s ⏱️ - 25m 52s
  432 tests ±0    380 ✅  -  44   7 💤 ±0  0 ❌  - 1   45 🔥 + 45 
1 296 runs  ±0  1 139 ✅  - 134  22 💤 ±0  0 ❌  - 1  135 🔥 +135 

For more details on these errors, see this check.

Results for commit 0d5ceee. ± Comparison against base commit 674ca89.

@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

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.

1 participant