stephen-webb commented on a change in pull request #30:
URL: https://github.com/apache/logging-log4cxx/pull/30#discussion_r457273830
##########
File path: pom.xml
##########
@@ -372,7 +372,6 @@
<reportSet>
<reports>
<report>scm</report>
-
<report>dependencies</report>
Review comment:
I was trying to add the compile time dependencies (APR, APR-Util) and
the testcase dependencies (sed, zip, gzip) to the report, but could not because
the maven expects all entries in dependencies to be jar files.
I can only think that the dependencies have to be mentioned elsewhere in
documentation (hence the the building page change). With the nav bar
Dependencies, that is the first place you look, and it is just basically wrong.
----------------------------------------------------------------
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]
