You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2013/03/08 21:56:03 UTC

svn commit: r1454548 - /hbase/trunk/dev-support/test-patch.sh

Author: stack
Date: Fri Mar  8 20:56:02 2013
New Revision: 1454548

URL: http://svn.apache.org/r1454548
Log:
HBASE-8035 Add site target check to precommit tests

Modified:
    hbase/trunk/dev-support/test-patch.sh

Modified: hbase/trunk/dev-support/test-patch.sh
URL: http://svn.apache.org/viewvc/hbase/trunk/dev-support/test-patch.sh?rev=1454548&r1=1454547&r2=1454548&view=diff
==============================================================================
--- hbase/trunk/dev-support/test-patch.sh (original)
+++ hbase/trunk/dev-support/test-patch.sh Fri Mar  8 20:56:02 2013
@@ -678,6 +678,34 @@ runTests () {
 }
 
 ###############################################################################
+### Check docbook site xml
+checkSiteXml () {
+  echo ""
+  echo ""
+  echo "======================================================================"
+  echo "======================================================================"
+  echo "    Checking Site generation"
+  echo "======================================================================"
+  echo "======================================================================"
+  echo ""
+  echo ""
+
+  echo "$MVN clean compile site -DskipTests -D${PROJECT_NAME}PatchProcess > $PATCH_DIR/patchSiteOutput.txt 2>&1"
+  export MAVEN_OPTS="${MAVEN_OPTS}"
+  $MVN clean compile site -DskipTests -D${PROJECT_NAME}PatchProcess  > $PATCH_DIR/patchSiteOutput.txt 2>&1
+  if [[ $? != 0 ]] ; then
+    JIRA_COMMENT="$JIRA_COMMENT
+
+    {color:red}-1 site{color}.  The patch appears to cause mvn site goal to fail."
+    return 1
+  fi
+  JIRA_COMMENT="$JIRA_COMMENT
+
+  {color:green}+1 site{color}.  The mvn site goal succeeds with this patch."
+  return 0
+}
+
+###############################################################################
 ### Run the inject-system-faults target
 checkInjectSystemFaults () {
   echo ""
@@ -817,6 +845,8 @@ checkReleaseAuditWarnings
 (( RESULT = RESULT + $? ))
 checkLineLengths
 (( RESULT = RESULT + $? ))
+checkSiteXml
+(( RESULT = RESULT + $?))
 ### Do not call these when run by a developer 
 if [[ $JENKINS == "true" ]] ; then
   runTests