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

Plugin Maven 4 Compatibility #1497

Closed
mikerimov opened this issue Jul 17, 2023 · 1 comment
Closed

Plugin Maven 4 Compatibility #1497

mikerimov opened this issue Jul 17, 2023 · 1 comment
Labels
declined: duplicate ❌ This issue or pull request already exists type: enhancement

Comments

@mikerimov
Copy link

  • JaCoCo version: 3.10.0
  • Operating system: OS X 13.4.1
  • Tool integration: Maven
  • Maven 3.9.2 is emitting the following warnings about jacoco-maven-plugin expressing dire warnings when maven 4 is released

Maven Output:

[WARNING] Plugin validation issues were detected in 1 plugin(s)
[WARNING] 
[WARNING]  * org.jacoco:jacoco-maven-plugin:0.8.10

snip for brevity

[WARNING]   Plugin issue(s):
[WARNING]    * Plugin is a Maven 2.x plugin, which will be not supported in Maven 4.x
[WARNING]    * Plugin mixes multiple Maven versions: [3.0, 2.0.2]
[WARNING]    * Plugin should declare these Maven artifacts in `provided` scope: [org.apache.maven:maven-repository-metadata:3.0, org.apache.maven:maven-artifact:3.0]
[WARNING]    * Plugin depends on plexus-container-default, which is EOL

Since jacoco-maven-plugin docs clearly state that maven 3 is required, I don't see any reason you couldn't begin pruning the maven2 required bits now rather than waiting for the bugs to come in later.

Replication. Extract the following pom.xml zip (simple file) and run mvn install -Dmaven.plugin.validation=VERBOSE

pom.xml.zip

@Godin
Copy link
Member

Godin commented Jul 17, 2023

Duplicate of #1435

@Godin Godin marked this as a duplicate of #1435 Jul 17, 2023
@Godin Godin closed this as completed Jul 17, 2023
@Godin Godin added the declined: duplicate ❌ This issue or pull request already exists label Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
declined: duplicate ❌ This issue or pull request already exists type: enhancement
Projects
None yet
Development

No branches or pull requests

2 participants