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

Always use full-named o.e.equinox.launcher artifact #519

Merged

Conversation

HannesWell
Copy link
Member

For for Mac OS X there was no equinox-launcher fragment for CARBON with full name, i.e. with .. suffix, but only a org.eclipse.equinox.launcher.carbon.macosx fragment. But since Eclipse 4.2, released 2012, CARBON is not supported anymore at all [1]. Therefore the code adjust to that special naming schema, which also imposed difficulties or new supported architectures on Macos should be removed.

See also eclipse-equinox/equinox#622

[1] - https://download.eclipse.org/eclipse/updates/4.2/R-4.2-201206081400

@HannesWell HannesWell requested review from merks and laeubi May 19, 2024 10:44
@HannesWell
Copy link
Member Author

@merks, @laeubi do you think this is something we should have for the upcoming summer release?
This would then probably allow us to remove the short-named launcher fragment for mac (see eclipse-equinox/equinox#622). But I would leave the actual removal for the next release-cycle.

Copy link

github-actions bot commented May 19, 2024

Test Results

  251 files  + 1    251 suites  +1   28m 39s ⏱️ - 1m 9s
1 887 tests ± 0  1 884 ✅ ± 0  3 💤 ±0  0 ❌ ±0 
4 482 runs  +26  4 476 ✅ +26  6 💤 ±0  0 ❌ ±0 

Results for commit b25e3a7. ± Comparison against base commit 5073d13.

♻️ This comment has been updated with latest results.

For for Mac OS X there was no equinox-launcher fragment for CARBON with
full name, i.e. with <ws>.<os>.<arch> suffix, but only a
org.eclipse.equinox.launcher.carbon.macosx fragment.
But since Eclipse 4.2, released 2012, CARBON is not supported anymore at
all [1]. Therefore the code adjust to that special naming schema, which
also imposed difficulties or new supported architectures on Macos should
be removed.

See also eclipse-equinox/equinox#622

[1] - https://download.eclipse.org/eclipse/updates/4.2/R-4.2-201206081400
@HannesWell HannesWell force-pushed the fully-named-equinox-launchers branch from 78cff17 to b25e3a7 Compare May 19, 2024 11:17
Copy link
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

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

It's rather weird and inconsistent in the current state:

image

So indeed it seems better to fix it sooner rather than later.


This logic is used primarily by Tycho so when Tycho starts using the new version then you can get rid of the older short name. As a result, when you get rid if it, everyone building products against the latest platform builds must use a new release to Tycho, or their builds will fail, right?

@HannesWell
Copy link
Member Author

HannesWell commented May 19, 2024

So indeed it seems better to fix it sooner rather than later.

Agree.

As a result, when you get rid if it, everyone building products against the latest platform builds must use a new release to Tycho, or their builds will fail, right?

That's right. But I think in general it's hard to expect/ensure that old tools work with new content because we can't predict the future. For example PDE warns if the TP content is newer than the runtime.
But to have at least a three-months transition period I think it would be best to have this in now and remove the short-named launcher fragment for the next release.

This logic is used primarily by Tycho so when Tycho starts using the new version then you can get rid of the older short name.

For the record, here is the pendant in eclipse-tycho/tycho#3860. I have to admit that I'm not absolutely sure which change will be the one that influence the product assemble, but I assume its this one.

@HannesWell HannesWell merged commit 68bad32 into eclipse-equinox:master May 19, 2024
11 checks passed
@HannesWell HannesWell deleted the fully-named-equinox-launchers branch May 19, 2024 20:13
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.

2 participants