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

Enable the Tycho Baselineplugin for m2e #1116

Merged
merged 2 commits into from
Dec 12, 2022

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Dec 3, 2022

This enables the new tycho-baseline-plugin for m2e to see how/if this works out and where Tycho must improve and test it on a real world example.

@github-actions
Copy link

github-actions bot commented Dec 3, 2022

Test Results

610 tests  ±0   603 ✔️ ±0   9m 16s ⏱️ -42s
  98 suites ±0       7 💤 ±0 
  98 files   ±0       0 ±0 

Results for commit e361aa8. ± Comparison against base commit bca1a06.

♻️ This comment has been updated with latest results.

@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch 4 times, most recently from c68882a to 4a70c7e Compare December 3, 2022 17:04
@laeubi
Copy link
Member Author

laeubi commented Dec 3, 2022

@HannesWell
Copy link
Contributor

HannesWell commented Dec 3, 2022

Looks good 😃

But I think it would be good to split this into two commits, one to update to Tycho 4.0.0-SNAPSHOT+build adjustments and the second one to introduce the baseline repo. Just to have concerns a bit separated.

Revealed a bug in Tycho 4.0

* [InstallableUnitGenerator must check for null manifest eclipse-tycho/tycho#1792](https://github.com/eclipse-tycho/tycho/pull/1792)

I wonder which jar in the M2E build does not have a Manifest?

Ah ok, check the logs, it was a dependency^^

@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch 2 times, most recently from 3fa8fe1 to eed4b0e Compare December 5, 2022 05:50
@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch 2 times, most recently from 5ec6dfc to 69916ae Compare December 9, 2022 13:09
@laeubi
Copy link
Member Author

laeubi commented Dec 9, 2022

@HannesWell any idea why the build fails with version complaint?

@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch 3 times, most recently from d6b8a5c to c5d6d68 Compare December 9, 2022 19:34
@HannesWell
Copy link
Contributor

@HannesWell any idea why the build fails with version complaint?

Should work again now, with #1130.

@HannesWell HannesWell force-pushed the use_tycho_baseline_plugin branch from c5d6d68 to 913ade0 Compare December 10, 2022 13:06
@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch 2 times, most recently from 9931e14 to 210d905 Compare December 10, 2022 15:01
@HannesWell HannesWell force-pushed the use_tycho_baseline_plugin branch from 210d905 to 51c2865 Compare December 11, 2022 08:33
@laeubi laeubi force-pushed the use_tycho_baseline_plugin branch from 51c2865 to 512ce55 Compare December 11, 2022 15:19
- org.eclipse.m2e.lemminx.feature has changed license text
- org.eclipse.m2e.pde.feature has upgraded bnd lib
- org.eclipse.m2e.sdk.feature includes the changed pde feature
@laeubi
Copy link
Member Author

laeubi commented Dec 12, 2022

I assume the test-failure is not really related to the maven build so we can merge this, all Tycho changes should be available now!

@laeubi laeubi merged commit 436a70d into eclipse-m2e:master Dec 12, 2022
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