On 25/05/07 03:28PM, Patrick Steinhardt wrote: > On Tue, May 06, 2025 at 10:02:48PM -0500, Justin Tobler wrote: > > @@ -630,6 +630,7 @@ integration_tests = [ > > 't5409-colorize-remote-messages.sh', > > 't5410-receive-pack-alternates.sh', > > 't5411-proc-receive-hook.sh', > > + 't5412-receive-pack.sh', > > Instead of creating a new test file, do we maybe want to generalize > "t5410-receive-pack-alternates.sh"? Just a suggestion, this is not a > strong requirement from my side. That makes sense to me. I'll generalize "t5410" and add the test there. > > + # Capture git-send-pack(1) output sent to git-receive-pack(1). > > + git -C repo send-pack ../setup.git --all \ > > The `-C repo` shouldn't be necessary at all, should it? The repository > in which it runs is specified via the first parameter. The repository specified by the first parameter is the repository the pack is being sent to. We still need to define the repository on the sending side. > > + test_grep "fatal: Failed to traverse parents" actual && > > + test_must_fail git -C remote.git cat-file -e $(git -C repo rev-parse HEAD) > > I'm a bit surprised by the error message though. First, why is it on > stdout? Second, shouldn't there be some hint that the connectivity check > has failed in the error message?
