[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