gh-129483: Make `TestLocalTimeDisambiguation`'s time format locale independent by gkirchou · Pull Request #142193 · python/cpython · GitHub
Skip to content

gh-129483: Make TestLocalTimeDisambiguation's time format locale independent - #142193

Merged
pganssle merged 4 commits into
python:mainfrom
gkirchou:gh-129483
Dec 4, 2025
Merged

pganssle merged 4 commits into
python:mainfrom
gkirchou:gh-129483

Conversation

@gkirchou

@gkirchou gkirchou commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Test with:

./configure --enable-optimizations --with-lto
make -s -j $(nproc);

@bedevere-app

This comment was marked as resolved.

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO making the tests locale independent would be a better solution than this (i.e. just replace %c with our desired order).

@bedevere-app

This comment was marked as resolved.

@gkirchou gkirchou changed the title gh-129483: Force TestLocalTimeDisambiguation to run on LC_ALL=C gh-129483: Clarify TestLocalTimeDisambiguation's time format Dec 3, 2025
@gkirchou

gkirchou commented Dec 3, 2025

Copy link
Copy Markdown
Contributor Author

@StanFromIreland StanFromIreland changed the title gh-129483: Clarify TestLocalTimeDisambiguation's time format gh-129483: Make TestLocalTimeDisambiguation's time format locale independent Dec 3, 2025

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@python-cla-bot

python-cla-bot Bot commented Dec 3, 2025

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pganssle
pganssle merged commit 8392095 into python:main Dec 4, 2025
44 checks passed
@pganssle

pganssle commented Dec 4, 2025

Copy link
Copy Markdown
Member

This is failing in 3.13, right? So we should backport it? Anyone know how far back it is causing issues?

@vstinner vstinner added the needs backport to 3.14 bugs and security fixes label Dec 4, 2025
@miss-islington-app

Copy link
Copy Markdown

Thanks @gkirchou for the PR, and @pganssle for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 4, 2025
…ale independent (pythonGH-142193)

* Change to update %c to the exact time format.

---------
(cherry picked from commit 8392095)

Co-authored-by: Kir Chou <148194051+gkirchou@users.noreply.github.com>
Co-authored-by: Kir Chou <note351@hotmail.com>
@bedevere-app

bedevere-app Bot commented Dec 4, 2025

Copy link
Copy Markdown

GH-142258 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Dec 4, 2025
@vstinner vstinner added the needs backport to 3.13 bugs and security fixes label Dec 4, 2025
@miss-islington-app

Copy link
Copy Markdown

Thanks @gkirchou for the PR, and @pganssle for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 4, 2025
…ale independent (pythonGH-142193)

* Change to update %c to the exact time format.

---------
(cherry picked from commit 8392095)

Co-authored-by: Kir Chou <148194051+gkirchou@users.noreply.github.com>
Co-authored-by: Kir Chou <note351@hotmail.com>
@bedevere-app

bedevere-app Bot commented Dec 4, 2025

Copy link
Copy Markdown

GH-142259 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Dec 4, 2025
@vstinner

vstinner commented Dec 4, 2025

Copy link
Copy Markdown
Member

This is failing in 3.13, right? So we should backport it? Anyone know how far back it is causing issues?

Right, the issue was reported on Python 3.13.1. We should backport the change to 3.13 and 3.14 branches. Other branches no longer accept bugfixes.

vstinner pushed a commit that referenced this pull request Dec 4, 2025
…cale independent (GH-142193) (#142259)

gh-129483: Make `TestLocalTimeDisambiguation`'s time format locale independent (GH-142193)

* Change to update %c to the exact time format.

---------
(cherry picked from commit 8392095)

Co-authored-by: Kir Chou <148194051+gkirchou@users.noreply.github.com>
Co-authored-by: Kir Chou <note351@hotmail.com>
vstinner pushed a commit that referenced this pull request Dec 4, 2025
…cale independent (GH-142193) (#142258)

gh-129483: Make `TestLocalTimeDisambiguation`'s time format locale independent (GH-142193)

* Change to update %c to the exact time format.

---------
(cherry picked from commit 8392095)

Co-authored-by: Kir Chou <148194051+gkirchou@users.noreply.github.com>
Co-authored-by: Kir Chou <note351@hotmail.com>
@bedevere-bot

Copy link
Copy Markdown

StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
…ale independent (python#142193)

* Change to update %c to the exact time format.

---------

Co-authored-by: Kir Chou <note351@hotmail.com>
@gkirchou
gkirchou deleted the gh-129483 branch December 8, 2025 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants