michael-o commented on pull request #641:
URL: https://github.com/apache/maven/pull/641#issuecomment-1002965979


   > @michael-o instead of copy/pasting the 3 methods, I think we could change 
"private Artifact newInstance" (from AbstractArtifact) to protected, then 
simply override this method
   
   While you are absolutely right, this is not the target of this ticket OR I 
need to broaden the ticket and solve this for all derved types. I absolutely do 
not like the fact that `AbstractArtifact` lacks abstraction and uses 
`DefaultArtifact` internally.
   
   Maybe: Mutating derived Artifact does not retain type


-- 
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-unsubscr...@maven.apache.org

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


Reply via email to