svn commit: r1577091 - /jena/trunk/jena-arq/bin/make_classpath_mvn

[rv...@apache.org]

Author: rvesse
Date: Thu Mar 13 10:28:27 2014
New Revision: 1577091

URL: http://svn.apache.org/r1577091
Log:
Explain why we exclude what we do from the maven output in make_classpath_maven comments

Modified:
    jena/trunk/jena-arq/bin/make_classpath_mvn

Modified: jena/trunk/jena-arq/bin/make_classpath_mvn
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/bin/make_classpath_mvn?rev=1577091&r1=1577090&r2=1577091&view=diff
==============================================================================
--- jena/trunk/jena-arq/bin/make_classpath_mvn (original)
+++ jena/trunk/jena-arq/bin/make_classpath_mvn Thu Mar 13 10:28:27 2014
@@ -29,11 +29,19 @@ which mvn > /dev/null
 HAS_MAVEN=$?
 if [ -e "$DIRROOT/pom.xml" ] && [ $HAS_MAVEN -eq 0 ]; then
     # Take advantage of maven dependency plugin
+    
+    # We want to filter out maven's INFO, WARNING and dependency download messages
+    # We leave ERROR messages alone even though they'll break the classpath we don't want
+    # to suppress them
     MVN_OUTPUT_PATTERN="(^\[INFO\]|\[WARNING\]|Download)"
+    
+    # Switch up to the root directory and run maven
     CURR_DIR=$PWD
     cd $DIRROOT
     MVN_CP=`mvn dependency:build-classpath -U | egrep -v ${MVN_OUTPUT_PATTERN}`
     cd $CURR_DIR
+    
+    # Append the result to any user defined classpath
     CP="${CP}${SEP}${MVN_CP}"
 else
     # Use the crude method