You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2021/01/20 06:59:27 UTC

[GitHub] [maven-artifact-plugin] bmarwell commented on pull request #7: prefer Apache Commons StringUtils

bmarwell commented on pull request #7:
URL: https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-763384330


   I'd say `-1` to this one. Either create a `maven-lang` lib for the same reason as guava, commons-lang, etc. exist, or just wait for Java 8. @elharo no offense intended, but I honestly think (while I fully agree with what you mentioned earlier!) pulling in another dependency replacing a not-removable dependency is not worth it. Even if more changes like this are to come. 
   It is well-tested enough as this code existed for years and did not break.
   
   So, Maven 4 + Java 8 is not far away. Lets wait a few months and do a *real* java8 clean up then, wdyt?


----------------------------------------------------------------
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:
users@infra.apache.org