[Merge] lp:~matsubara/ubuntu/trusty/python-urllib3/bug-1412545 into lp:ubuntu/trusty/python-urllib3

Diogo Matsubara diogo.matsubara at canonical.com
Tue Jan 27 11:11:58 UTC 2015


Hi Robie,

thanks for the review!

> Assuming the patch works, the packaging changes look good.
> 
> A few minor corrections:
> 
> Please use 1.7.1-1ubuntu0.1 as the version number from an SRU (taken from
> https://wiki.ubuntu.com/SecurityTeam/UpdatePreparation#Update_the_packaging,
> which is recommended from
> https://wiki.ubuntu.com/StableReleaseUpdates#Procedure).

Done.

> 
> Please can you add dep3 headers to the quilt patch? Described at
> http://dep.debian.net/deps/dep3/, these make it easier for other developers to
> track where the patch came from and what its current status is. I'm sorry if I
> failed to mention this in our Hangout.

You did mention, I forgot to update it but have done so now.

> 
> Finally, please can you check step 3 in
> https://wiki.ubuntu.com/StableReleaseUpdates#Procedure? The SRU team will need
> this to process the upload, so we don't upload until all steps before step 5
> are complete. As you backported the patch you're probably the best person to
> assess the potential for regressions and pass this information on the SRU
> team.

Done.
-- 
https://code.launchpad.net/~matsubara/ubuntu/trusty/python-urllib3/bug-1412545/+merge/247593
Your team Ubuntu branches is subscribed to branch lp:ubuntu/trusty/python-urllib3.



More information about the Ubuntu-reviews mailing list