renechoi commented on PR #13081:
URL: https://github.com/apache/maven/pull/13081#issuecomment-5601557357

   Maven 4 already has a relocation SPI: `MavenArtifactRelocationSource` 
(`distributionManagement`, `userProperties`), walked with cycle detection in 
`DefaultArtifactDescriptorReader.loadPom`. This adds a second walker for 
`distributionManagement` only, and imports resolve via `DefaultModelResolver`, 
which never consults those sources, so `maven.relocations.entries` cannot reach 
an imported BOM. Intended?
   
   Also a relocation-only loop reports "dependencies of type=pom and with 
scope=import form a cycle" though one import is declared; `loadPom` words it 
"Artifact relocations form 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]

Reply via email to