From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gabe Evans Date: Wed, 10 Feb 2016 22:14:49 +0000 Subject: [Buildroot] [PATCH 1/1] package/systemd: fix dbus activation In-Reply-To: <56BA7138.6000408@mind.be> References: <1454971831-11139-1-git-send-email-gabe@hashrabbit.co> <56BA7138.6000408@mind.be> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hi Arnout, On Tue, Feb 9, 2016 at 3:07 PM Arnout Vandecappelle wrote: > On 08-02-16 23:50, Gabe Evans wrote: > > Various DBus activated services fail to start with errors similar to: > > > > Unit dbus-org.freedesktop.hostname1.service failed to load: File > exists. > > > > The message itself is rather vague and can be seen as a warning in > > systemd-networkd logs. Meanwhile, tools like hostnamectl don't work > > at all. > > > > The post-install target hook SYSTEMD_SANITIZE_PATH_IN_UNITS was replacing > > symlinks with duplicate files. The sed command could have used > > --follow-symlinks to avoid this > > Or better yet, add -type f to the find command. > Right! I might have been suffering from some tunnel vision while figuring this out. > > > but I instead chose to remove the hook > > since this fix doesn't seem to be needed anymore. > > > > Some configure options are also removed: > > > > * --with-dbuspolicydir, --with-dbussessionservicedir, and > > --with-dbussystemservicedir are no longer needed since these are > > defaults in recent releases. > > > > * --disable-dbus has no effect when combined with --disable-tests. The > > option itself only applies to tests, not the runtime. > > This part really should have been in a separate patch, with as subject > "systemd: remove redundant configure options". > Still, this patch gets my > > Acked-by: Arnout Vandecappelle (Essensium/Mind) > Tested-by: Arnout Vandecappelle (Essensium/Mind) > (checked that there is not HOST_DIR reference and that the dbus paths are > still > correct). > > > If you repost as two patches, you can carry my acked/tested tags. > Thanks for your feedback. I'll split this up and send v2 when I have some free time. Thanks, Gabe > > Regards, > Arnout > > > > > Signed-off-by: Gabe Evans > > --- > > package/systemd/systemd.mk | 12 +----------- > > 1 file changed, 1 insertion(+), 11 deletions(-) > > > > diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk > > index e432069..05121cf 100644 > > --- a/package/systemd/systemd.mk > > +++ b/package/systemd/systemd.mk > > @@ -32,14 +32,10 @@ SYSTEMD_CONF_OPTS += \ > > --disable-selinux \ > > --disable-pam \ > > --disable-libcryptsetup \ > > - --with-dbuspolicydir=/etc/dbus-1/system.d \ > > - --with-dbussessionservicedir=/usr/share/dbus-1/services \ > > - --with-dbussystemservicedir=/usr/share/dbus-1/system-services \ > > --disable-efi \ > > --disable-gnuefi \ > > --disable-ldconfig \ > > --disable-tests \ > > - --disable-dbus \ > > --without-python > > > > SYSTEMD_CFLAGS = $(TARGET_CFLAGS) -fno-lto > > @@ -150,16 +146,10 @@ define SYSTEMD_INSTALL_MACHINEID_HOOK > > touch $(TARGET_DIR)/etc/machine-id > > endef > > > > -define SYSTEMD_SANITIZE_PATH_IN_UNITS > > - find $(TARGET_DIR)/lib/systemd/system -name '*.service' \ > > - -exec $(SED) 's,$(HOST_DIR),,g' {} \; > > -endef > > - > > SYSTEMD_POST_INSTALL_TARGET_HOOKS += \ > > SYSTEMD_INSTALL_INIT_HOOK \ > > SYSTEMD_INSTALL_MACHINEID_HOOK \ > > - SYSTEMD_INSTALL_RESOLVCONF_HOOK \ > > - SYSTEMD_SANITIZE_PATH_IN_UNITS > > + SYSTEMD_INSTALL_RESOLVCONF_HOOK > > > > define SYSTEMD_USERS > > systemd-journal -1 systemd-journal -1 * /var/log/journal - - > Journal > > > > > -- > Arnout Vandecappelle arnout at mind be > Senior Embedded Software Architect +32-16-286500 > Essensium/Mind http://www.mind.be > G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven > LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle > GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF > -- Gabe Evans | Co-Founder & CTO hashrabbit.co ? angel.co/hashrabbit ? github.com/gevans -------------- next part -------------- An HTML attachment was scrubbed... URL: