On Thu, Apr 12, 2012 at 6:02 PM, William Grant <william.gr...@canonical.com> wrote: > On 13/04/12 06:49, Deryck Hodge wrote: >> Hi, William. >> >> I noticed we're maintaining a patched version of python-openid 2.2.1. >> versions.cfg suggests you were the last to patch it. >> >> I'm doing some work that needs the latest python-openid to work well >> with pape extension's max_auth_age. Basically, pape extension support >> in our patched version is buggy. Can you fill me in on the story about >> why we forked? Or did our patches get upstream? Is upgrading >> python-openid safe or dangerous? Do you know? >> >> It seems the forked openid code may pre-date your work, so maybe you >> don't know the complete history here either. If not, I welcome >> anyone's input here to sort out if python-openid is safe to upgrade. >> FWIW, running tests locally with general regex like "login" or "auth" >> all seem to pass with the newer python-openid. I haven't run the >> complete test suite yet to know for sure about everything. > > The only change is to fix the linked bug, > https://bugs.launchpad.net/launchpad/+bug/676372. > http://bazaar.launchpad.net/~wgrant/python-openid/python-openid-2.2.1-fix676372/revision/6 > is the diff. > > http://bazaar.launchpad.net/~keturn/python-openid/master/revision/1969 > seems to be the equivalent upstream fix, which looks to be in upstream's > 2.2.5 release. >
Awesome, we look good to upgrade then. Thanks for the follow up, William! Cheers, deryck -- Deryck Hodge https://launchpad.net/~deryck http://www.devurandom.org/ _______________________________________________ Mailing list: https://launchpad.net/~launchpad-dev Post to : launchpad-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~launchpad-dev More help : https://help.launchpad.net/ListHelp