diff options
| author | Junio C Hamano <gitster@pobox.com> | 2019-03-20 06:16:06 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-03-20 06:16:06 (GMT) |
| commit | 27cdbdd134f181fc97f9589039ed7c0d12759b5a (patch) | |
| tree | 3a42ab156427cfa7c1b7bd2f7d08469b6fdfc434 /builtin/fetch.c | |
| parent | ea327760d38b03be552418de044843cd2f0dba2e (diff) | |
| parent | 143588949c8a0672302403c722fc433a5bb2ea2a (diff) | |
| download | git-27cdbdd134f181fc97f9589039ed7c0d12759b5a.zip git-27cdbdd134f181fc97f9589039ed7c0d12759b5a.tar.gz git-27cdbdd134f181fc97f9589039ed7c0d12759b5a.tar.bz2 | |
Merge branch 'jk/no-sigpipe-during-network-transport'
On platforms where "git fetch" is killed with SIGPIPE (e.g. OSX),
the upload-pack that runs on the other end that hangs up after
detecting an error could cause "git fetch" to die with a signal,
which led to a flakey test. "git fetch" now ignores SIGPIPE during
the network portion of its operation (this is not a problem as we
check the return status from our write(2)s).
* jk/no-sigpipe-during-network-transport:
fetch: ignore SIGPIPE during network operation
fetch: avoid calling write_or_die()
Diffstat (limited to 'builtin/fetch.c')
| -rw-r--r-- | builtin/fetch.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/builtin/fetch.c b/builtin/fetch.c index b620fd5..4ba63d5 100644 --- a/builtin/fetch.c +++ b/builtin/fetch.c @@ -1556,7 +1556,9 @@ static int fetch_one(struct remote *remote, int argc, const char **argv, int pru sigchain_push_common(unlock_pack_on_signal); atexit(unlock_pack); + sigchain_push(SIGPIPE, SIG_IGN); exit_code = do_fetch(gtransport, &rs); + sigchain_pop(SIGPIPE); refspec_clear(&rs); transport_disconnect(gtransport); gtransport = NULL; |