diff options
author | John Kacur <jkacur@redhat.com> | 2016-03-24 15:21:54 +0100 |
---|---|---|
committer | John Kacur <jkacur@redhat.com> | 2016-03-24 16:26:32 +0100 |
commit | 967cb01bd7fd4996dc1c6f8a91577468e439bfc5 (patch) | |
tree | 68f5f9203a5e4145cbdcce1c019db74e81626033 | |
parent | 39889fd0b2eb562c346ebb3ada8b76bdf8f82089 (diff) | |
download | rt-tests-967cb01bd7fd4996dc1c6f8a91577468e439bfc5.tar.gz |
Revert changes to rt-migrate-test for exit(1)
Commit 3290f8412a4562adbc05b38d8e5212090e41587e changed
exit(-1) to exit(1) for many programs in rt-tests for consistency.
rt-migrate-test needs tri-state exit statuses, because of the way
some existing bash scripts consume it's results.
This could no-doubt be improved in the future, and there are some
inconsistencies in the exit statuses in this program alone, but as they
are not urgent to fix, and the current fix makes things worse, reverting
this for now.
Signed-off-by: John Kacur <jkacur@redhat.com>
Reviewed-by: Luiz Capitulino <lcapitulino@redhat.com>
-rw-r--r-- | src/rt-migrate-test/rt-migrate-test.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/rt-migrate-test/rt-migrate-test.c b/src/rt-migrate-test/rt-migrate-test.c index e18beca..d7b68dd 100644 --- a/src/rt-migrate-test/rt-migrate-test.c +++ b/src/rt-migrate-test/rt-migrate-test.c @@ -138,7 +138,7 @@ static void perr(char *fmt, ...) perror(buffer); fflush(stderr); - exit(1); + exit(-1); } static void print_progress_bar(int percent) @@ -226,6 +226,7 @@ static void parse_options (int argc, char *argv[]) break; } } + } static unsigned long long get_time(void) @@ -603,7 +604,7 @@ int main (int argc, char **argv) exit(1); } if (check < 0) - exit(1); + exit(-1); else exit(0); |