[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2013-06-17 Thread Jakub Wilk
Jakub Wilk added the comment: The documentations says Minor deficiency: mktime_tz() interprets the first 8 elements of tuple as a local time and then compensates for the timezone difference. This may yield a slight error around changes in daylight savings time, though not worth worrying about

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2013-06-17 Thread R. David Murray
R. David Murray added the comment: Since this issue has been closed, it would be great if you'd open a new issue just for the doc change. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14653

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2013-06-17 Thread Jakub Wilk
Jakub Wilk added the comment: Fair enough, filed as issue #18243. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14653 ___ ___ Python-bugs-list

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-06-21 Thread Roundup Robot
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset ffc048f43a70 by Alexander Belopolsky in branch '3.2': Issue #14653: email.utils.mktime_tz() no longer relies on system http://hg.python.org/cpython/rev/ffc048f43a70 New changeset 9f88c38318ac by Alexander Belopolsky

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-06-21 Thread Roundup Robot
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset a283563c8cc4 by Alexander Belopolsky in branch '2.7': Issue #14653: email.utils.mktime_tz() no longer relies on system http://hg.python.org/cpython/rev/a283563c8cc4 --

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-06-21 Thread Alexander Belopolsky
Changes by Alexander Belopolsky alexander.belopol...@gmail.com: -- resolution: - fixed stage: commit review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14653

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-06-20 Thread Alexander Belopolsky
Alexander Belopolsky alexander.belopol...@gmail.com added the comment: That said, this might be a worthwhile as a bug fix. I think this is a reasonable bug fix. Note that apart from OS-dependent date range, some mktime implementations reportedly don't support tm_isdst values other than -1.

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-05-24 Thread R. David Murray
Changes by R. David Murray rdmur...@bitdance.com: -- assignee: r.david.murray - components: +email -Library (Lib) nosy: +barry ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14653 ___

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-05-09 Thread Martin Panter
Changes by Martin Panter vadmium...@gmail.com: -- nosy: +vadmium ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14653 ___ ___ Python-bugs-list

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-04-24 Thread R. David Murray
R. David Murray rdmur...@bitdance.com added the comment: I think that what is going to happen is that both of these functions are going to be deprecated in favor of functions that use datetimes. That said, this might be a worthwhile as a bug fix. I'm adding Alexander as nosy to see what he

[issue14653] Improve mktime_tz to use calendar.timegm instead of time.mktime

2012-04-23 Thread Mitar
New submission from Mitar mmi...@gmail.com: I would suggest improvement of mktime_tz to use calendar.timegm internally instead of time.mktime. The problem is that on Windows mktime_tz fails with mktime argument out of range for this code: mktime_tz(parsedate_tz('Thu, 1 Jan 1970 00:00:00