Re: Git 2.36, ls-tree submodule regression?

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

 



> Bisecting the error selects:
> 
> $ git bisect good
> 9c4d58ff2c385f49585197c8650356955e1fa02e is the first bad commit
> commit 9c4d58ff2c385f49585197c8650356955e1fa02e
> Author: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
> Date:   Wed Mar 23 17:13:15 2022 +0800
> 
>     ls-tree: split up "fast path" callbacks
> 
>     Make the various if/else in the callbacks for the "fast path" a lot
>     easier to read by just using common functions for the parts that are
>     common, and have per-format callbacks for those parts that are
>     different.
> 
>     Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
>     Signed-off-by: Teng Long <dyroneteng@xxxxxxxxx>
>     Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>
> 
>  builtin/ls-tree.c | 199 ++++++++++++++++++++++++++++++++++--------------------
>  1 file changed, 125 insertions(+), 74 deletions(-)
> 
> (I've CC'd Ævar and Teng on this message.)
> 
> Looking at the changes in that commit I see differences in the
> handling for long listings, so I tried removing the -l. At that point,
> the command works again:

Yes, it's a bug but already found and fixed by Ævar Arnfjörð Bjarmason.

> Given the command works without -l, and all -l is supposed to do is
> add sizes to blobs, it seems like something has regressed in the logic
> that decides whether to continue recursing or move on to the next
> sibling?

As you described blow, I almost comfirmed it's the problem that
found in [1] and Junio C Hamano already replied it in [2]
 
> Can someone else whose C is a little less rusty check this and see if
> it's a bug? It almost seems like it _has_ to be, though. Bitbucket
> Server manages bare repositories which often include submodules, but
> none of those submodules (even assuming they're valid, which is not
> always the case for repositories in the wild) are ever _fetched_ into
> that bare repository. That implies this would always fail in any
> repository that contains submodules referencing other repositories.

Sorry for the trouble.

Thank you for debugging this and reporting this problem. 

Links:

  1. https://public-inbox.org/git/YktP0LltNKq7JZZs@xxxxxxxxxx/
  2. https://public-inbox.org/git/xmqq8rsi1ap4.fsf@gitster.g/



[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