[GitHub] [accumulo] ctubbsii opened a new pull request #2548: Remove log4j2 test config from jar differently

[GitBox <gi...@apache.org>]

ctubbsii opened a new pull request #2548:
URL: https://github.com/apache/accumulo/pull/2548


   Redo #2537 to fix #2527 differently. The solution in #2537 was
   insufficient because it removed the log4j2-test.properties configuration
   file from the class path in IDEs while running tests there, and then
   added it back in for the maven-failsafe-plugin. In doing so, it avoided
   putting the file in the jar. This change instead explicitly excludes the
   file from the jar using the maven-jar-plugin's own excludes
   configuration. This ensures the file is on the class path for executing
   the unit tests inside the IDE, as well as for maven-failsafe-plugin but
   does not place the file in the jar.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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

[GitHub] [accumulo] ctubbsii merged pull request #2548: Remove log4j2 test config from jar differently

[GitBox <gi...@apache.org>]

ctubbsii merged pull request #2548:
URL: https://github.com/apache/accumulo/pull/2548


   


-- 
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: notifications-unsubscribe@accumulo.apache.org

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