From e8d4ec73c7db16ebab9879d762c0027a9de620cb Mon Sep 17 00:00:00 2001
From: nlevitt <nlevitt@daa5b2f2-a927-0410-8b2d-f5f262fa301a>
Date: Fri, 21 Oct 2011 16:54:40 +0000
Subject: [PATCH] * bin/heritrix     when starting in background, make better
 effort to write pid to $HERITRIX_HOME/heritrix.pid on successful start, plus
 this change:      -    echo -n "`date` Starting heritrix"      +    echo -n
 "`date` Heritrix starting (pid $HERITRIX_PID)"

---
 dist/src/main/bin/heritrix | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/dist/src/main/bin/heritrix b/dist/src/main/bin/heritrix
index 9430c6b1..3175a1d9 100644
--- a/dist/src/main/bin/heritrix
+++ b/dist/src/main/bin/heritrix
@@ -144,7 +144,7 @@ else
     HERITRIX_PID=$!
 
     # Wait for content in the heritrix_dmesg.log file.
-    echo -n "`date` Starting heritrix"
+    echo -n "`date` Heritrix starting (pid $HERITRIX_PID)"
     while true 
     do
         sleep 1
@@ -156,10 +156,8 @@ else
             break
         fi
         kill -0 $HERITRIX_PID > /dev/null 2>&1
-        if [ $? -eq 0 ]
+        if [ $? -ne 0 ]
         then
-            echo $HERITRIX_PID > heritrix.pid
-        else
             echo
             echo "ERROR: JVM terminated without running Heritrix."
             echo "This could be due to invalid JAVA_OPTS or JMX_PORT, etc."
@@ -171,4 +169,10 @@ else
         fi
         echo -n '.'
     done
+
+    kill -0 $HERITRIX_PID > /dev/null 2>&1
+    if [ $? -eq 0 ] 
+    then
+        echo $HERITRIX_PID > heritrix.pid
+    fi
 fi