Author: branden
Date: 2003-10-21 19:12:15 -0500 (Tue, 21 Oct 2003)
New Revision: 689

Modified:
   trunk/debian/xdm.prerm.in
Log:
Simplify the construction used to freak out if we use pidof and get back
something that isn't a number.


Modified: trunk/debian/xdm.prerm.in
===================================================================
--- trunk/debian/xdm.prerm.in   2003-10-22 00:09:10 UTC (rev 688)
+++ trunk/debian/xdm.prerm.in   2003-10-22 00:12:15 UTC (rev 689)
@@ -44,10 +44,8 @@
     if [ -n "$PARENTS" ]; then
       for PROCESS in $PARENTS; do
         # make sure we got numbers back
-        if ! [ $PROCESS -eq $PROCESS ] 2> /dev/null; then
-          # freak out
+        [ $PROCESS -eq $PROCESS ] 2> /dev/null ||
           errormsg "ERROR: pidof returned non-numeric value!"
-        fi
         # we could use grep -q here if ps would ignore SIGPIPE :-P
         if (ps axj | grep "^ *$PROCESS" > /dev/null 2>&1); then
           CHILDREN=yes


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to