Skip to content

gh-53203: Fix strptime() for %c and %x formats on many locales #124946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
serhiy-storchaka merged 18 commits into from
Oct 12, 2024
Merged

gh-53203: Fix strptime() for %c and %x formats on many locales #124946

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
1305285
gh-53203: Fix strptime(..,'%c') on locales with short month names
serhiy-storchaka Oct 3, 2024
dafd06c
Do not run tests on default locale if it does not worl on all locales.
serhiy-storchaka Oct 4, 2024
b5bace6
Skip some tests on Emscripten.
serhiy-storchaka Oct 4, 2024
b0d1fb5
Skip some tests on WASI.
serhiy-storchaka Oct 7, 2024
e5be6fa
Add many comments and do not run tests on potentially broken locales.
serhiy-storchaka Oct 7, 2024
a674f29
Fx also for the Basque locale.
serhiy-storchaka Oct 7, 2024
34320d6
Fix for Arabic.
serhiy-storchaka Oct 7, 2024
684cb84
Merge branch 'main' into strptime-short-month-names
serhiy-storchaka Oct 8, 2024
8d8b01d
Update NEWS file.
serhiy-storchaka Oct 8, 2024
f9ff92a
Refactoring.
serhiy-storchaka Oct 8, 2024
71be06d
Update test comments.
serhiy-storchaka Oct 9, 2024
dcf8045
Polishing.
serhiy-storchaka Oct 9, 2024
39e16b5
Fix also Breton, Kashubian and Yoruba.
serhiy-storchaka Oct 9, 2024
391296b
Fix also Ge'ez.
serhiy-storchaka Oct 9, 2024
ef8c18e
Fix also Bodo.
serhiy-storchaka Oct 9, 2024
f64b374
Fix testing br_FR on old glibc.
serhiy-storchaka Oct 12, 2024
fdfbef6
Merge branch 'main' into strptime-short-month-names
serhiy-storchaka Oct 12, 2024
408bd34
Remove added by mistake test for id_ID.
serhiy-storchaka Oct 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update test comments.
  • Loading branch information
@serhiy-storchaka
serhiy-storchaka committed Oct 9, 2024
commit 71be06d55577d0f4844777bab421d9c9f09a0f93
10 changes: 6 additions & 4 deletions Lib/test/test_strptime.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -478,9 +478,9 @@ def test_bad_timezone(self):
# * Year is not included: ha_NG.
# * Use non-Gregorian calendar: lo_LA, thai, th_TH.
#
# BUG: Generates invalid regexp for br_FR, csb_PL, Arabic.
# BUG: Generates invalid regexp for br_FR, csb_PL.
# BUG: Generates regexp that does not match the current date and time
# for fa_IR, gez_ER, gez_ET, lzh_TW, my_MM, or_IN, shn_MM, yo_NG.
# for az_IR, brx_IN, fa_IR, gez_ER, gez_ET, lzh_TW, my_MM, or_IN, shn_MM, yo_NG.
@run_with_locales('LC_TIME', 'C', 'en_US', 'fr_FR', 'de_DE', 'ja_JP',
'he_IL', 'eu_ES', 'ar_AE', 'mfe_MU')
def test_date_time_locale(self):
Expand All @@ -502,7 +502,8 @@ def test_date_time_locale(self):

# NB: Dates before 1969 do not roundtrip on some locales:
# bo_CN, bo_IN, dz_BT, eu_ES, eu_FR.
@run_with_locales('LC_TIME', 'C', 'en_US', 'fr_FR', 'de_DE', 'ja_JP', 'he_IL')
@run_with_locales('LC_TIME', 'C', 'en_US', 'fr_FR', 'de_DE', 'ja_JP',
'he_IL', 'ar_AE', 'mfe_MU')
def test_date_time_locale2(self):
# Test %c directive
self.roundtrip('%c', slice(0, 6), (1900, 1, 1, 0, 0, 0, 0, 1, 0))
Expand Down Expand Up @@ -531,7 +532,8 @@ def test_date_locale(self):
support.is_emscripten or support.is_wasi,
"musl libc issue on Emscripten, bpo-46390"
)
@run_with_locales('LC_TIME', 'en_US', 'fr_FR', 'de_DE', 'ja_JP')
@run_with_locales('LC_TIME', 'en_US', 'fr_FR', 'de_DE', 'ja_JP',
'eu_ES', 'ar_AE')
def test_date_locale2(self):
# Test %x directive
self.roundtrip('%x', slice(0, 3), (1900, 1, 1, 0, 0, 0, 0, 1, 0))
Expand Down