Re: git: 91506dba7f2e - main - zoneinfo: also install tzdata.zi and version
- In reply to: John Baldwin : "Re: git: 91506dba7f2e - main - zoneinfo: also install tzdata.zi and version"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
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