# [PATCH] fetch-pack: Add missing line-feed character when sending depth-request packet line 2 messages from 2016-04-11 to 2016-04-12. Participants: Stan Hu, Junio C Hamano. Thread: https://gitlist.dev/t/41988 ## Stan Hu, 2016-04-11 07:48 Subject: [PATCH] fetch-pack: Add missing line-feed character when sending depth-request packet line Message-ID: <1460360928-95956-1-git-send-email-stanhu@gmail.com> URL: https://gitlist.dev/e/1460360928-95956-1-git-send-email-stanhu%40gmail.com ``` The pkt-line format mandates: "a sender should include a LF, but the receive MUST NOT complain if it is not present." This patch is not absolutely necessary since receivers handle the missing the LF, but this patch adds it for good measure. Signed-off-by: Stan Hu --- fetch-pack.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fetch-pack.c b/fetch-pack.c index f96f6df..77299d9 100644 --- a/fetch-pack.c +++ b/fetch-pack.c @@ -330,7 +330,7 @@ static int find_common(struct fetch_pack_args *args, if (is_repository_shallow()) write_shallow_commits(&req_buf, 1, NULL); if (args->depth > 0) - packet_buf_write(&req_buf, "deepen %d", args->depth); + packet_buf_write(&req_buf, "deepen %d\n", args->depth); packet_buf_flush(&req_buf); state_len = req_buf.len; -- 2.7.3 ``` ## Junio C Hamano, 2016-04-12 22:31 Subject: Re: [PATCH] fetch-pack: Add missing line-feed character when sending depth-request packet line Message-ID: URL: https://gitlist.dev/e/xmqqpotuo3vt.fsf%40gitster.mtv.corp.google.com In-Reply-To: <1460360928-95956-1-git-send-email-stanhu@gmail.com> ``` Stan Hu writes: > The pkt-line format mandates: "a sender should include a LF, but the > receive MUST NOT complain if it is not present." This patch > is not absolutely necessary since receivers handle the missing the LF, > but this patch adds it for good measure. s/since receivers handle/since receivers are expected to handle/; Have you checked various re-implementations of upload-pack to see which one will break with this change, and made them aware about their non-compliance? > > Signed-off-by: Stan Hu > --- > fetch-pack.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fetch-pack.c b/fetch-pack.c > index f96f6df..77299d9 100644 > --- a/fetch-pack.c > +++ b/fetch-pack.c > @@ -330,7 +330,7 @@ static int find_common(struct fetch_pack_args *args, > if (is_repository_shallow()) > write_shallow_commits(&req_buf, 1, NULL); > if (args->depth > 0) > - packet_buf_write(&req_buf, "deepen %d", args->depth); > + packet_buf_write(&req_buf, "deepen %d\n", args->depth); > packet_buf_flush(&req_buf); > state_len = req_buf.len; ```