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

Support new GAV of plexus-build-api #944

Open
laeubi opened this issue Sep 27, 2022 · 7 comments
Open

Support new GAV of plexus-build-api #944

laeubi opened this issue Sep 27, 2022 · 7 comments
Assignees
Labels
sponsored usability Improvements that can give a better usability or user experience

Comments

@laeubi
Copy link
Member

laeubi commented Sep 27, 2022

See codehaus-plexus/plexus-build-api@9c2e013

codehaus-plexus/plexus-build-api#21

We could simply have both APIs supported.

@HannesWell
Copy link
Contributor

See codehaus-plexus/plexus-build-api@9c2e013

codehaus-plexus/plexus-build-api#21

We could simply have both APIs supported.

The dependency can be added/changed here (just in case you didn't already know :) ):

<groupId>org.sonatype.plexus</groupId>

If the fully-qualified names did not change we have to or can use that as drop in replacement.

@laeubi laeubi added the usability Improvements that can give a better usability or user experience label Dec 16, 2022
laeubi added a commit to laeubi/m2e-core that referenced this issue Mar 9, 2023
@laeubi laeubi self-assigned this Mar 9, 2023
laeubi added a commit to laeubi/m2e-core that referenced this issue Mar 9, 2023
laeubi added a commit to laeubi/m2e-core that referenced this issue Mar 9, 2023
laeubi added a commit to laeubi/m2e-core that referenced this issue Mar 9, 2023
@kwin
Copy link
Member

kwin commented May 22, 2023

@laeubi Is something still missing in master...laeubi:m2e-core:issue_944_new_plexus_api or can we include this in the upcoming release?

@laeubi
Copy link
Member Author

laeubi commented May 22, 2023

It is at least missing that I need to finish/finalize that :-)

@laeubi
Copy link
Member Author

laeubi commented May 26, 2023

@gnodet @kwin if you like to speed this up, the most critical part (for me) currently is that we do not have good test-coverage but high impact if we break things.
So if we can have some test in https://github.com/eclipse-m2e/m2e-core/tree/master/org.eclipse.m2e.core.tests that cover the functionality this would help a lot, I already tried before getting an integration-test with a dummy-maven plugin but can't get it reliable working that a plugin compiled in the current reactor is used in a test.

@gnodet
Copy link

gnodet commented May 26, 2023

@laeubi the branch does not build anymore from the command line.
I end up with

[ERROR] Failed to execute goal org.eclipse.tycho.extras:tycho-p2-extras-plugin:4.0.0-SNAPSHOT:compare-version-with-baselines (check-no-version-regression) on project org.eclipse.m2e.model.edit: Version have moved backwards for (org.eclipse.m2e.model.edit/2.0.200.20230130-0743). Baseline has 2.0.300.20230320-2130) with delta: 0.0.-100 -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.eclipse.tycho.extras:tycho-p2-extras-plugin:4.0.0-SNAPSHOT:compare-version-with-baselines (check-no-version-regression) on project org.eclipse.m2e.model.edit: Version have moved backwards for (org.eclipse.m2e.model.edit/2.0.200.20230130-0743). Baseline has 2.0.300.20230320-2130) with delta: 0.0.-100

but there's no references to either version in the source tree.

@gnodet
Copy link

gnodet commented May 26, 2023

Also and fwiw, the project does not load correctly in IntelliJ (all the Tyco projects are not recognised) nor Eclipse Java (class path does not seem to be initialised correctly).

I've done quite a lof to OSGi development in the past, but not using Tycho, Eclipse or M2e, so I'm not familiar with the whole setup.

@laeubi
Copy link
Member Author

laeubi commented May 26, 2023

M2E is an eclipse project so Eclipse IDE is somehow a requirement (even though it might work to some degree with IntelliJ).

You can setup a Dev-IDE here:
https://github.com/eclipse-m2e/m2e-core/blob/master/CONTRIBUTING.md#%EF%B8%8F-setting-up-the-development-environment-automatically-using-the-eclipse-installer-oomph

Regarding the branch, the best would be to just start from the master and as a first step add a test-case before try to apply any of the changes this can then be merged already as a prerequisite and will show that the dev-workflow works already.

If we have this I can rebase my branch ontop of this to see if it already breaks something, and add support for the new GAV.

gnodet referenced this issue in pkriens/bnd Jun 1, 2023
The current  Eclipse we're using is still based on the sonatype code base.

In Eclipse, with ecj, the patch generates an error that it clearly does not
do on javac/gradle.

@gnodet, can you take a look?

This reverts commit 2443973.

---
 Signed-off-by: Peter Kriens <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sponsored usability Improvements that can give a better usability or user experience
Projects
None yet
Development

No branches or pull requests

4 participants