<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 6, 2022 at 7:07 PM Paul Eggert <<a href="mailto:eggert@cs.ucla.edu">eggert@cs.ucla.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 6/6/22 18:23, enh wrote:<br>
> i think the safest choice for bionic is probably just "disable the new<br>
> check and preserve the old behavior".<br>
<br>
Yes, that sounds right.<br>
<br>
Come to think of it, that's simpler (and arguably better) on all <br>
platforms. That is, if you give strftime a format containing %s, you <br>
must also pass a struct tm that came from a successful localtime call; <br>
otherwise behavior is undefined. In the undefined case tzdb strftime can <br>
simply output (time_t)-1 for %s.<br>
<br>
Proposed patch attached and installed into the development repository. <br>
This doesn't affect tzdb itself, as tzdb never gets into this situation.</blockquote><div><br></div><div>awesome, thanks! not having to keep a diff relative to upstream tzcode is great. (it's the large number of diffs we have so that tzdata can be one big file rather than lots of little files that got us into this mess where we're so far behind ToT in the first place!) </div></div></div>