You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As proposed in #108533 the time zone was renamed but we can't simply rename it in the test cause it fails on old(?) MacOS systems. I would propose to prepend it with:
@unittest.skipUnless(os.path.exists('/usr/share/zoneinfo/Europe/Kyiv') oros.path.exists('/usr/lib/zoneinfo/Europe/Kyiv'),"Can't find the Olson's TZ database")
I do not like relying on the particular location of the database. Although it was already here in the existing code, the comment notes that it is not a robust way. Now we have zoneinfo in the stdlib, so perhaps it could be used.
Activity
corona10 commentedon Sep 10, 2023
@serhiy-storchaka Would you like to take a look since you are the Ukranian?
Use non alternate name for Kyiv (Fixes: python#109017)
pythongh-109017: Use non alternate name for Kyiv
serhiy-storchaka commentedon Sep 11, 2023
I do not like relying on the particular location of the database. Although it was already here in the existing code, the comment notes that it is not a robust way. Now we have
zoneinfoin the stdlib, so perhaps it could be used.pythongh-109017: Use non alternate name for Kyiv
jspricke commentedon Sep 11, 2023
@serhiy-storchaka thanks for the proposal, I've updated my PR.
pythongh-109017: Use non alternate name for Kyiv
gh-109017: Use non alternate name for Kyiv (GH-109251)
pythongh-109017: Use non alternate name for Kyiv (pythonGH-109251)
pythongh-109017: Use non alternate name for Kyiv (pythonGH-109251)
[3.12] gh-109017: Use non alternate name for Kyiv (GH-109251) (GH-111280
[3.11] gh-109017: Use non alternate name for Kyiv (GH-109251) (GH-111279
pythongh-109017: Use non alternate name for Kyiv (pythonGH-109251)
pythongh-109017: Use non alternate name for Kyiv (pythonGH-109251)