[Merge] lp:~nathwill/ubuntu/quantal/pam/lp110287 into lp:~ubuntu-core-dev/pam/ubuntu

Steve Langasek steve.langasek at canonical.com
Tue Jul 3 07:22:23 UTC 2012


On Tue, Jul 03, 2012 at 07:09:22AM -0000, Nathan Williams wrote:

> ugh. fixed that. thanks for catching. went ahead and flipped the dpkg
> version comparison as recommended.  left the empty string comparison,
> because i believe that accounts for the new-install scenario?

dpkg --compare-versions lt knows that an empty string "" is less than any
other version, so it already does the right thing here.  (In fact,
--compare-versions has a separate operator, 'lt-nl', to exclude empty
version strings as "not less than".)

Uploaded now, anyway :)

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slangasek at ubuntu.com                                     vorlon at debian.org

https://code.launchpad.net/~nathwill/ubuntu/quantal/pam/lp110287/+merge/112911
Your team Ubuntu Core Development Team is subscribed to branch lp:~ubuntu-core-dev/pam/ubuntu.



More information about the Ubuntu-reviews mailing list