- killproc() s%!&*ks

svn-id: @7537

diff --git a/TODO b/TODO
index 01afc7d4ebca648163e1d0f423d5378faa760b9e..a75b54458cc0df2056ae97cfd1bf8f7cf78e7fe8 100644 (file)
--- a/TODO
+++ b/TODO
@@ -6,13 +6,20 @@ Important:
 - merge baggins changes in atm-rc (added support for RFC1483/2684 Bridged
   protocol)
 - 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.
 
-
 Other:
 - maybe merge ATM rc scripts