Bug#823322: please build "systemd-sysusers" binary

Felipe Sateler fsateler at debian.org
Thu May 5 13:55:54 BST 2016


On 5 May 2016 at 00:35, Martin Pitt <mpitt at debian.org> wrote:
> Hey Felipe,
>
> Felipe Sateler [2016-05-03 10:06 -0300]:
>> So, if we don't want to get in the business of defining users, just
>> providing the binary would be sufficient for this purpose.
>
> TBH, I see little reason to not just enable it wholesale [1]. There
> are no new build or binary dependencies, it does not change the
> default behaviour as the .service is ConditionNeedsUpdate=/etc (which
> will practically never trigger -- we don't support empty /etc in
> Debian yet), but this introduces useful functionality for moving the
> adduser business to become declarative. The main downside is that it
> increases the package size by ~ 380 kB (uncompressed).

Hmm, I can't say I'm happy to enlarge the base package installed
everywhere... but 380k is not that much.

>
> If we do this, we should do it correctly though and patch
> /usr/lib/sysusers.d/basic.conf to have Debian's static gids and remove
> wheel.

I think we should instead, (possibly at build time), parse
/usr/share/base-passwd/{group,passwd}.master, and insert a new file
debian-static.conf. This should make it relatively easy to follow
changes from the distro:

cat /usr/share/base-passwd/passwd.master | \
  awk 'BEGIN { FS=":" } ; { print "u " $1 " " $3 " \"" $5 "\" " $6 " " $7 }'
# mutatis mutandis for group
cat /usr/share/base-passwd/passwd.master | \
  awk 'BEGIN { FS=":" } ; { print "m " $1 " " $4  }'
# Although I'm not sure sysusers accepts numeric groups here...

Keeping the list in sync will otherwise be a pain.

-- 

Saludos,
Felipe Sateler




More information about the Pkg-systemd-maintainers mailing list