Sitelet https://github.com/apache/maven/pull/13016
Skip to content

Fix #12991: skip shade-plugin upgrade when custom ResourceTransformers are present (backport to 4.0.x) - #13016

Merged
gnodet merged 2 commits into
maven-4.0.xfrom
backport/12991-shade-plugin-4.0.x
Sep 2, 2026
Merged

gnodet merged 2 commits into
maven-4.0.xfrom
backport/12991-shade-plugin-4.0.x

Conversation

@gnodet

@gnodet gnodet commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Backport of #12997 to maven-4.0.x.

  • mvnup detects custom ResourceTransformer classes in shade-plugin configuration and skips the upgrade when found
  • Per-module check in multi-module builds — unaffected modules can still be upgraded
  • Warning logged with the list of custom transformers found

Test plan

  • All 11 shade-plugin tests pass on 4.0.x branch
  • CI passes

🤖 Generated with Claude Code

…s are present

mvnup upgrades maven-shade-plugin to 3.5.0, but this breaks projects
using custom ResourceTransformer implementations that depend on
transitive dependencies not available in the newer version (e.g.
org.jdom:jdom used by BeansXmlTransformer in myfaces-extcdi).

This change detects custom transformer classes in shade-plugin
configuration (both top-level and per-execution) and skips the upgrade
when found, logging a warning with the list of custom transformers.
The check is per-module in multi-module builds, so unaffected modules
can still be upgraded.
@gnodet gnodet added this to the 4.0.0-rc-7 milestone Sep 2, 2026
@gnodet gnodet added bug Something isn't working backport labels Sep 2, 2026
@gnodet gnodet self-assigned this Sep 2, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gnodet
gnodet merged commit 335ad9a into maven-4.0.x Sep 2, 2026
23 checks passed
@gnodet
gnodet deleted the backport/12991-shade-plugin-4.0.x branch September 2, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant