[Bug 207604] Re: pytz includes a hard coded list of time zone names
Benjamin Drung
207604 at bugs.launchpad.net
Thu Mar 30 15:33:37 UTC 2023
Everytime a new timezone is added to tzdata in an update, this timezone
cannot be used in pytz with the hardcoded list of timezones. For jammy
this applies to America/Ciudad_Juarez (when comparing pytz to tzdata
2023c).
Test case for jammy:
```
import pytz
pytz.timezone("America/Ciudad_Juarez")
```
This should not raise UnknownTimeZoneError.
--
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/207604
Title:
pytz includes a hard coded list of time zone names
Status in pytz:
New
Status in python-tz package in Ubuntu:
Fix Released
Status in python-tz source package in Jammy:
Incomplete
Status in python-tz source package in Kinetic:
Incomplete
Bug description:
[ Impact ]
The Debian/Ubuntu packages of pytz include a patch to not install
pytz's zoneinfo database and instead load data from the system
zoneinfo database. This keeps pytz up to date when system timezone
updates are installed which is generally a good idea.
The pytz/__init__.py includes common_timezones and all_timezones
lists, which enumerate the time zones in the database pytz was
released with. If pytz is instead using the system database, these
lists may not be correct (e.g. missing newly added time zones).
all_timezones and common_timezones should be calculated at run time.
all_timezones will need to use os.walk to discover what timezones
exist. common_timezones should be loaded from zone.tab, plus a hard
coded list of extras such as UTC and US/Eastern.
[ Test Plan ]
Two autopkgtest test cases were added:
1. Run the upstream unittests
2. Regression tests with all tests that I could come up with.
There are no manual test, because all those test should also be run
when there is a tzdata update to prevent regressions.
[ Where problems could occur ]
Dynamically determine the timezones relies on a correct environment
and can fail more easily if the environment is broken. python-tz is
used in several places (maybe in the installer). The patch might have
a performance impact, because the list of timezones is determined
completely (but only once) instead of constructing it lazy.
[ Other Info ]
Upstream recommend our approach:
https://github.com/stub42/pytz/issues/91#issuecomment-1356628324
To manage notifications about this bug go to:
https://bugs.launchpad.net/pytz/+bug/207604/+subscriptions
More information about the foundations-bugs
mailing list