Skip to content

enable build with Maven 4#1311

Open
hboutemy wants to merge 1 commit into
masterfrom
maven4-enabled
Open

enable build with Maven 4#1311
hboutemy wants to merge 1 commit into
masterfrom
maven4-enabled

Conversation

@hboutemy

@hboutemy hboutemy commented May 11, 2026

Copy link
Copy Markdown
Member
[INFO] -------------------------------------------------
[INFO] Build Summary:
[INFO]   Passed: 139, Failed: 11, Errors: 0, Skipped: 0
[INFO] -------------------------------------------------
Error:  The following builds failed:
Error:  *  projects/container-descriptors/custom-handler-withMixedEntries/pom.xml
Error:  *  projects/file-sets/multimodule-win-lineEndings/pom.xml
Error:  *  projects/dependency-sets/massembly-969/pom.xml
Error:  *  projects/container-descriptors/custom-containerDescriptorHandler/pom.xml
Error:  *  projects/mojo-configuration/massembly-291/pom.xml
Error:  *  projects/mojo-configuration/massembly-301/pom.xml
Error:  *  projects/file-sets/filtered-module-sources/pom.xml
Error:  *  projects/container-descriptors/custom-handler-withFileEntries/pom.xml
Error:  *  projects/dependency-sets/massembly-1008/pom.xml
Error:  *  projects/file-sets/filtered-multimodule/pom.xml
Error:  *  projects/bugs/massembly-891/pom.xml
[INFO] -------------------------------------------------

@hboutemy

Copy link
Copy Markdown
Member Author

last IT failure fixed by #1320 , as it was also failing with Maven 3.10.0-SNAPSHOT

remaining ITs failures are specific to Maven 4

@hboutemy

Copy link
Copy Markdown
Member Author

running now with Maven 4.0.0-SNAPSHOT gives me

[INFO] -------------------------------------------------
[INFO] Build Summary:
[INFO]   Passed: 146, Failed: 4, Errors: 0, Skipped: 0
[INFO] -------------------------------------------------
[ERROR] The following builds failed:
[ERROR] *  projects/dependency-sets/massembly-1008/pom.xml
[ERROR] *  projects/mojo-configuration/massembly-291/pom.xml
[ERROR] *  projects/mojo-configuration/massembly-301/pom.xml
[ERROR] *  projects/dependency-sets/massembly-969/pom.xml
[INFO] -------------------------------------------------

@Bukama

Bukama commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

An old issue about same topic (failing ITs with Maven 4): #1257

@hboutemy

Copy link
Copy Markdown
Member Author
[ERROR] *  projects/mojo-configuration/massembly-291/pom.xml
[ERROR] *  projects/mojo-configuration/massembly-301/pom.xml

are caused by the same interpolation issue from test.properties as what was fixed in apache/maven-deploy-plugin@dd96008

@Bukama

Bukama commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

projects\dependency-sets\massembly-969

fails with invalid model

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:3.8.1-SNAPSHOT:single (assembly) on project massembly-969: Execution assembly of goal org.apache.maven.plugins:maven-assembly-plugin:3.8.1-SNAPSHOT:single failed: Duplicate key linux-x86_64 (attempted merging values org.apache.maven.api.model.Activation@4ea48b2e and org.apache.maven.api.model.Activation@72c704f1) -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:3.8.1-SNAPSHOT:single (assembly) on project massembly-969: Execution assembly of goal org.apache.maven.plugins:maven-assembly-plugin:3.8.1-SNAPSHOT:single failed: Duplicate key linux-x86_64 (attempted merging values org.apache.maven.api.model.Activation@4ea48b2e and org.apache.maven.api.model.Activation@72c704f1)

Guess this comes from all the FX dependencies for all the different OS and they all bring in their profile activation definition. Not yet sure how to fix this, except different ITs with only one dependency per OS. Will think about dinner first 🍽️

@Bukama

Bukama commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

projects\dependency-sets\massembly-969 fixed by updating dependency used in test (tested on Maven 3.9.16 and Maven 4.0.0-SNAPSHOT)

@Bukama

Bukama commented Jun 13, 2026

Copy link
Copy Markdown
Contributor
[ERROR] *  projects/mojo-configuration/massembly-291/pom.xml
[ERROR] *  projects/mojo-configuration/massembly-301/pom.xml

are caused by the same interpolation issue from test.properties as what was fixed in apache/maven-deploy-plugin@dd96008

applied

# under the License.

altDeploymentRepository=test::default::file://${project.build.directory}/deploy
altDeploymentRepository=test::default::file:./target/repo/deploy

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't add repo that was not existent before: this will remove unwanted impact

@hboutemy

hboutemy commented Jun 14, 2026

Copy link
Copy Markdown
Member Author

I don't get how we can have so many fixes, and latest build on GHA says with Maven 4.0.0-RC5:

[INFO] -------------------------------------------------
[INFO] Build Summary:
[INFO]   Passed: 144, Failed: 6, Errors: 0, Skipped: 0
[INFO] -------------------------------------------------
Error:  The following builds failed:
Error:  *  projects/container-descriptors/custom-handler-withMixedEntries/pom.xml
Error:  *  projects/file-sets/multimodule-win-lineEndings/pom.xml
Error:  *  projects/container-descriptors/custom-containerDescriptorHandler/pom.xml
Error:  *  projects/file-sets/filtered-module-sources/pom.xml
Error:  *  projects/container-descriptors/custom-handler-withFileEntries/pom.xml
Error:  *  projects/file-sets/filtered-multimodule/pom.xml
[INFO] -------------------------------------------------

what is happening?
were the ITs rebased on master, to benefit from other fixes done in master?

@hboutemy

Copy link
Copy Markdown
Member Author

just tested on my laptop: got it

works with Maven 4.0.0-SNAPSHOT, failures are specific to 4.0.0-RC5

great news: RC6 has fixed issues from RC5

@Bukama it seems you'll have to decouple the ITs improvements from the current PR, as enabling Maven 4 here will require rc6

@Bukama

Bukama commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Reverted all my commits, so this branch is back to only enable Maven 4 build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants