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

Add an option to the product editor to flag it contain a JRE #1075

Merged
merged 1 commit into from
Jan 27, 2024

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Jan 26, 2024

Currently if one wants to include a JRE it is needed to specify a concrete EE for each of the linux/win/mac what is quite cumbersome and could easily forgotten to update.

Instead this adds a new flag "includeJRE" in the product that enables this for all os at once and always uses the default vm (what is the one from the target platform or user chosen default).

This will also make it easier for Tools like Tycho that already investigate the 'includeLaunchers' flag.

grafik

Copy link

github-actions bot commented Jan 26, 2024

Test Results

   279 files  +  4     279 suites  +4   1h 10m 31s ⏱️ + 12m 1s
 3 498 tests ±  0   3 440 ✅ +  1   58 💤  -  1  0 ❌ ±0 
10 791 runs  +245  10 615 ✅ +193  176 💤 +52  0 ❌ ±0 

Results for commit 890bea0. ± Comparison against base commit 2da79f1.

♻️ This comment has been updated with latest results.

Copy link
Member

@HannesWell HannesWell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general this looks good.

But the .api_filters are unrelated, aren't they? Please add them in a separate PR, if necessary.

@laeubi
Copy link
Contributor Author

laeubi commented Jan 27, 2024

But the .api_filters are unrelated, aren't they? Please add them in a separate PR, if necessary.

See #1077

Currently if one wants to include a JRE it is needed to specify a
concrete EE for each of the linux/win/mac what is quite cumbersome and
could easily forgotten to update.

Instead this adds a new flag "includeJRE" in the product that enables
this for all os at once and always uses the default vm (what is the one
from the target platform or user chosen default).

This will also make it easier for Tools like Tycho that already
investigate the 'includeLaunchers' flag.
@laeubi laeubi merged commit b078db2 into eclipse-pde:master Jan 27, 2024
17 checks passed
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.

Fix eclipse-tycho#2064
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.

Fix eclipse-tycho#2064
laeubi added a commit to eclipse-tycho/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.

Fix #2064
laeubi added a commit to laeubi/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.

Fix eclipse-tycho#2064
laeubi added a commit to eclipse-tycho/tycho that referenced this pull request May 5, 2024
PDE recently added a new flag for the product to mark it to include a
jre eclipse-pde/eclipse.pde#1075 but this
currently has no effect in Tycho.

This now adds support for automatically include a JustJ JRE if the
option is selected.

Fix #2064
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.

None yet

2 participants