[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

karl added the comment: @gaurav The pull request https://github.com/python/cpython/pull/10870 has been closed in favor of https://github.com/python/cpython/pull/15773 which has already been merged. So we can probably close here. -- message_count: 7.0 -> 8.0 nosy: +karlcow nosy_count

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Alexander Belopolsky added the comment: I submitted Gaurav's patch as PR 10870. Please review. -- ___ Python tracker ___ ___ Python

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Change by Alexander Belopolsky : -- pull_requests: +10105 stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-l

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Natal Ngétal added the comment: Please can you convert your patch to a pull request on github. -- nosy: +hobbestigrou ___ Python tracker ___

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Change by Paul Ganssle : -- nosy: +p-ganssle ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Gaurav Tatke added the comment: Hi, I am new to python and would like to contribute. I find this issue easy so changed the documentation of datetime module. I tested it with sphinx. Attaching a patch. Please let me know if it is correct or needs changes. Appreciate it! Regards, Gaurav --

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Mark Dickinson : -- nosy: -mark.dickinson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Alexander Belopolsky : -- versions: +Python 3.6 -Python 3.5 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubsc

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Alexander Belopolsky : -- keywords: +easy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Alexander Belopolsky added the comment: Reclassifying this as a doc issue. -- components: +Documentation -Extension Modules stage: test needed -> needs patch versions: +Python 3.5 -Python 3.3 ___ Python tracker

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Alexander Belopolsky : -- nosy: +lemburg ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Éric Araujo added the comment: > Just recommend the astimezone use in the docs and recommend creating tz-aware instances in the first time (i.e. calling now(utc) instead of utcnow()), +1. -- nosy: +eric.araujo ___ Python tracker

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Alexander Belopolsky : -- versions: +Python 3.3 -Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubsc

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Changes by Alexander Belopolsky : -- nosy: +pitrou ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

Alexander Belopolsky added the comment: With timezone.utc available in datetime module, users should be encouraged to use dt.astimezone(timezone.utc).timetuple() instead of dt.utctimetuple(). Note that the later will set tm_isdst to -1. This observation can be used to argue for either of tw

[issue9004] datetime.utctimetuple() should not set tm_isdst flag to 0

New submission from Alexander Belopolsky : I find the following quite misleading: >>> from datetime import datetime >>> import time >>> time.strftime('%c %z %Z', datetime.utcnow().utctimetuple()) 'Wed Jun 16 03:26:26 2010 -0500 EST' As far as I can tell, the only other function that uses the tm