Re: CI Failures (osx with p4)

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 17, 2025, 00:25 UTC
Message-ID: <xmqqms4lbj4t.fsf@gitster.g>
In-Reply-To: <CAOTNsDzqXZVMhW+kuWG_ZnxSc0U-g01AXkG7EbP8HXzR16LO7Q@mail.gmail.com>

Koji Nakamaru <koji.nakamaru@gree.net> writes:

Show 28 quoted lines

> On Mon, Nov 17, 2025 at 2:18 AM Junio C Hamano <gitster@pobox.com> wrote:
>> > --
>> > diff --git a/ci/install-dependencies.sh b/ci/install-dependencies.sh
>> > index 50628ee2dd..16ee593d30 100755
>> > --- a/ci/install-dependencies.sh
>> > +++ b/ci/install-dependencies.sh
>> > @@ -7,7 +7,7 @@
>> >
>> >  begin_group "Install dependencies"
>> >
>> > -P4WHENCE=https://cdist2.perforce.com/perforce/r23.2
>> > +P4WHENCE=https://cdist2.perforce.com/perforce/r24.2
>>
>> Just being curious, other than "because it is newer than r23.2", was
>> there a particular reason why r24.2 was picked?  It is not the
>> latest that has arm64 macos (r25.1 also has bin.macosx12arm64
>> subdirectory at the above distribution site), and r23.2 also has
>> bin.macosx12arm64 subdirectory, too, which makes the above update
>> curious if it comes without explanation, given that this would also
>> affect Ubuntu that slurps bin.linux26x86_64 from there.
>
> When I first looked in r23.2 in the browser, somehow I could not find
> bin.macosx12arm64, so I looked for another version that included
> it. It now exists in r23.2 as you mentioned, and I totally agree that
> it is better to keep using r23.2. I confirmed r23.2 runs without any
> error [1].
>
> [1]: https://github.com/KojiNakamaru/git/actions/runs/19413263298

Thanks!