Bug 51309 -
Patch to better stop support with
CATALINA_PID
Patch to better stop support with
CATALINA_PID
|
|
|
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||
|
|||||
When using the CATALINA_PID variable the catalina.sh stop action verify if the PID file is empty (-s $CATALINA_PID) and after that verify if the file is a file (-f $CATALINA_PID). But if the file doesn't exist the "test -s" return 1 consequently the message "\$CATALINA_PID was set but the specified file does not exist. Is Tomcat running? Stop aborted." is never printed. I made a patch to fix this, but what you need to do is just invert "-s" and "-f". PATCH: --- catalina.sh.bkp 2011-06-01 12:02:07.541350449 -0300 +++ catalina.sh 2011-06-01 12:02:37.916302955 -0300 @@ -403,19 +403,19 @@ fi if [ ! -z "$CATALINA_PID" ]; then - if [ -s "$CATALINA_PID" ]; then - if [ -f "$CATALINA_PID" ]; then + if [ -f "$CATALINA_PID" ]; then + if [ -s "$CATALINA_PID" ]; then kill -0 `cat "$CATALINA_PID"` >/dev/null 2>&1 if [ $? -gt 0 ]; then echo "PID file found but no matching process was found. Stop aborted." exit 1 fi else - echo "\$CATALINA_PID was set but the specified file does not exist. Is Tomcat running? Stop aborted." - exit 1 + echo "PID file is empty and has been ignored." fi else - echo "PID file is empty and has been ignored." + echo "\$CATALINA_PID was set but the specified file does not exist. Is Tomcat running? Stop aborted." + exit 1 fi fi
Thanks for the patch. For future reference patches work best as attachments the lines are likely to get wrapped if you paste them in the comments box. The patch has been applied to 7.0.x and will be included in 7.0.15 onwards. I have proposed the patch for 6.0.x.
(In reply to comment #1) > Thanks for the patch. > > For future reference patches work best as attachments the lines are likely to > get wrapped if you paste them in the comments box. > > The patch has been applied to 7.0.x and will be included in 7.0.15 onwards. > > I have proposed the patch for 6.0.x. Thanks Mark
This has been fixed in 6.0.x and will be included in 6.0.33 onwards.