You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by tu...@apache.org on 2012/09/20 19:41:41 UTC

svn commit: r1388130 - in /oozie/trunk: bin/test-patch-05-patch-raw-analysis release-log.txt

Author: tucu
Date: Thu Sep 20 17:41:41 2012
New Revision: 1388130

URL: http://svn.apache.org/viewvc?rev=1388130&view=rev
Log:
OOZIE-998 test-patch doesn't allow lines that are exactly 132 characters (rkanter via tucu)

Modified:
    oozie/trunk/bin/test-patch-05-patch-raw-analysis
    oozie/trunk/release-log.txt

Modified: oozie/trunk/bin/test-patch-05-patch-raw-analysis
URL: http://svn.apache.org/viewvc/oozie/trunk/bin/test-patch-05-patch-raw-analysis?rev=1388130&r1=1388129&r2=1388130&view=diff
==============================================================================
--- oozie/trunk/bin/test-patch-05-patch-raw-analysis (original)
+++ oozie/trunk/bin/test-patch-05-patch-raw-analysis Thu Sep 20 17:41:41 2012
@@ -101,7 +101,8 @@ checkNoTrailingSpaces() {
 }
 ###############################################################################
 checkLinesLength() {
-  longLines=`grep "^+ " ${PATCHFILE} | awk 'BEGIN{count=0}{if ( length > 132 ) { count=count+1} }END{ print count}'`
+  # We check for > 133 to account for the "+" sign
+  longLines=`grep "^+ " ${PATCHFILE} | awk 'BEGIN{count=0}{if ( length > 133 ) { count=count+1} }END{ print count}'`
   if [[ ${longLines} != 0 ]] ; then
     REPORT+=("-1 the patch contains ${longLines} line(s) longer than 132 characters")
   else

Modified: oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1388130&r1=1388129&r2=1388130&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Thu Sep 20 17:41:41 2012
@@ -1,5 +1,6 @@
 -- Oozie 3.4.0 release (trunk - unreleased)
 
+OOZIE-998 test-patch doesn't allow lines that are exactly 132 characters (rkanter via tucu)
 OOZIE-739 a coord action fails because the uri points to a namenode that is not in whitelist. the E0901 error shows in the oozie.log, but not written to the database (mona,mbattisha via virag)
 OOZIE-989 Testcases failing intermittently where coordinator jobs are in catchup mode (virag)
 OOZIE-990 TestLogStreamer.testStreamLog fails in very rare cases (rkanter via tucu)