Re: [PATCH v8 0/6] set-head/fetch remote/HEAD updates, small change from v7

From: Taylor Blau <me@ttaylorr.com>
Date: Oct 16, 2024, 21:05 UTC
Message-ID: <ZxAqhqHwomsRq2Ck@nand.local>
In-Reply-To: <D4X2ZZAT5LKS.1FZ375SIMBV2F@ferdinandy.com>

On Wed, Oct 16, 2024 at 10:18:44AM +0200, Bence Ferdinandy wrote:

Show 23 quoted lines

>
> On Wed Oct 16, 2024 at 02:26, Taylor Blau <me@ttaylorr.com> wrote:
> > On Tue, Oct 15, 2024 at 12:53:09AM +0200, Bence Ferdinandy wrote:
> >> Bence Ferdinandy (6):
> >>   refs: atomically record overwritten ref in update_symref
> >>   remote set-head: add new variable for readability
> >>   remote set-head: better output for --auto
> >>   refs: add TRANSACTION_CREATE_EXISTS error
> >>   refs: add create_only option to refs_update_symref
> >>   fetch: set remote/HEAD if it does not exist
> >
> > I integrated this new round into my copy of 'seen' today and noticed
> > some test breakage in t5505 here:
> >
> >     https://github.com/ttaylorr/git/actions/runs/11356267070
> >
> > I'm going to temporarily eject this topic out of 'seen' until we can get
> > an analysis of what's going on here.
>
> Thanks for the heads up! I see that indeed the failing output is different than
> what I have locally (e.g. no "apis/HEAD -> apis/main" in any of the test files
> I have). On the other hand I can't reproduce it so I will need some help with
> this I think.

I similarly could not reproduce it (I ran 'make test' on this topic before integrating it into 'seen', and it passed, otherwise I wouldn't have picked it up).

I am not sure what the differences are. The 'ci' directory has some more bits on how the various suites are run, and the '.github/workflows' directory has some more bits still.

Thanks, Taylor