Re: [PATCH] fetch-pack: Add missing line-feed character when sending depth-request packet line
From: Junio C Hamano <hidden>
Date: 2016-06-16 02:18:45
Stan Hu [off-list ref] 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?
quoted hunk
Signed-off-by: Stan Hu <redacted> --- 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;