diff options
author | Jeff King <peff@peff.net> | 2017-01-06 20:17:48 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-01-09 13:41:35 -0800 |
commit | 246f0edec0b789ccfeebcf7fef85417b7cb04425 (patch) | |
tree | 5ab877ad21dceefc6ce5233270b1863c00f18ecb /contrib/fast-import | |
parent | execv_dashed_external: use child_process struct (diff) | |
download | tgif-246f0edec0b789ccfeebcf7fef85417b7cb04425.tar.xz |
execv_dashed_external: stop exiting with negative code
When we try to exec a git sub-command, we pass along the
status code from run_command(). But that may return -1 if we
ran into an error with pipe() or execve(). This tends to
work (and end up as 255 due to twos-complement wraparound
and truncation), but in general it's probably a good idea to
avoid negative exit codes for portability.
We can easily translate to the normal generic "128" code we
get when syscalls cause us to die.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/fast-import')
0 files changed, 0 insertions, 0 deletions