slachiewicz opened a new pull request, #835:
URL: https://github.com/apache/maven-shade-plugin/pull/835
`70f54a2` ("Guice isn't needed (#752)") added a test-scoped
`org.eclipse.sisu.plexus` declaration under the `<!-- Test -->` marker, but the
same dependency was already declared a few lines further down.
Maven 4 treats a duplicate
`dependencies.dependency.(groupId:artifactId:type:classifier)` as a
model-building **error** rather than a warning, so `master` no longer loads at
all under Maven 4:
```
[ERROR] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must
be unique:
org.eclipse.sisu:org.eclipse.sisu.plexus:jar -> duplicate
declaration of version (?)
@ line 217, column 5
```
Reproduced with both `4.0.0-rc-5` and `4.0.0-rc-6`; `mvn validate` fails
before any module builds. In a multi-module reactor that includes this plugin,
the error propagates to the aggregator and nothing builds.
This drops the older bare declaration and keeps the one carrying the `Used
by: TransformerTesterRule only` comment, so the dependency itself is unchanged.
`mvn validate` passes on 4.0.0-rc-6 afterwards. Nothing changes for Maven 3,
where this was only a warning.
🤖 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]
