Sorry, it's the right, after a discussion in debian-user-spanish list.
http://lists.debian.org/debian-user-spanish/2007/05/msg01358.html
--oknodo is redundant (default behaviour), but it should go on to be
consistent with the dæmons...
Attached the patch.
Cheers.
Javi.
--- start-stop-daemon.c 2007-05-15 14:33:25.000000000 +0200
+++ start-stop-daemon.new.c 2007-06-01 00:26:57.000000000 +0200
@@ -117,7 +117,7 @@
static int testmode = 0;
static int quietmode = 0;
-static int exitnodo = 1;
+static int exitnodo = 0;
static int start = 0;
static int stop = 0;
static int background = 0;
@@ -341,7 +341,8 @@
" -m|--make-pidfile create the pidfile before starting\n"
" -R|--retry <schedule> check whether processes die, and retry\n"
" -t|--test test mode, don't do anything\n"
-" -o|--oknodo exit status 0 (not 1) if nothing done\n"
+" -e|--errornodo exit status 1 (not 0) if nothing done\n"
+" -o|--oknodo obsoleted (default behaviour)\n"
" -q|--quiet be more quiet\n"
" -v|--verbose be more verbose\n"
"\n"
@@ -351,7 +352,7 @@
" forever repeat remainder forever\n"
"or <schedule> may be just <timeout>, meaning <signal>/<timeout>/KILL/<timeout>\n"
"\n"
-"Exit status: 0 = done 1 = nothing done (=> 0 if --oknodo)\n"
+"Exit status: 0 = done 0 = nothing done (=> 1 if --errornodo)\n"
" 3 = trouble 2 = with --retry, processes wouldn't die\n",
VERSION);
}
@@ -525,6 +526,7 @@
{ "version", 0, NULL, 'V'},
{ "startas", 1, NULL, 'a'},
{ "name", 1, NULL, 'n'},
+ { "errornodo", 0, NULL, 'e'},
{ "oknodo", 0, NULL, 'o'},
{ "pidfile", 1, NULL, 'p'},
{ "quiet", 0, NULL, 'q'},
@@ -573,11 +575,13 @@
case 'n': /* --name <process-name> */
cmdname = optarg;
break;
- case 'o': /* --oknodo */
- exitnodo = 0;
+ case 'e': /* --errornodo */
+ exitnodo = 1;
break;
+ case 'o': /* --oknodo */
+ break;
case 'p': /* --pidfile <pid-file> */
- pidfile = optarg;
+ pidfile = optarg;
break;
case 'q': /* --quiet */
quietmode = 1;
