bpo-17659: Add locale.getfirstweekday #18142
Conversation
cedk
force-pushed
the
locale-first-weekday
branch
from
1082d3b to
304fa76
Compare
|
Since you based your patch on the original patch, please give credit to the original author. |
cedk
force-pushed
the
locale-first-weekday
branch
from
90e4df6 to
f31e4d5
Compare
Misc/NEWS.d/next/Library/2020-01-23-13-05-44.bpo-17659.UfyH00.rst
Outdated
Show resolved
Hide resolved
This allows to determine the starting day of week depending on the OS locale. Co-authored-by: Kyle McMartin <kyle@mcmartin.ca>
Using setlocale does not change Windows configuration.
cedk
force-pushed
the
locale-first-weekday
branch
from
e98a14f to
c5227db
Compare
|
@tirkarthi would you like to do a final review here? I wonder if we should add the buildbot label to make sure this compiles and works everywhere (the test also needs fr_FR locale, which may be installed on no buildbot). |
|
My point was around a general question on the assertion. Other than that I have less knowledge to review the PR. Feel free to skip my review. |
|
I think it's better to return |
Indeed it is more flexible. So I made the change. |
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase And if you don't make the requested changes, you will be poked with soft cushions! |
| /* Magic constant is defined by glibc as the third value of the week | ||
| * keyword of LC_TIME */ | ||
| switch (week_1stday) { | ||
| default: |
There was a problem hiding this comment.
Indeed it seems that some version of glibc under en_US does not return one of the two values.
| elif platform.system() == 'Windows': | ||
| self.assertEqual(locale.getfirstweekday(), 6) | ||
| else: | ||
| self.assertEqual(locale.getfirstweekday(), None) |
There was a problem hiding this comment.
Have you verified that the test passes on musl libc?
There was a problem hiding this comment.
I do not have access to such system.
tiran
added
the
🔨 test-with-buildbots
bedevere-bot
removed
the
🔨 test-with-buildbots
|
All commit authors signed the Contributor License Agreement. |
This allows to determine the starting day of week depending on the OS
locale.
Co-authored-by: Kyle McMartin kyle@mcmartin.ca
https://bugs.python.org/issue17659