Skip to content

[3.12] gh-109017: Use non alternate name for Kyiv (GH-109251) #111280

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.

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
merged 1 commit into from
Oct 24, 2023

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 24, 2023

tzdata provides Kiev as an alternative to Kyiv:
https://sources.debian.org/src/tzdata/2023c-10/backward/?hl=314GH-L314

But Debian moved it to the tzdata-legacy package breaking the test:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050530

This patch switches to the name provided by tzdata.

Also check that the new name is actually available.
(cherry picked from commit 46407fe)

Co-authored-by: Jochen Sprickerhof github@jochen.sprickerhof.de

tzdata provides Kiev as an alternative to Kyiv:
https://sources.debian.org/src/tzdata/2023c-10/backward/?hl=314GH-L314

But Debian moved it to the tzdata-legacy package breaking the test:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050530

This patch switches to the name provided by tzdata.

Also check that the new name is actually available.
(cherry picked from commit 46407fe)

Co-authored-by: Jochen Sprickerhof <github@jochen.sprickerhof.de>
@miss-islington miss-islington requested a review from a team as a code owner October 24, 2023 17:13
@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Oct 24, 2023
@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) October 24, 2023 17:14
@serhiy-storchaka serhiy-storchaka merged commit b6e508c into python:3.12 Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants