Re: [OE-core] [PATCH v3 1/5] systemd: update 246 -> 247

Richard Purdie
2020-12-19T14:55:51Z

On Sat, 2020-12-19 at 14:09 +0000, Peter Kjellerstedt wrote:
> Now this made me feel really stupid. We have had the following lines
> in our
> systemd_%.bbappend for years and I never got around to upstream them:

I do worry people have useful fixes lying around :/.

> # FIXME: These are incorrectly determined by meson based on the
> host's
> # /etc/login.defs file. If there is a way to get meson to do the
> right thing
> # when configuring for cross compilation, then that should be
> upstreamed to
> # systemd, otherwise this change should be upstreamed to OE.
> EXTRA_OEMESON += "-Dsystem-uid-max=999 \
> -Dsystem-gid-max=999"
>
> Sorry about that.
>
> > [rpurdie@centos8-ty-1 ~]$ grep SYS /etc/login.defs
> > SYS_UID_MIN 201
> > SYS_UID_MAX 999
> > SYS_GID_MIN 201
> > SYS_GID_MAX 999
> >
> > so I think we have our trigger. I have a simply patch to specify
> > defaults in the recipe which I think should resolve this. I'll add
> > it
> > to master-next and see if that helps.
>
> Looking at your patch in master-next, I believe you should set
> -Dsystem-alloc-uid-min and -Dsystem-alloc-gid-min to 101 rather than
> 1
> to match what is in /etc/login.defs.

Good point, I'll tweak accordingly, thanks.

Cheers,

Richard