[ mailman-Bugs-1433673 ] Wrong timezone in list archive index pages

Bugs item #1433673, was opened at 2006-02-17 08:08 Message generated for change (Comment added) made by msapiro You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1433673&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Pipermail Group: 2.1 (stable) Status: Closed Resolution: Fixed Priority: 5 Submitted By: Schoinobates Volans (schoinobates) Assigned to: Nobody/Anonymous (nobody) Summary: Wrong timezone in list archive index pages Initial Comment: This is Debian bug #352876 (see http://bugs.debian.org/352876) The timezone used on the list archive index pages for the starting, ending, last message and archived on dates is wrong. All these times are given in GMT (the standard timezone for the UK) but are claimed to be BST (British Summer Time, the daylight saving time used between March and October, equivalent to GMT+1). For example, the last message to one of my lists was sent at Fri, 10 Feb 2006 10:07:40 +0000, but according to the archives by thread page, the ending and last message dates are "Fri Feb 10 10:07:40 BST 2006". The timezone is correct in the archive page for the message though, where it is given as "Fri Feb 10 10:07:40 GMT 2006" The times given during the summer are correct, where everything is in BST. Original submitter claims timezone data and clock is correct on the machine. ----------------------------------------------------------------------
Comment By: Mark Sapiro (msapiro) Date: 2006-02-22 20:34
Message: Logged In: YES user_id=1123998 I've thought about this some more and I've come up with a different fix which will use tm_isdst from time.localtime() in those cases where time.strptime() returns tm_isdst = -1. Thanks to Roger Lynn for the motivation. ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2006-02-22 08:47 Message: Logged In: YES user_id=1123998
Isn't the real problem that the time zone has been stripped from the string in the first place?
Sort of. The problem here is that the string is produced by time.asctime() which doesn't put a timezone in the string. Changing this would be a significant change to the archiver which might well break something else - at least if I did it :-) ---------------------------------------------------------------------- Comment By: Roger Lynn (rogerlynn) Date: 2006-02-22 07:58 Message: Logged In: YES user_id=781651 Isn't the real problem that the time zone has been stripped from the string in the first place? ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2006-02-19 13:48 Message: Logged In: YES user_id=1123998 Patch has been committed to CVS. ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2006-02-17 19:35 Message: Logged In: YES user_id=1123998 The underlying problem is a bug in i18n.ctime(). When i18n.ctime is passed a time string, it uses time.strptime() to convert the string into a struct_time. If the string doesn't contain a time zone, strptime() returns tm_isdst = -1 which the following code treats as 1 rather than unknown. The attached patch (2.1.7 base) will use the phrase "Server Local Time" translated into the list language instead of BST or whatever. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1433673&group_id=103
participants (1)
-
SourceForge.net