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

[Runtime] Embed Maven 3.9.4 #1370

Merged
merged 1 commit into from
Aug 22, 2023
Merged

Conversation

HannesWell
Copy link
Contributor

@HannesWell HannesWell force-pushed the maven_3.9.2 branch 2 times, most recently from e70f28e to e2cfc94 Compare May 8, 2023 21:43
@github-actions
Copy link

github-actions bot commented May 8, 2023

Test Results

106 files  106 suites   13m 20s ⏱️
660 tests 650 ✔️ 10 💤 0
660 runs  649 ✔️ 11 💤 0

Results for commit 3458a3a.

♻️ This comment has been updated with latest results.

@HannesWell
Copy link
Contributor Author

@laeubi I think the following failure is new.
Can you tell if this is a regression in Maven 3.9.2 or if M2E just has to adapt?

Should not return a project scoped extension
java.lang.AssertionError: Should not return a project scoped extension
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.assertTrue(Assert.java:42)
	at org.eclipse.m2e.core.ExtensionsTest.testProjectExtensions(ExtensionsTest.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
	at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:61)
	at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:61)
	at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
	at org.junit.runners.BlockJUnit4ClassRunner$1.evaluate(BlockJUnit4ClassRunner.java:100)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:366)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:103)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:63)
	at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:79)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:329)
	at org.junit.runners.ParentRunner.access$100(ParentRunner.java:66)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:293)
	at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
	at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:316)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:240)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:214)
	at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:155)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.apache.maven.surefire.api.util.ReflectionUtils.invokeMethodWithArray2(ReflectionUtils.java:137)
	at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:148)
	at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:88)
	at org.eclipse.tycho.surefire.osgibooter.OsgiSurefireBooter.run(OsgiSurefireBooter.java:142)
	at org.eclipse.tycho.surefire.osgibooter.HeadlessTestApplication.start(HeadlessTestApplication.java:29)
	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:203)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:136)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:402)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:255)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:659)
	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:596)
	at org.eclipse.equinox.launcher.Main.run(Main.java:1467)
	at org.eclipse.equinox.launcher.Main.main(Main.java:1440)

@HannesWell HannesWell added this to the 2.4.0 milestone May 23, 2023
@laeubi
Copy link
Member

laeubi commented Jun 13, 2023

@laeubi I think the following failure is new. Can you tell if this is a regression in Maven 3.9.2 or if M2E just has to adapt?

Should not return a project scoped extension

I'm not aware of anything new there, this would be handled by the plexus container we configure so it should still succeed.

@HannesWell HannesWell force-pushed the maven_3.9.2 branch 2 times, most recently from 0bbe529 to c6b31c1 Compare June 16, 2023 21:57
@HannesWell
Copy link
Contributor Author

Maven 3.9.3 is on vote, lets try this. Maybe we are lucky and the problem is solved auto-magically.

@laeubi
Copy link
Member

laeubi commented Jun 27, 2023

3.9.3 is out!

@HannesWell HannesWell changed the title [Runtime] Embed Maven 3.9.2 [Runtime] Embed Maven 3.9.3 Jun 27, 2023
@HannesWell
Copy link
Contributor Author

Something is strange here. Could this be related to #1459?

@laeubi
Copy link
Member

laeubi commented Aug 16, 2023

@HannesWell Maven 3.9.4 is out!

@HannesWell HannesWell changed the title [Runtime] Embed Maven 3.9.3 [Runtime] Embed Maven 3.9.4 Aug 19, 2023
@HannesWell
Copy link
Contributor Author

Something is strange here. Could this be related to #1459?

Note to myself: The attribute to specify the lower-bound of a required bundle is bundle-version not version ...
The explains the sever errors while launching the test-runtime, the m2e.maven.runtime bundle could not have been resolved because of that error.

@HannesWell HannesWell marked this pull request as ready for review August 22, 2023 23:15
@HannesWell
Copy link
Contributor Author

HannesWell commented Aug 22, 2023

@laeubi I think the following failure is new. Can you tell if this is a regression in Maven 3.9.2 or if M2E just has to adapt?

Should not return a project scoped extension

I'm not aware of anything new there, this would be handled by the plexus container we configure so it should still succeed.

Looks like this problem has vanished auto-magically in the meantime. 🎉
So lets merge this and see how it works in the IDE.

@HannesWell HannesWell merged commit 3410597 into eclipse-m2e:master Aug 22, 2023
@HannesWell HannesWell deleted the maven_3.9.2 branch August 22, 2023 23:30
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