gnodet commented on code in PR #12657:
URL: https://github.com/apache/maven/pull/12657#discussion_r3698616679
##########
pom.xml:
##########
@@ -1078,6 +1078,16 @@ under the License.
<artifactId>maven-resolver-tools</artifactId>
<version>${resolverVersion}</version>
</dependency>
+ <dependency>
+ <groupId>org.apache.maven</groupId>
Review Comment:
Adding `maven-core` with `${project.version}` as a plugin dependency in the
`graph` profile of the root pom introduces a reactor cycle
(`maven-api-annotations → maven-api → maven → maven-core →
maven-api-annotations`). Since the `graph` profile auto-activates on
non-Windows, this breaks all Maven commands on a clean checkout.
Consider relocating the page generation step to a leaf module (e.g.,
`apache-maven`) that can depend on `maven-core` without creating a cycle.
--
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]