Re: [PATCH 1/2] refspec: consolidate ref-prefix generation logic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Brandon Williams wrote:

> When using protocol v2 a client constructs a list of ref-prefixes which
> are sent across the wire so that the server can do server-side filtering
> of the ref-advertisement.  The logic that does this exists for both
> fetch and push (even though no push support for v2 currently exists yet)
> and is roughly the same so lets consolidate this logic and make it
> general enough that it can be used for both the push and fetch cases.
>
> Signed-off-by: Brandon Williams <bmwill@xxxxxxxxxx>
> ---
>  builtin/fetch.c | 13 +------------
>  refspec.c       | 29 +++++++++++++++++++++++++++++
>  refspec.h       |  4 ++++
>  transport.c     | 21 +--------------------
>  4 files changed, 35 insertions(+), 32 deletions(-)

I assume this is meant to be a refactoring with no functional change.
Alas, it's causing fetch-by-SHA-1 to fail for me:

 $ rm -fr /tmp/r
 $ git init /tmp/r
 Initialized empty Git repository in /tmp/r/.git/
 $ bin-wrappers/git -C /tmp/r -c protocol.version=2 fetch \
	https://kernel.googlesource.com/pub/scm/git/git \
	6373cb598e1a4e0340583ad75d5abba01ff79774
 fatal: no matching remote head

The parent commit succeeds.

Known issue?

Thanks,
Jonathan



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux