diff options
author | Johannes Sixt <j6t@kdbg.org> | 2009-07-05 20:57:46 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2009-07-05 12:16:26 -0700 |
commit | 47e3de0e7968a4176e2c54a36b214d3e7b24ad15 (patch) | |
tree | 0d24466e12853253e4f6dea495272a9e6817a14c /compat | |
parent | 606475f3178784e5a6b3a01dce1a54314345cf43 (diff) | |
download | git-47e3de0e7968a4176e2c54a36b214d3e7b24ad15.tar.gz |
MinGW: truncate exit()'s argument to lowest 8 bits
For some reason, MinGW's bash cannot reliably detect failure of the child
process if a negative value is passed to exit(). This fixes it by
truncating the exit code in all calls of exit().
This issue was worked around in run_builtin() of git.c (2488df84 builtin
run_command: do not exit with -1, 2007-11-15). This workaround is no longer
necessary and is reverted.
Suggested-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'compat')
-rw-r--r-- | compat/mingw.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/compat/mingw.h b/compat/mingw.h index 4f7ba4c13f..c1859c5480 100644 --- a/compat/mingw.h +++ b/compat/mingw.h @@ -92,6 +92,8 @@ static inline int fcntl(int fd, int cmd, long arg) errno = EINVAL; return -1; } +/* bash cannot reliably detect negative return codes as failure */ +#define exit(code) exit((code) & 0xff) /* * simple adaptors |