aalmiray commented on pull request #90:
URL: https://github.com/apache/maven-shade-plugin/pull/90#issuecomment-841656807


   Switching to `skipClassified` would help only when the shaded artifact has a 
classifier. Given that the default behavior of the shade plugin is to override 
the original artifact there can not be a classifier; the use case I presented 
also does not configure a classifier (matter of fact if it did then skip might 
not even be needed as what _I_ need is the original artifact to begin with, 
then again it's standard procedure to **not** provide a classifier (and I 
understand it's not mandated, it's a preference)).
   
   Thus `skipClassifier` would be limited in this regard en provide not 
solution to the use case at hand. I think sticking with just `<skip>` is OK for 
now.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to