Subject: Re: [PATCH v2 3/3] remote set-head: set followRemoteHEAD to "warn" if "always"

From: "Bence Ferdinandy" <[email-masked]>
Date: Thu, 05 Dec 2024 09:14:00 +0100

On Wed Dec 04, 2024 at 21:44, Kristoffer Haugsbakk <[email-masked]> wrote:
> On Wed, Dec 4, 2024, at 21:40, Junio C Hamano wrote:
>> "Kristoffer Haugsbakk" <[email-masked]> writes:
>>
>>> On Wed, Dec 4, 2024, at 11:39, Bence Ferdinandy wrote:
>>>> diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
>>>> index 7411aa770d..daf70406be 100755
>>>> --- a/t/t5505-remote.sh
>>>> +++ b/t/t5505-remote.sh
>>>> @@ -504,6 +504,17 @@ test_expect_success 'set-head --auto has no
>>>> problem w/multiple HEADs' '
>>>>  	)
>>>>  '
>>>>
>>>> +test_expect_success 'set-head changes followRemoteHEAD always to warn' '
>>>> +	(
>>>> +		cd test &&
>>>
>>> I think you need to `cd` in a subshell here.  See `t/README`, “Don't
>>> chdir around in tests.”.
>>
>> Puzzled.  Isn't this inside a (subshell) already?
>
> Aha, then I didn’t read the context properly.

Thanks for the review! What I gathered is that v3 needs an s/output/actual and
we're good. I'll send that soonish.

Thanks,
Bence