diff options
| author | Csaba Henk <csaba@redhat.com> | 2012-04-24 11:53:56 +0200 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-06-01 16:49:39 -0700 |
| commit | 2084c0e3d748b7e28d2fc9749ad9d1e2bf63208c (patch) | |
| tree | 0137bf6ad134ed2d81aaa1c2c9fc3cc046b3d24e /cli/src/input.c | |
| parent | 982be4925851e13b027c5dce59e7a92a76cb9aae (diff) | |
cli: let commands specify the exit value in batch mode
Old behavior: when cli is ran in batch mode (sequence of commands are
fed to it in stdin), if a command returns an error (ie. -1), the cli
exits upon it with 255 (-1 on 8 bit).
New behavior: consider any non-zero return from cli commands as error
and use the negative of that return value as exit value, thus giving
control to cli commands over the exit value, while (as of the existing
command set) adhering to the convention of exiting with 1 on error.
Spotted upon stumbling upon mount/umount commands which did want
to exit with 1 on error but that was not possible as of old behavior.
Change-Id: I6f41191cdc718c3e676cfae1e404152f4cb715c5
BUG: 765214
Signed-off-by: Csaba Henk <csaba@redhat.com>
Reviewed-on: http://review.gluster.com/3218
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'cli/src/input.c')
| -rw-r--r-- | cli/src/input.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cli/src/input.c b/cli/src/input.c index a88d35874..004c9e300 100644 --- a/cli/src/input.c +++ b/cli/src/input.c @@ -71,11 +71,11 @@ cli_input (void *d) if (len > 0 && cmd[len - 1] == '\n') //strip trailing \n cmd[len - 1] = '\0'; ret = cli_cmd_process_line (state, cmd); - if (ret == -1 && state->mode & GLUSTER_MODE_ERR_FATAL) + if (ret != 0 && state->mode & GLUSTER_MODE_ERR_FATAL) break; } - exit (ret); + exit (-ret); return NULL; } |