Hello Sebastian Ramacher. Thanks for your bug report and patch!
Judging from the severity you set I guess you don't see this as a blocker to be fixed before letting the package migrate to testing? I have a few comments about improvements to your patch below... On Mon, Oct 06, 2014 at 05:48:41PM +0200, Sebastian Ramacher wrote: [...] > Setting up libuuid1:amd64 (2.25.1-3) ... > Password: chsh: PAM: Authentication failure > dpkg: error processing package libuuid1:amd64 (--configure): [...] > diff -Nru util-linux-2.25.1/debian/libuuid1.postinst > util-linux-2.25.1/debian/libuuid1.postinst > --- util-linux-2.25.1/debian/libuuid1.postinst 2014-10-05 > 19:06:55.000000000 +0200 > +++ util-linux-2.25.1/debian/libuuid1.postinst 2014-10-06 > 17:06:13.000000000 +0200 > @@ -17,7 +17,7 @@ > if getent passwd libuuid >/dev/null && ! getent passwd uuidd >/dev/null ; > then > # switch existing libuuid user to not have a login shell. > if [ "$(getent passwd libuuid | cut -d: -f7)" = "/bin/sh" ]; then > - chsh -s /bin/false libuuid > + usermod -s /bin/false libuuid > fi > > # attempt to stop potentially running uuidd, otherwise usermod will > fail. See the final line included in the context here. --^ :) If we use the usermod approach the shell changing snippet needs to be moved further below. Please also see the comment in the top of the file, we need to make the same changes to uuid-runtime.postinst as well! Possibly we could also look at merging the shell changing into the same usermod invocation as the rest. If you're interested in working on improving the patch that would be welcome (otherwise I'll take care of it). Regards, Andreas Henriksson -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
