From e63c9825da0a71c27e68cd76c6ebf5468b36cdb1 Mon Sep 17 00:00:00 2001
From: Pádraig Brady <P@draigBrady.com>
Date: Mon, 9 Jun 2008 16:10:47 +0100
Subject: timeout: use system-independent exit values

Change exit values from ETIMEDOUT and ECANCELED,
the values of which are system dependent, to
124 and 125 respectively.
* src/timeout.c (EXIT_TIMEDOUT, EXIT_CANCELED): Define.
(usage, main): Adjust.
* coreutils.texi (timeout invocation): Update.
* tests/misc/timeout: Adjust.
---
 doc/coreutils.texi |  2 +-
 src/timeout.c      | 27 ++++++++++++++-------------
 tests/misc/timeout |  2 +-
 3 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index fa7959d01..a626b4563 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -14106,7 +14106,7 @@ or a number. Also see @xref{Signal specifications}.
 Exit status:
 
 @display
-110 if @var{command} times out
+124 if @var{command} times out
 125 if @command{timeout} itself fails
 126 if @var{command} is found but cannot be invoked
 127 if @var{command} cannot be found
diff --git a/src/timeout.c b/src/timeout.c
index f7f9af012..53ebf3c05 100644
--- a/src/timeout.c
+++ b/src/timeout.c
@@ -20,14 +20,14 @@
    We try to behave like a shell starting a single (foreground) job,
    and will kill the job if we receive the alarm signal we setup.
    The exit status of the job is returned, or one of these errors:
-     ETIMEDOUT          110      job timed out
-     ECANCELED          125      internal error
+     EXIT_TIMEDOUT      124      job timed out
+     EXIT_CANCELED      125      internal error
      EXIT_CANNOT_INVOKE 126      error executing job
      EXIT_ENOENT        127      couldn't find job to exec
 
    Caveats:
      If user specifies the KILL (9) signal is to be sent on timeout,
-     the monitor is killed and so exits with 128+9 rather than ETIMEDOUT.
+     the monitor is killed and so exits with 128+9 rather than 124.
 
      If you start a command in the background, which reads from the tty
      and so is immediately sent SIGTTIN to stop, then the timeout
@@ -73,10 +73,11 @@
 
 #define AUTHORS proper_name_utf8 ("Padraig Brady", "P\303\241draig Brady")
 
+/* Note ETIMEDOUT is 110 on linux but this is non standard */
+#define EXIT_TIMEDOUT 124
+
 /* Internal failure.  */
-#ifndef ECANCELED
-# define ECANCELED 125
-#endif
+#define EXIT_CANCELED 125
 
 static int timed_out;
 static int term_signal = SIGTERM;  /* same default as kill command.  */
@@ -151,7 +152,7 @@ Mandatory arguments to long options are mandatory for short options too.\n\
       fputs (HELP_OPTION_DESCRIPTION, stdout);
       fputs (VERSION_OPTION_DESCRIPTION, stdout);
       fputs (_("\n\
-If the command times out, then we exit with status ETIMEDOUT,\n\
+If the command times out, then we exit with status 124,\n\
 otherwise the normal exit status of the command is returned.\n\
 If no signal is specified, the TERM signal is sent. The TERM signal\n\
 will kill processes which do not catch this signal. For other processes,\n\
@@ -227,7 +228,7 @@ main (int argc, char **argv)
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
-  initialize_exit_failure (ECANCELED);
+  initialize_exit_failure (EXIT_CANCELED);
   atexit (close_stdout);
 
   parse_long_options (argc, argv, PROGRAM_NAME, PACKAGE_NAME, VERSION,
@@ -240,16 +241,16 @@ main (int argc, char **argv)
         case 's':
           term_signal = operand2sig (optarg, signame);
           if (term_signal == -1)
-            usage (ECANCELED);
+            usage (EXIT_CANCELED);
           break;
         default:
-          usage (ECANCELED);
+          usage (EXIT_CANCELED);
           break;
         }
     }
 
   if (argc - optind < 2)
-    usage (ECANCELED);
+    usage (EXIT_CANCELED);
 
   if (xstrtoul (argv[optind], &ep, 10, &timeout, NULL)
       /* Invalid interval. Note 0 disables timeout  */
@@ -260,7 +261,7 @@ main (int argc, char **argv)
       || !apply_time_suffix ((unsigned int *) &timeout, *ep))
     {
       error (0, 0, _("invalid time interval %s"), quote (argv[optind]));
-      usage (ECANCELED);
+      usage (EXIT_CANCELED);
     }
   optind++;
 
@@ -321,7 +322,7 @@ main (int argc, char **argv)
         status = WTERMSIG (status) + 128;     /* what sh does at least.  */
 
       if (timed_out)
-        return ETIMEDOUT;
+        return EXIT_TIMEDOUT;
       else
         return status;
     }
diff --git a/tests/misc/timeout b/tests/misc/timeout
index 7f52f6e27..8996fbce3 100755
--- a/tests/misc/timeout
+++ b/tests/misc/timeout
@@ -39,6 +39,6 @@ timeout 1 false && fail=1
 
 # timeout
 timeout 1 sleep 2
-test $? = 110 || fail=1
+test $? = 124 || fail=1
 
 (exit $fail); exit $fail
-- 
cgit v1.2.3-54-g00ecf