[GitHub] [maven] gnodet commented on pull request #1228: resolve Circular dependencies for project org.apache.maven:maven-xml-impl and rename name

["gnodet (via GitHub)" <gi...@apache.org>]

gnodet commented on PR #1228:
URL: https://github.com/apache/maven/pull/1228#issuecomment-1713728163

   > I agree that there is a problem
   > I don't think the current PR is the right approach
   > as found [on the ML while discussing on the topic](https://lists.apache.org/thread/8t5szmn8fhlrklhg5dgv496k26tz3w3p), I thin the approach is to create a Maven Shared Component that provides MetaAnnotations, XML API and XML Implementation separately from Maven core
   > I don't know how to name this release: "Maven fundamentals"?
   
   While I'm not opposed to moving things, I don't think this will solve the problem.
   The problem here is the cycle between maven-xml-impl / plexus-xml / maven-xml-impl.
   Moving maven-xml-impl in another location won't change that problem, so while this may not be a sufficient solution, I think this PR is still needed.


-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org