From: Junio C Hamano Date: Fri, 19 Sep 2014 21:05:13 +0000 (-0700) Subject: Merge branch 'et/spell-poll-infinite-with-minus-one-only' into maint X-Git-Tag: v2.1.1~1 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/b8f7239058e6f7c8bb68bd4276ff797d83e03229?ds=inline;hp=-c Merge branch 'et/spell-poll-infinite-with-minus-one-only' into maint * et/spell-poll-infinite-with-minus-one-only: upload-pack: keep poll(2)'s timeout to -1 --- b8f7239058e6f7c8bb68bd4276ff797d83e03229 diff --combined upload-pack.c index 01de944a0a,12c6b4d1a0..433211a238 --- a/upload-pack.c +++ b/upload-pack.c @@@ -17,7 -17,7 +17,7 @@@ static const char upload_pack_usage[] = "git upload-pack [--strict] [--timeout=] "; -/* bits #0..7 in revision.h, #8..10 in commit.c */ +/* Remember to update object flag allocation in object.h */ #define THEY_HAVE (1u << 11) #define OUR_REF (1u << 12) #define WANTED (1u << 13) @@@ -70,14 -70,6 +70,14 @@@ static ssize_t send_client_data(int fd return sz; } +static int write_one_shallow(const struct commit_graft *graft, void *cb_data) +{ + FILE *fp = cb_data; + if (graft->nr_parent == -1) + fprintf(fp, "--shallow %s\n", sha1_to_hex(graft->sha1)); + return 0; +} + static void create_pack_file(void) { struct child_process pack_objects; @@@ -89,10 -81,12 +89,10 @@@ const char *argv[12]; int i, arg = 0; FILE *pipe_fd; - const char *shallow_file = NULL; if (shallow_nr) { - shallow_file = setup_temporary_shallow(NULL); argv[arg++] = "--shallow-file"; - argv[arg++] = shallow_file; + argv[arg++] = ""; } argv[arg++] = "pack-objects"; argv[arg++] = "--revs"; @@@ -120,9 -114,6 +120,9 @@@ pipe_fd = xfdopen(pack_objects.in, "w"); + if (shallow_nr) + for_each_commit_graft(write_one_shallow, pipe_fd); + for (i = 0; i < want_obj.nr; i++) fprintf(pipe_fd, "%s\n", sha1_to_hex(want_obj.objects[i].item->sha1)); @@@ -167,7 -158,9 +167,9 @@@ if (!pollsize) break; - ret = poll(pfd, pollsize, 1000 * keepalive); + ret = poll(pfd, pollsize, + keepalive < 0 ? -1 : 1000 * keepalive); + if (ret < 0) { if (errno != EINTR) { error("poll failed, resuming: %s", @@@ -800,7 -793,7 +802,7 @@@ int main(int argc, char **argv packet_trace_identity("upload-pack"); git_extract_argv0_path(argv[0]); - read_replace_refs = 0; + check_replace_refs = 0; for (i = 1; i < argc; i++) { char *arg = argv[i];