Subject: Re: [PATCH] Document LF appearing in shallow command during send-pack/receive-pack

From: Jonathan Nieder <[email-masked]>
Date: Wed, 27 Aug 2014 15:10:17 -0700

Shawn Pearce wrote:

> The implementation sends an LF, but the protocol documentation was
> missing this detail.
>
> Signed-off-by: Shawn Pearce <[email-masked]>
> ---
>  Documentation/technical/pack-protocol.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/technical/pack-protocol.txt b/Documentation/technical/pack-protocol.txt
> index 18dea8d..569c48a 100644
> --- a/Documentation/technical/pack-protocol.txt
> +++ b/Documentation/technical/pack-protocol.txt
> @@ -467,7 +467,7 @@ references.
>  ----
>    update-request    =  *shallow command-list [pack-file]
>  
> -  shallow           =  PKT-LINE("shallow" SP obj-id)
> +  shallow           =  PKT-LINE("shallow" SP obj-id LF)

In git, the client sends LF and the server is happy with or
without LF.  JGit and libgit2 don't send 'shallow' lines.

Reviewed-by: Jonathan Nieder <[email-masked]>

>  
>    command-list      =  PKT-LINE(command NUL capability-list LF)
>  		       *PKT-LINE(command LF)
> --