svn commit: r1392729 - in /oozie/trunk: bin/test-patch bin/test-patch-09-javadoc release-log.txt

[tu...@apache.org]

Author: tucu
Date: Tue Oct  2 04:03:21 2012
New Revision: 1392729

URL: http://svn.apache.org/viewvc?rev=1392729&view=rev
Log:
OOZIE-992 Amendment (tucu)

Modified:
    oozie/trunk/bin/test-patch
    oozie/trunk/bin/test-patch-09-javadoc
    oozie/trunk/release-log.txt

Modified: oozie/trunk/bin/test-patch
URL: http://svn.apache.org/viewvc/oozie/trunk/bin/test-patch?rev=1392729&r1=1392728&r2=1392729&view=diff
==============================================================================
--- oozie/trunk/bin/test-patch (original)
+++ oozie/trunk/bin/test-patch Tue Oct  2 04:03:21 2012
@@ -381,13 +381,19 @@ postPatchRun
 createReports
 echo "" >> ${SUMMARYFILE}
 echo "----------------------------" >> ${SUMMARYFILE}
-MINUSONES=`grep -c "\-1" ${SUMMARYFILE}`
+MINUSONES=`grep -c "\}\-1" ${SUMMARYFILE}`
 if [[ $MINUSONES == 0 ]]; then
   echo "{color:green}Overall result: Good!, no -1s{color}" >> ${SUMMARYFILE}
 else
   echo "{color:red}Overall result: Please check the reported -1(s){color}" >> ${SUMMARYFILE}
 fi
 echo "" >> ${SUMMARYFILE}
+WARNINGS=`grep -c "\}WARNING" ${SUMMARYFILE}`
+if [[ $WARNINGS != 0 ]]; then
+  echo "{color:red}There is at least one warning, please check{color}" >> ${SUMMARYFILE}
+fi
+echo "" >> ${SUMMARYFILE}
+
 
 if [ ! -z "${JIRAISSUE}" ]; then
   echo "The full output of the test-patch run is available at"  >> ${SUMMARYFILE}

Modified: oozie/trunk/bin/test-patch-09-javadoc
URL: http://svn.apache.org/viewvc/oozie/trunk/bin/test-patch-09-javadoc?rev=1392729&r1=1392728&r2=1392729&view=diff
==============================================================================
--- oozie/trunk/bin/test-patch-09-javadoc (original)
+++ oozie/trunk/bin/test-patch-09-javadoc Tue Oct  2 04:03:21 2012
@@ -88,7 +88,7 @@ checkForWarnings() {
     newWarns=1
   fi
   if [[ $cleanWarns != 0 ]] ; then
-    REPORT+=("WARNING: the current HEAD has $cleanWarns Javadoc warning(s)")
+    REPORT+=("{color:red}WARNING{color}: the current HEAD has $cleanWarns Javadoc warning(s)")
   fi
 }
 ###############################################################################

Modified: oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1392729&r1=1392728&r2=1392729&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Tue Oct  2 04:03:21 2012
@@ -1,5 +1,6 @@
 -- Oozie 3.4.0 release (trunk - unreleased)
 
+OOZIE-992 Add overall status to test-patch messages and add some color-coding for negative results (tucu)
 OOZIE-1003 TestOozieCLI.testSubmitDoAs() should disable anonymous request (tucu)
 OOZIE-1004 Oozie client needs to bundle slf4j JARs (tucu)
 OOZIE-1000 Remove Yahoo branding from docs, tests, etc (rkanter via virag)