gnodet opened a new pull request, #12430:
URL: https://github.com/apache/maven/pull/12430
## Summary
- Add `PluginUpgrade` entries for `quarkus-maven-plugin` (both `io.quarkus`
and `io.quarkus.platform` groupIds) with minimum version 3.26.0 and reason
"Maven 4 compatibility (Aether API changes)"
- When the plugin version references a property shared with a Quarkus BOM in
`dependencyManagement`, introduce a separate `quarkus-plugin.version` property
instead of modifying the shared BOM property
- Emit a warning when the platform BOM version is significantly older than
the plugin version after upgrade
## Details
Quarkus plugin versions < 3.26.0 use `smallrye-beanbag` to bootstrap the
Maven Resolver, which is incompatible with Maven 4's reorganized Aether API.
The fix was landed in Quarkus 3.26.0 (quarkusio/quarkus#46889).
Many Quarkus projects reuse a single property (e.g.
`${quarkus.platform.version}`) for both the platform BOMs and the plugin
version. When mvnup bumps the plugin version, it must not change the BOM
version — doing so would alter the entire dependency tree. The decoupling logic
detects this scenario and introduces a new property.
The version gap warning advises users that the plugin cannot be upgraded
beyond 3.31.x without a corresponding platform upgrade (due to binary
incompatibility in `SerializedApplication.write()` starting from 3.32.0).
## Test plan
- [x] Test basic upgrade of `io.quarkus:quarkus-maven-plugin` from old
version to 3.26.0
- [x] Test basic upgrade of `io.quarkus.platform:quarkus-maven-plugin` from
old version to 3.26.0
- [x] Test no upgrade when version is already >= 3.26.0
- [x] Test BOM property decoupling when property is shared with Quarkus BOM
- [x] Test no decoupling when plugin has its own dedicated property
- [x] Test no decoupling when BOM version is already sufficient
- [x] Test version gap warning emission
- [x] Test plugin upgrade in pluginManagement section
- [x] Test predefined plugin upgrade list includes both groupIds
- [x] All existing PluginUpgradeStrategy tests pass (40 total)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
