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

Remove publish-osgi-ee from the default life-cycle #3568

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Mar 11, 2024

This serves both as a reminder as well as to track progress with the fact that the osgi-ee in P2 repositories is actually not useful.

Required tasks:

  • Check that Tycho can work without published EE
  • Check that such updatesites can be consumed by PDE / P2
  • Write release notes with migration for Tycho 4 > 5 that explains how to restore old behavior

@laeubi laeubi added this to the 5.0 milestone Mar 11, 2024
Copy link

github-actions bot commented Mar 11, 2024

Test Results

  603 files  ±0    603 suites  ±0   4h 13m 24s ⏱️ + 2m 7s
  431 tests ±0    418 ✅  -  6   7 💤 ±0  2 ❌ +2   4 🔥 + 4 
1 293 runs  ±0  1 253 ✅  - 18  22 💤 ±0  6 ❌ +6  12 🔥 +12 

For more details on these failures and errors, see this check.

Results for commit 2292cde. ± Comparison against base commit db1d038.

♻️ This comment has been updated with latest results.

@sratz
Copy link
Contributor

sratz commented Apr 30, 2024

@laeubi
Copy link
Member Author

laeubi commented Apr 30, 2024

That is something different, but the JustJ story is also on my table.

@laeubi laeubi force-pushed the remove_publishee branch from a666210 to 3f42991 Compare May 4, 2024 17:02
@laeubi
Copy link
Member Author

laeubi commented May 5, 2024

@sratz I now have created:

this will allow to include JustJ in a product without any special / additional configuration or workarounds.

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