WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@wschaub
Copy link
Contributor

@wschaub wschaub commented Aug 1, 2014

This patch replaces the init of FixedOffset with a set method
instead. This is so we can unpickle datetime objects using this tzinfo
based class as a tzinfo= argument. you can read all about it at
https://docs.python.org/2/library/datetime.html#tzinfo-objects

I ran into this while trying to unpickle a pickled dictionary filled with
UrlSetElement objects that I had read in from the sitemap library which
had a lastmod set in the parsed sitemap you get a nasty traceback
complaining about init requiring 4 arguments but only one given with
out much useful info on what inside the pickle file caused it.

This patch replaces the __init__ of FixedOffset with a set method
instead. This is so we can unpickle datetime objects using this tzinfo
based class as a tzinfo= argument. you can read all about it at
https://docs.python.org/2/library/datetime.html#tzinfo-objects

I ran into this while trying to unpickle a pickled dictionary filled with
UrlSetElement objects that I had read in from the sitemap library which
had a lastmod set in the parsed sitemap you get a nasty traceback
complaining about __init__ requiring 4 arguments but only one given with
out much useful info on what inside the pickle file caused it.
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.

1 participant