Re: git: 91506dba7f2e - main - zoneinfo: also install tzdata.zi and version

From: Philip Paeps <philip_at_freebsd.org>
Date: Tue, 06 May 2025 05:00:12 UTC
On 2025-05-02 21:52:04 (+0800), John Baldwin wrote:
> On 4/30/25 20:48, Philip Paeps wrote:
>> On 2025-05-01 00:58:04 (+0800), Ed Maste wrote:
>>> On Wed, 30 Apr 2025 at 00:58, Philip Paeps <philip@freebsd.org> 
>>> wrote:
>>>> On 2025-04-30 02:44:46 (+0800), Ed Maste wrote:
>>>>> On Tue, 1 Apr 2025 at 04:10, Philip Paeps <philip@freebsd.org>
>>>>> wrote:
>>>>>> The branch main has been updated by philip:
>>>>>>
>>>>>> URL:
>>>>>> https://cgit.FreeBSD.org/src/commit/?id=91506dba7f2eb147eeb55ec30640bc11755a1e8e
>>>>>>
>>>>>> commit 91506dba7f2eb147eeb55ec30640bc11755a1e8e
>>>>>> Author:     Philip Paeps <philip@FreeBSD.org>
>>>>>> AuthorDate: 2025-04-01 08:09:37 +0000
>>>>>> Commit:     Philip Paeps <philip@FreeBSD.org>
>>>>>> CommitDate: 2025-04-01 08:09:37 +0000
>>>>>>
>>>>>>      zoneinfo: also install tzdata.zi and version
>>>>>>
>>>>>>      Build and install the shrunk version of the time zone 
>>>>>> database
>>>>>> required
>>>>>>      by libc++ 19 std::chrono::current_zone (etc).  While here, 
>>>>>> also
>>>>>> install
>>>>>>      the tzdata 'version' file to align more closely with other
>>>>>> vendors.
>>>>>>
>>>>>>      PR:             282377
>>>>>>      Reviewed by:    dim, emaste, imp
>>>>>>      Differential Revision:  https://reviews.freebsd.org/D49577
>>>>>
>>>>> Do you plan to MFC this to stable/14 for 14.3 as well?
>>>>
>>>> I was not planning to MFC this change.
>>>>
>>>> Does libc++ in stable/14 need this support?
>>>
>>> I suspect so - std::chrono::tzdb is C++20 and we presumably want it 
>>> to
>>> exist/work in stable/14.  Dimitry?
>>
>> Thanks.  I thought that was a 15+ thing.
>>
>> If it's expected by the toolchain in stable/14, I'll merge it there 
>> too.
>
> LLVM 19 is in stable/13 as well.

Thanks.  I've merged this to stable/13 too.

Note that I don't think we're doing any more releases on stable/13 and I 
don't think this warrants an erratum, so this is a little bit academic. 
:)

Philip