[GitHub] [maven] rfscholte commented on pull request #544: [MNG-7253] Display relocation message defined in model

2021-09-17 Thread GitBox
rfscholte commented on pull request #544: URL: https://github.com/apache/maven/pull/544#issuecomment-921828962 It makes the addition of message to the ResolcationArtifact absolete. The title of the ticket still reflects the goal: Display relocation message as defined in the pom. -- Th

[GitHub] [maven] rfscholte commented on pull request #544: [MNG-7253] Display relocation message defined in model

2021-09-17 Thread GitBox
rfscholte commented on pull request #544: URL: https://github.com/apache/maven/pull/544#issuecomment-921816524 > The entire logging? The relocation warning (which is basically everything apart from the visitEnter debug logging) -- This is an automated message from the Apache G

[GitHub] [maven] rfscholte commented on pull request #544: [MNG-7253] Display relocation message defined in model

2021-09-17 Thread GitBox
rfscholte commented on pull request #544: URL: https://github.com/apache/maven/pull/544#issuecomment-921591731 Why not move the logging to `DefaultArtifactDescriptorReader.java`? To me `relocation` is the mechanism to relocate and that's where the mesage belongs. the `relocationArtifact` i