severity 448524 serious
thanks
Hi!
A package that fails to upgrade because the service is stopped is a
policy violation, thus serious.
* era eriksson <e...@iki.fi> [2009-08-12 12:59:32 CEST]:
> Ouch, I was too quick -- apologies. The patch I attached to the
> previous message did not properly implement the fix I described. Here's
> another attempt.
There is another issue with the patch (or the init.d script in the
first place): The existence of $PIDFILE is assumed, which is a bad
assumption if one has set their rc links to stop and only starts the
server when needed.
Why not directly using ps "$(cat $PIDFILE)" instead of fiddling around
with grepping through complete ps output? Much more straight forward and
ressource friendly.
#v+
--- thttpd-current/debian/thttpd.init.d.orig 2009-08-12 13:52:08.000000000
+0300
+++ thttpd-current/debian/thttpd.init.d 2009-08-12 13:54:32.000000000 +0300
@@ -36,7 +36,7 @@
stop)
echo -n "Stopping $DESC: "
- if ps ax | grep "$(cat $PIDFILE)" | grep -qv grep
+ if test -s $PIDFILE && ps "$(cat $PIDFILE)"
then
start-stop-daemon -K -q -p $PIDFILE -x $DAEMON --signal
10
fi
#v-
This should catch it all.
Rhonda
--
To UNSUBSCRIBE, email to debian-qa-packages-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
