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

["Richard Purdie" <richard.purdie@linuxfoundation.org>]

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