Skip to content

Prepare for Maven 4 (#2322)#2588

Merged
holgerfriedrich merged 1 commit intoapache:karaf-4.4.xfrom
holgerfriedrich:pr-maven4-44x
Apr 25, 2026
Merged

Prepare for Maven 4 (#2322)#2588
holgerfriedrich merged 1 commit intoapache:karaf-4.4.xfrom
holgerfriedrich:pr-maven4-44x

Conversation

@holgerfriedrich
Copy link
Copy Markdown
Contributor

Backport of #2322

This basically allows building Karaf with maven4-rc5.

First addition is the directory .mvn to mark the top directory of the workspace. Not strictly necessary, but it suppresses the warning about missing top level directory marker during build with Maven 4.
The approach to add .mvn dir is the option that does not impact our standard builds with Maven 3.

The second addition is a missing version specification in a pom file. Without it, Maven4 just picks the wrong version of the maven-resources-plugin, which is built against an old API and is causing the build to fail.

@holgerfriedrich
Copy link
Copy Markdown
Contributor Author

holgerfriedrich commented Apr 25, 2026

@jbonofre a nice to have on 4.4.x, I would appreciate if I could stay at Maven 4 😎
At least the compilation fail is unfortunate and may affect others.

@jbonofre jbonofre self-requested a review April 25, 2026 11:28
@jbonofre
Copy link
Copy Markdown
Member

@holgerfriedrich it makes sense. As the change is pretty minimal, it's totally fine to have it for 4.4.x.

@holgerfriedrich holgerfriedrich merged commit e47603a into apache:karaf-4.4.x Apr 25, 2026
11 of 12 checks passed
@holgerfriedrich holgerfriedrich deleted the pr-maven4-44x branch April 25, 2026 13:07
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