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

update for python 3.10 #184

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

Conversation

allen-munsch
Copy link

@allen-munsch allen-munsch commented Oct 19, 2022

vobject didn't quite work using 3.10.7

@da4089
Copy link
Collaborator

da4089 commented Feb 29, 2024

@allen-munsch, when you found this problem, do you recall what kind of tzinfo you were using?

It looks like the code checks for tzinfo sub-classes from PyICU, pytz, and tzical.
Perhaps it was datetime.timezone.utc?

@allen-munsch
Copy link
Author

allen-munsch commented Mar 2, 2024

@da4089 i don't have access to the source code any longer, but it would have been something in the django defaults for 3.2 at the time, or, possibly pytz, perhaps something in the changelogs between 3.9/3.10.

I think it was in a custom localization function of some kind. The type it was though, i do not recall, whatever it was though definitely had a __str__/__repr__ method which returned the 'UTC' string.

I'd guess it might have been here, very likely, which aligns with your hypothesis:

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