Merge lp://qastaging/~nathwill-deactivatedaccount-deactivatedaccount/ubuntu/quantal/pam/lp110287 into lp://qastaging/~ubuntu-core-dev/pam/ubuntu

Thanks, this patch looks good. Committing and uploading!

FWIW:

-if [ "$1" = "configure" ] && (dpkg --compare-versions 0.79-3ubuntu6 ge "$2" || [ "$2" = "" ]); then
+if [ "$1" = "configure" ] && (dpkg --compare-versions 1.1.3-7ubuntu2 ge "$2" || [ "$2" = "" ]); then

I think the original wasn't very well written here, and I'm going to clean it up as part of the merge to read:

+if [ "$1" = "configure" ] && dpkg --compare-versions "$2" le 1.1.3-7ubuntu2; then

In fact, make that...

if [ "$1" = "configure" ] && dpkg --compare-versions "$2" lt 1.1.3-7ubuntu3; then

thus keying on the version that *does* introduce the change, rather than on the last known version that doesn't.

Whoops, one more issue, caught on package install test:

- elif ! grep -qs "^PATH.*/usr/local/games"; then
+ elif ! grep -qs "^PATH.*/usr/local/games" /etc/environment; then

postinst hanging forever is not the desired behavior :)

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?

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/
<email address hidden> <email address hidden>