anmolnar commented on pull request #1793: URL: https://github.com/apache/zookeeper/pull/1793#issuecomment-1017235459
> The top-level pom.xml still contains some references to Log4J. It's needed for contrib projects to build. I'll remove it in the next patch. > zookeeperAdmin.md contains a java -cp ... example that references Log4J jars. Thought it's only an example and the CP is probably already outdated, so not worth replacing it. I'll fix it anyway. > ReconfigExceptionTest contains a comment with a java -cp ... command referencing Log4J jars. I'll fix this too. > ZooTrace contains a comment that "Log4J must be correctly configured". Fixed. > zookeeper-server/src/main/resources/NOTICE.txt still mentions Log4J (though I'm unclear on whether or not this file is actually getting into the distribution). Weird. I've never come across this file, but replace it with logback alternatives. -- 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-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
