Author: rkanter
Date: Mon Feb 4 17:57:28 2013
New Revision: 1442217
URL: http://svn.apache.org/viewvc?rev=1442217&view=rev
Log:
OOZIE-992 Amendment (tucu)
Modified:
oozie/branches/branch-3.3/bin/test-patch
oozie/branches/branch-3.3/bin/test-patch-09-javadoc
oozie/branches/branch-3.3/release-log.txt
Modified: oozie/branches/branch-3.3/bin/test-patch
URL:
http://svn.apache.org/viewvc/oozie/branches/branch-3.3/bin/test-patch?rev=1442217&r1=1442216&r2=1442217&view=diff
==============================================================================
--- oozie/branches/branch-3.3/bin/test-patch (original)
+++ oozie/branches/branch-3.3/bin/test-patch Mon Feb 4 17:57:28 2013
@@ -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/branches/branch-3.3/bin/test-patch-09-javadoc
URL:
http://svn.apache.org/viewvc/oozie/branches/branch-3.3/bin/test-patch-09-javadoc?rev=1442217&r1=1442216&r2=1442217&view=diff
==============================================================================
--- oozie/branches/branch-3.3/bin/test-patch-09-javadoc (original)
+++ oozie/branches/branch-3.3/bin/test-patch-09-javadoc Mon Feb 4 17:57:28 2013
@@ -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/branches/branch-3.3/release-log.txt
URL:
http://svn.apache.org/viewvc/oozie/branches/branch-3.3/release-log.txt?rev=1442217&r1=1442216&r2=1442217&view=diff
==============================================================================
--- oozie/branches/branch-3.3/release-log.txt (original)
+++ oozie/branches/branch-3.3/release-log.txt Mon Feb 4 17:57:28 2013
@@ -1,5 +1,6 @@
-- Oozie 3.3.2 (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)
