On Tue, Aug 13, 2024 at 5:18 AM Patrick Steinhardt <ps@xxxxxx> wrote: > The `git bundle unbundle` subcommand requires a repository to unbundle > the contents into. As thus, the subcommand checks whether we have a > startup repository in the first place, and if not it dies. Perhaps you meant: s/As thus/As such/ > This check happens after we have already opened the bundle though. This > causes a segfault when running outside of a repository starting with > c8aed5e8da (repository: stop setting SHA1 as the default object hash, > 2024-05-07) because we have no hash function set up, but we do try to > parse refs advertised by the bundle's header. > > The next commit will fix that underlying issue by defaulting to the SHA1 > object format for bundles, which will also the described segfault here. Presumably: s/also the/also fix the/ > But as we know that we will die anyway, we can do better than that and > avoid some vain work by moving the check for a repository before we try > to open the bundle. > > Reported-by: ArcticLampyrid <ArcticLampyrid@xxxxxxxxxxx> > Suggested-by: Jeff King <peff@xxxxxxxx> > Signed-off-by: Patrick Steinhardt <ps@xxxxxx>
