fridrich commented on PR #13259:
URL: https://github.com/apache/maven/pull/13259#issuecomment-5809607318

   I have another patch that would remove the dependency from maven-core by 
restoring back the 
maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/MavenSessionBuilderSupplier.java
 the same way maven 4 keeps them. That would completely remove the cycle and 
the graph would be:
   ```text
   [maven-resolver]
       (core)
          │
          ▼
       [maven]
          │
          ▼
   [maven-resolver-supplier-mvn3]
   ```
   But this one would partially revert Tamas' commit 
27f7b546886b9d8fd6c7695523bb544db9eeed26 and I did not want to join the two 
together. Eventually, if you are opened towards it, I could submit as a 
follow-up PR.


-- 
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]

Reply via email to