You are viewing a plain text version of this content. The canonical link for it is here.
Posted to svn@forrest.apache.org by gm...@apache.org on 2010/12/20 14:19:28 UTC

svn commit: r1051093 - /forrest/zone/config/update_check.sh

Author: gmcdonald
Date: Mon Dec 20 13:19:28 2010
New Revision: 1051093

URL: http://svn.apache.org/viewvc?rev=1051093&view=rev
Log:
This works fine, though ultimately, what fixed the issue was specifying the full path to svn so crons minimal shell could find it and therefore also provide a true value of 0 for the test when appropriate. Fix for FOR-1209

Modified:
    forrest/zone/config/update_check.sh

Modified: forrest/zone/config/update_check.sh
URL: http://svn.apache.org/viewvc/forrest/zone/config/update_check.sh?rev=1051093&r1=1051092&r2=1051093&view=diff
==============================================================================
--- forrest/zone/config/update_check.sh (original)
+++ forrest/zone/config/update_check.sh Mon Dec 20 13:19:28 2010
@@ -1,7 +1,6 @@
-#!/bin/sh
+#!/bin/bash
 
-svn st -u /home/forrest/sources/forrest-trunk | grep '\*' > /dev/null
-
-if [ $? = 0 ]; then
-  /home/forrest/config/update-forrest-trunk.sh
+/usr/local/bin/svn st -u /home/forrest/sources/forrest-trunk | grep '\*' > /dev/null
+if [ ${?} -eq 0 ]; then
+    /home/forrest/config/update-forrest-trunk.sh
 fi