GNU bug report logs - #21020
24.4; `display-time-world' tampers with TZ

Previous Next

Package: emacs;

Reported by: William G. Gardella <wgg2 <at> member.fsf.org>

Date: Thu, 9 Jul 2015 19:41:02 UTC

Severity: normal

Tags: patch

Found in version 24.4

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 21020 <at> debbugs.gnu.org, wgg2 <at> member.fsf.org
Subject: bug#21020: 24.4; `display-time-world' tampers with TZ
Date: Sun, 26 Jul 2015 10:55:38 -0700
Eli Zaretskii wrote:

>> test/automated/icalender-tests.el exercises the new code.
>
> But the tests there that play with TZ are all expected failures on
> MS-Windows, because they use Posix format of time-zone definition that
> Windows runtime doesn't support.  So that probably means there are no
> tests that could be used on Windows.

OK, didn't know that.  Presumably this could be addressed by adding tests for 
MS-Windows TZ settings, if somebody has the time for this (pun intended...).

>> Also, the MS-Windows port can't entirely skip the time module any
>> more, as that module arranges for time.h to declare localtime_rz and
>> friends.
>
> This could conflict with time.h in nt/inc/sys/, no?

I don't see why.  <sys/time.h> is a different API.  This patch affects only 
<time.h>.

>lib/time.h seems to want to replace 'localtime', which might
> clash with 'sys_localtime' we currently use as replacement on w32.

The replacement is conditional, and the condition should be false on w32.




This bug report was last modified 9 years and 299 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.