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
Differential D49577
zoneinfo: also install tzdata.zi and version philip on Sun, Mar 30, 5:27 AM. Authored by Tags None Referenced Files
Subscribers
Details Build and install the shrunk version of the time zone database required PR: 282377
Diff Detail
Event TimelineComment Actions This builds the "main" format of the time zone database. That includes backward compatibility links but it doesn't include historical ("backzone") data. The "redo" bit controls how leapseconds are handled. We may want this to be posix_right. I'm not sure what other vendors do here. Comment Actions Setting it to posix_right i think is a terrible idea as the default. One that should be carefully considered in the larger backdrop. Iirc, being able to opt into posix_right can be good for some applications, but switching the default is HUGE pola and requires a systemic review for far too many use cases, many of which won't be known untils years later. Anybody using smeared leap seconds won't want it, for example. There's two, completely irreconcilable warring factions and way stronger evidence than just what other vendors are doing is going to be needed since there's about a 40s difference in what you set the system time to for starters... it shouldn't be done in phab. There's talk of ditching leaps entirely and its unclear if we will have another one too.. . Comment Actions Thanks for the feedback. That was my reasoning for doing posix_only instead of the tzdata default posix_right. It sounds like it's currently not worth the bother of doing a build knob for changing this. I'll put this in main as-is. I don't currently have a plan to merge this to stable. The earliest I would merge it is after a future tzdata release in the second half of this year or so. While there are no current consumers of this, this will want a long time to soak if libc++ picks it up. |