glen cvs at pld-linux.org
Thu Jul 20 16:13:12 CEST 2006

Author: glen
Date: Thu Jul 20 16:13:09 2006
New Revision: 7537

- killproc() s%!&*ks

Modified: rc-scripts/trunk/TODO
--- rc-scripts/trunk/TODO	(original)
+++ rc-scripts/trunk/TODO	Thu Jul 20 16:13:09 2006
@@ -6,13 +6,20 @@
 - merge baggins changes in atm-rc (added support for RFC1483/2684 Bridged
 - LVM on DMRAID device isn't started at bootup (missing vchange -ay somewhere)
+- killproc() sucks and has no consistency, error reporting
+  you'll never know which side of PROCNAME you should add the --waitforname or -SIGNAL args
+  because if you put -SIGNAL at wrong side of PROGNAME it will enter infinitive loop:
+    killproc -TERM --waitforname slapd --waitfortime 300 slapd
+  if you put --waitforname at wrong side, it will just ignore them:
+    killproc slapd -TERM --waitforname slapd --waitfortime 300 
+  while correct would be:
+    killproc --waitforname slapd --waitfortime 300 slapd -TERM
 Not realy important but mayby nice:
 - Add checking for privileges during restarting
   and if they are not enough say fuck off in a
   nice way.
 - maybe merge ATM rc scripts

More information about the pld-cvs-commit mailing list