Re: [OE-Core][RFC PATCH v3 02/13] systemd: Package udev rules explicitly

["Stefan Agner" <stefan@agner.ch>]

On 2020-03-27 18:25, Alex Kiernan wrote:
> udev is packaged before systemd so any wildcard inclusions in FILES will
> override later specifics. List all udev rules explicitly so that the
> systemd specific rules, packaged alongside systemd, appear in the
> correct package.
> 
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>

Just found that this fix broke our Plymouth based splash screen. It
seems that Plymouth relies systemd/logind style "seat" TAGS to identify
the DRM devices to show the splash screen on. So far our
initramfs-framework based initramfs just installed udev, which in turn
installed system's 71-seat.rules which set up those tags accordingly.
However, now that 71-seat.rules is no longer part of the udev package,
the "seat" TAGS isn't present anymore, and the splash screen does not
show any longer.

Not sure what the proper fix is for this, maybe creating a
systemd-udev-rules package which contains the udev rules separately, so
that they can be installed in case needed...

FWIW, I think this patch is correct. This message is more meant as a FYI
for others stumble upon such an issue and information gathering.

--
Stefan

> ---
> 
> Changes in v3: None
> Changes in v2: None
> 
>  meta/recipes-core/systemd/systemd_244.3.bb | 28 +++++++++++++++++++++-
>  1 file changed, 27 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/systemd/systemd_244.3.bb
> b/meta/recipes-core/systemd/systemd_244.3.bb
> index f0cf102dc250..809dbcb9a69b 100644
> --- a/meta/recipes-core/systemd/systemd_244.3.bb
> +++ b/meta/recipes-core/systemd/systemd_244.3.bb
> @@ -599,7 +599,33 @@ FILES_udev += "${base_sbindir}/udevd \
>                 ${rootlibexecdir}/udev/scsi_id \
>                 ${rootlibexecdir}/udev/v4l_id \
>                 ${rootlibexecdir}/udev/keymaps \
> -               ${rootlibexecdir}/udev/rules.d/*.rules \
> +               ${rootlibexecdir}/udev/rules.d/50-udev-default.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-autosuspend-chromiumos.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-block.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-cdrom_id.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-drm.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-evdev.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-fido-id.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-input-id.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-persistent-alsa.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-persistent-input.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-persistent-storage.rules \
> +              
> ${rootlibexecdir}/udev/rules.d/60-persistent-storage-tape.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-persistent-v4l.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-sensor.rules \
> +               ${rootlibexecdir}/udev/rules.d/60-serial.rules \
> +               ${rootlibexecdir}/udev/rules.d/61-autosuspend-manual.rules \
> +               ${rootlibexecdir}/udev/rules.d/64-btrfs.rules \
> +               ${rootlibexecdir}/udev/rules.d/70-joystick.rules \
> +               ${rootlibexecdir}/udev/rules.d/70-mouse.rules \
> +               ${rootlibexecdir}/udev/rules.d/70-power-switch.rules \
> +               ${rootlibexecdir}/udev/rules.d/70-touchpad.rules \
> +               ${rootlibexecdir}/udev/rules.d/75-net-description.rules \
> +               ${rootlibexecdir}/udev/rules.d/75-probe_mtd.rules \
> +               ${rootlibexecdir}/udev/rules.d/78-sound-card.rules \
> +               ${rootlibexecdir}/udev/rules.d/80-drivers.rules \
> +               ${rootlibexecdir}/udev/rules.d/80-net-setup-link.rules \
> +               ${rootlibexecdir}/udev/rules.d/90-vconsole.rules \
>                 ${sysconfdir}/udev \
>                 ${sysconfdir}/init.d/systemd-udevd \
>                 ${systemd_unitdir}/system/*udev* \
>