summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohn Kacur <jkacur@redhat.com>2016-03-24 15:21:54 +0100
committerJohn Kacur <jkacur@redhat.com>2016-03-24 16:26:32 +0100
commit967cb01bd7fd4996dc1c6f8a91577468e439bfc5 (patch)
tree68f5f9203a5e4145cbdcce1c019db74e81626033
parent39889fd0b2eb562c346ebb3ada8b76bdf8f82089 (diff)
downloadrt-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.c5
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);