Skip to content

Fix mvnup PLUGIN_UPGRADES for compiler and exec plugins#12172

Merged
gnodet merged 1 commit into
apache:masterfrom
gnodet:thorn-club
Jun 4, 2026
Merged

Fix mvnup PLUGIN_UPGRADES for compiler and exec plugins#12172
gnodet merged 1 commit into
apache:masterfrom
gnodet:thorn-club

Conversation

@gnodet

@gnodet gnodet commented May 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fixed maven-compiler-plugin version from 3.2.0 to 3.2 in PLUGIN_UPGRADES — version 3.2.0 does not exist on Maven Central (confirmed via HTTP 404), causing build resolution failures in projects like servicemix4-bundles and synapse
  • Fixed exec-maven-plugin entry: corrected groupId from org.apache.maven.plugins to org.codehaus.mojo and artifactId from maven-exec-plugin to exec-maven-plugin, making PLUGIN_UPGRADES consistent with getPluginUpgradesMap()

Test plan

  • Verified maven-compiler-plugin:3.2 exists on Maven Central (HTTP 200)
  • Verified maven-compiler-plugin:3.2.0 does NOT exist on Maven Central (HTTP 404)
  • Verified exec-maven-plugin:3.2.0 exists on Maven Central (HTTP 200)
  • Updated tests to match corrected artifact IDs
  • All 27 PluginUpgradeStrategyTest tests pass

🤖 Generated with Claude Code

@Bukama Bukama left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't we upgrade the plugin versions to a higher version by default?

maven-compiler-plugin was listed as 3.2.0 but Maven Central only has 3.2,
causing resolution failures. exec-maven-plugin had wrong groupId
(org.apache.maven.plugins) and artifactId (maven-exec-plugin), inconsistent
with getPluginUpgradesMap() which correctly uses org.codehaus.mojo:exec-maven-plugin.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gnodet gnodet merged commit c7d96d3 into apache:master Jun 4, 2026
1 check passed
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

@gnodet Please assign appropriate label to PR according to the type of change.

gnodet added a commit that referenced this pull request Jun 4, 2026
…ns (#12172) (#12225)

maven-compiler-plugin was listed as 3.2.0 but Maven Central only has 3.2,
causing resolution failures. exec-maven-plugin had wrong groupId
(org.apache.maven.plugins) and artifactId (maven-exec-plugin), inconsistent
with getPluginUpgradesMap() which correctly uses org.codehaus.mojo:exec-maven-plugin.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants