This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Bug in strftime %z


On Jan 12 18:32, Christian Franke wrote:
> Corinna Vinschen wrote:
> >On Jan  7 22:50, Corinna Vinschen wrote:
> >>Hi Christian,
> >>
> >>
> >>thanks for the testcase!
> >>
> >>On Dec 19 19:44, Christian Franke wrote:
> >>>$ cat strftest.c
> >>>#include <time.h>
> >>>#include <stdio.h>
> >>>#include <stdlib.h>
> >>>
> >>>int main(int argc, char **argv)
> >>>{
> >>>   time_t t = (argc > 1 ? atol(argv[1]) : time(NULL));
> >>>   struct tm *tm = localtime(&t);
> >>>   char buf[100];
> >>>   strftime(buf, sizeof(buf), "%Y-%m-%d %H:%M:%S %z (%Z)", tm);
> >>>   printf("%s\n", buf);
> >>>   return 0;
> >>>}
> >>>
> >>>$ gcc -o strftest strftest.c
> >>>
> >>>$ echo $TZ
> >>>Europe/Berlin
> >>>
> >>>$ ./strftest 1419010000
> >>>2014-12-19 18:26:40 +0200 (CET)
> >>>
> >>>$ ./strftest 1436200000
> >>>2015-07-06 18:26:40 +0053 (CEST)
> >>Surprisingly this is an old problem which exists in this form since 2011.
> >>Weird that nobody noticed it so far.  Or I missed earlier bug reports :}
> >>The underlying cause was that the info for std and dst offsets was taken
> >>from the wrong spot in the file.  The weird 0053 offset is a result of
> >>wrongly using the LMT offset for Europe/Berlin.
> >>
> >>While this fixes the problem for recent timestamps, it will probably
> >>fail for older timestamps when other dst rules were active.
> >>
> >>The latest Cygwin versions come with an extended struct tm which adds
> >>members tm_offset and tm_zone for newly built applications.  I'm working
> >>on a patch so strftime utilizes these members if they are available.
> >>I hope to get this finished tomorrow.
> >I uploaded a snapshot containing the fixes in tzload and strftime to
> >https://cygwin.com/snapshots/  Please give it a try.
> 
> %z now works as expected.

Cool, thank for testing!


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: pgpmI2mrywxUy.pgp
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]