Skip to content
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

tzutils: tzdata provides py3:tzdata #53607

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

classabbyamp
Copy link
Member

according to the Python documentation, the zoneinfo module uses the system timezone data if available, and also provides an installable tzdata library on PyPi. Some Python packages (like pandas) spuriously always depend on this library. By having tzdata provide this py3: package, it would work around Python packages that make this mistake, and avoid needing to package python3-tzdata or set noverifypydeps (which would be overkill).

@sgn, this is your package, what do you think?

Testing the changes

  • I tested the changes in this PR: YES

@sgn
Copy link
Member

sgn commented Dec 21, 2024

It doesn't matter anyway, since our python3-tzdata is almost empty and uses system zoneinfo.

@classabbyamp
Copy link
Member Author

oh, python3-pytzdata?

@classabbyamp
Copy link
Member Author

that's not the same thing as tzdata i think

@sgn
Copy link
Member

sgn commented Dec 21, 2024

Ah yes! Anyway, what is different between: provides="python3-tzdata-${version}_1" and provides="py3:tzdata-${version}_1"?

@sgn
Copy link
Member

sgn commented Dec 21, 2024

/me removes pytzdata too! #53609

@classabbyamp
Copy link
Member Author

what is different between: provides="python3-tzdata-${version}_1" and provides="py3:tzdata-${version}_1"?

py3: will get used by the python dependency verification hook, but we could have it provide both

according to the Python documentation[1], the `zoneinfo` module uses the
system timezone data if available, and also provides an installable `tzdata`
library on PyPi. Some Python packages (like pandas) spuriously always
depend on this library. By having `tzdata` provide this py3: package,
it would work around Python packages that make this mistake, and avoid
needing to package `python3-tzdata` or set `noverifypydeps` (which would
be overkill).

[1]: https://docs.python.org/3/library/zoneinfo.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants