Skip to content

gh-84683: Check in <prefix>/share/zoneinfo for zoneinfo files on Windows - #28495

Merged
zooba merged 6 commits into
python:mainfrom
isuruf:zoneinfo
Aug 6, 2025
Merged

gh-84683: Check in <prefix>/share/zoneinfo for zoneinfo files on Windows#28495
zooba merged 6 commits into
python:mainfrom
isuruf:zoneinfo

Conversation

@isuruf

@isurufisuruf commented Sep 21, 2021

Copy link
Copy Markdown
Contributor

…dows

This is a relocatable path and conda uses this path for zoneinfo files on windows.

https://bugs.python.org/issue40503

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actionsgithub-actionsBot added the stale Stale PR or inactive for long period of time. label Oct 22, 2021
Comment threadLib/sysconfig.py Outdated
@zooba

Copy link
Copy Markdown
Member

LGTM, though I'd like to have a comment at least acknowledging that this path is one guess, and not in any way a standard or expectation.

There's also a merge conflict of some kind.

@StanFromIrelandStanFromIreland changed the title bpo-40503: Check in <prefix>/share/zoneinfo for zoneinfo files on windowsgh-84683: Check in <prefix>/share/zoneinfo for zoneinfo files on WindowsJul 7, 2025
@StanFromIreland

Copy link
Copy Markdown
Member

@zooba, do you still want to implement this? I can fix the conflict.

@zooba

zooba commented Jul 7, 2025

Copy link
Copy Markdown
Member

Yeah, it seems a reasonable thing to allow.

@StanFromIrelandStanFromIreland removed the stale Stale PR or inactive for long period of time. label Jul 8, 2025
@isuruf

Copy link
Copy Markdown
ContributorAuthor

@StanFromIreland thanks for rebasing. Ready for a review again.

@zooba
zooba merged commit 3000594 into python:mainAug 6, 2025
42 checks passed
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@isuruf@zooba@StanFromIreland@serhiy-storchaka@the-knights-who-say-ni@ezio-melotti@bedevere-bot