<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Yann E. MORIN <<a href="mailto:yann.morin.1998@free.fr">yann.morin.1998@free.fr</a>> schrieb am Sa., 11. Juli 2020, 13:27:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Norbert, All,<br>
<br>
On 2020-07-11 00:26 +0200, Norbert Lange spake thusly:<br>
> udevadm has been deprecated for a while(for bulding the DB),<br>
> systemd-hwdb should be used in its place.<br>
> <br>
> this tool further has a switch to generate the database in<br>
> /usr which is a better place for a one-time created file.<br>
<br>
... rather than the default /etc/udev.<br>
<br>
> Also remove the config files from both paths<br>
> (rootfs overlay could add stuff) aswell as the service and tool<br>
> from the target fs.<br>
<br>
This 'also' hints at the need to split the patch in two: one to switch<br>
to using systemd-hwdb, and one to remove the service.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ok.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
However, I'm against removing the service altogether, because in the<br>
past, some people have expressed the need to be able to update the hwdb<br>
on-target.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That would be challenging, as the source files for the database were already removed before this patch.</div><div dir="auto"><br></div><div dir="auto">If they had to re-add those, then a adding a tool and service from the target directory won't be too much to ask?</div><div dir="auto"><br></div><div dir="auto">I mean buildroot pretty much doesn't support some sorta packet manager system by design.</div><div dir="auto">Why the exception here?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I would be OK with having a drop-in that disables the service by default<br>
when BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW is not set, though.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">It will only run when /usr is newer than /etc AFAIR (not sure how this is determined), so disabling does very little.</div><div dir="auto"><br></div><div dir="auto">BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW already has too many </div><div dir="auto">surprising effects for me.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> Signed-off-by: Norbert Lange <<a href="mailto:nolange79@gmail.com" target="_blank" rel="noreferrer">nolange79@gmail.com</a>><br>
> <br>
> ---<br>
> v1 -> v2<br>
> -   fix removal of service<br>
> ---<br>
>  package/systemd/<a href="http://systemd.mk" rel="noreferrer noreferrer" target="_blank">systemd.mk</a> | 9 ++++++---<br>
>  1 file changed, 6 insertions(+), 3 deletions(-)<br>
> <br>
> diff --git a/package/systemd/<a href="http://systemd.mk" rel="noreferrer noreferrer" target="_blank">systemd.mk</a> b/package/systemd/<a href="http://systemd.mk" rel="noreferrer noreferrer" target="_blank">systemd.mk</a><br>
> index 1a8eb86e53..05c1d1ff9a 100644<br>
> --- a/package/systemd/<a href="http://systemd.mk" rel="noreferrer noreferrer" target="_blank">systemd.mk</a><br>
> +++ b/package/systemd/<a href="http://systemd.mk" rel="noreferrer noreferrer" target="_blank">systemd.mk</a><br>
> @@ -229,11 +229,14 @@ endif<br>
>  ifeq ($(BR2_PACKAGE_SYSTEMD_HWDB),y)<br>
>  SYSTEMD_CONF_OPTS += -Dhwdb=true<br>
>  define SYSTEMD_BUILD_HWDB<br>
> -     $(HOST_DIR)/bin/udevadm hwdb --update --root $(TARGET_DIR)<br>
> +     $(HOST_DIR)/bin/systemd-hwdb update --root $(TARGET_DIR) --usr<br>
<br>
Don;t we want to use --strict too?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ok.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
Yann E. MORIN.<br>
<br>
>  endef<br>
>  SYSTEMD_TARGET_FINALIZE_HOOKS += SYSTEMD_BUILD_HWDB<br>
>  define SYSTEMD_RM_HWDB_SRV<br>
> -     rm -rf $(TARGET_DIR)/$(HOST_EUDEV_SYSCONFDIR)/udev/hwdb.d/<br>
> +     rm -rf $(TARGET_DIR)/usr/lib/udev/hwdb.d/ $(TARGET_DIR)/etc/udev/hwdb.d/ \<br>
> +             $(TARGET_DIR)/usr/lib/systemd/system/systemd-hwdb-update.service \<br>
> +             $(TARGET_DIR)/usr/lib/systemd/system/*/systemd-hwdb-update.service \<br>
> +             $(TARGET_DIR)/usr/bin/systemd-hwdb<br>
>  endef<br>
>  SYSTEMD_ROOTFS_PRE_CMD_HOOKS += SYSTEMD_RM_HWDB_SRV<br>
>  else<br>
> @@ -633,7 +636,7 @@ HOST_SYSTEMD_CONF_OPTS = \<br>
>       -Dsysusers=false \<br>
>       -Dtmpfiles=false \<br>
>       -Dimportd=false \<br>
> -     -Dhwdb=false \<br>
> +     -Dhwdb=true \<br>
>       -Drfkill=false \<br>
>       -Dman=false \<br>
>       -Dhtml=false \<br>
> -- <br>
> 2.27.0<br>
> <br>
<br>
-- <br>
.-----------------.--------------------.------------------.--------------------.<br>
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |<br>
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |<br>
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |<br>
| <a href="http://ymorin.is-a-geek.org/" rel="noreferrer noreferrer" target="_blank">http://ymorin.is-a-geek.org/</a> | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |<br>
'------------------------------^-------^------------------^--------------------'<br>
</blockquote></div></div></div>