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

Adopt new o.e.e.p2.publisher.eclipse API for IMacOsBundleUrlType #4248

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

sratz
Copy link
Contributor

@sratz sratz commented Sep 10, 2024

After #4246, we can now consume the new

org.eclipse.equinox.internal.p2.publisher.eclipse.IProductDescriptor.getMacOsBundleUrlTypes()

API in the Tycho wrapper
org.eclipse.tycho.p2.tools.publisher.ExpandedProduct.

This is necessary to properly support link handler registrations in product's launchers.

See
eclipse-platform/eclipse.platform.ui#1901 eclipse-equinox/p2#533
for more information.

After eclipse-tycho#4246, we can now consume the new

org.eclipse.equinox.internal.p2.publisher.eclipse.IProductDescriptor.getMacOsBundleUrlTypes()

API in the Tycho wrapper
org.eclipse.tycho.p2.tools.publisher.ExpandedProduct.

This is necessary to properly support link handler registrations in
product's launchers.

See
eclipse-platform/eclipse.platform.ui#1901
eclipse-equinox/p2#533
for more information.
Copy link

Test Results

  600 files    600 suites   4h 23m 49s ⏱️
  429 tests   422 ✅  7 💤 0 ❌
1 287 runs  1 265 ✅ 22 💤 0 ❌

Results for commit 59c3d64.

@sratz
Copy link
Contributor Author

sratz commented Sep 10, 2024

@laeubi:
Can we also get this to 4.0.x? That would require to
bump org.eclipse.platform:org.eclipse.equinox.p2.publisher.eclipse from 1.6.0 to 1.6.200
there, too. Do you see any problem with that?

@laeubi
Copy link
Member

laeubi commented Sep 10, 2024

Just create a PR with required adjustment against Tycho 4.x branch.

@laeubi
Copy link
Member

laeubi commented Sep 10, 2024

Should we merge or is anything left?

@sratz sratz merged commit a74c58c into eclipse-tycho:main Sep 10, 2024
10 of 11 checks passed
@sratz sratz deleted the macos-bundleurltypes branch September 10, 2024 16:09
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