I rejected the latest upload, too, since the changed call to adduser
looks invalid to me:

-        adduser --quiet --system --ingroup nut --ingroup dialout --home 
/var/lib/nut --no-create-home nut
+       adduser --quiet --system nut --ingroup dialout --home /var/lib/nut 
--no-create-home nut

It specifies two non-option arguments now (nut nut), which isn't valid
according to the manpage, and the semantics is unclear, too. Also, the
user won't be in group nut any more.

The changelog is inconsistent, too:

+  * debian/nut.preinst
+    - Make sure the nut user is acutally in the nut group or upsd wont start.
+      (LP: #222761)

Please document the dialout change, too, and fix the adduser calls to
put them in both groups. I suggest to make 'nut' the primary group and
add a second "adduser nut dialout", and wrap both commands into a
"getent passwd nut" check, and only do the two adduser calls if the user
didn't exist before.

-- 
[SRU] upsd doesn't start NOT 221737
https://bugs.launchpad.net/bugs/222761
You received this bug notification because you are a member of Ubuntu
Server Team, which is subscribed to nut in ubuntu.

-- 
Ubuntu-server-bugs mailing list
Ubuntu-server-bugs@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-server-bugs

Reply via email to