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)
