On Wed, Jun 05, 2019 at 02:54:05PM +0200, Juergen Borleis wrote: > Signed-off-by: Juergen Borleis <[email protected]> > --- > projectroot/usr/lib/systemd/system/connman-ignore.service | 1 + > projectroot/usr/lib/systemd/system/connman.service | 1 + > rules/connman.in | 3 ++- > rules/connman.make | 1 + > 4 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/projectroot/usr/lib/systemd/system/connman-ignore.service > b/projectroot/usr/lib/systemd/system/connman-ignore.service > index b31df07fb..df52faaed 100644 > --- a/projectroot/usr/lib/systemd/system/connman-ignore.service > +++ b/projectroot/usr/lib/systemd/system/connman-ignore.service > @@ -7,6 +7,7 @@ ConditionPathExists=!/run/connman-ignore > Type=oneshot > RemainAfterExit=yes > ExecStart=/usr/lib/systemd/connman-ignore > +StateDirectory=conman
I've dropped this patch and others that set StateDirectory= (and other directories like it) from my test queue. The problem ist that without the volatile /var the services fail to start with 9p in qemu or nfsroot: Systemd tries to change the permission of the directory and abort startup because this fails on a read-only rootfs. Michael > > [Install] > WantedBy=connman.service > diff --git a/projectroot/usr/lib/systemd/system/connman.service > b/projectroot/usr/lib/systemd/system/connman.service > index 4353276b5..c7fc639d6 100644 > --- a/projectroot/usr/lib/systemd/system/connman.service > +++ b/projectroot/usr/lib/systemd/system/connman.service > @@ -10,6 +10,7 @@ Restart=on-failure > EnvironmentFile=-/run/connman-ignore > ExecStart=/usr/sbin/connmand -n $CONNMAN_IGNORE_OPTS > StandardOutput=null > +StateDirectory=conman > > [Install] > WantedBy=multi-user.target > diff --git a/rules/connman.in b/rules/connman.in > index 9d4b1dbd3..3c2eb11a0 100644 > --- a/rules/connman.in > +++ b/rules/connman.in > @@ -13,7 +13,8 @@ menuconfig CONNMAN > select DBUS_PYTHON if CONNMAN_TESTS && RUNTIME > select WPA_SUPPLICANT if CONNMAN_WIFI && RUNTIME > select WPA_SUPPLICANT_CTRL_IFACE_DBUS if CONNMAN_WIFI > - select BUSYBOX_START_STOP_DAEMON if CONNMAN_STARTSCRIPT > + select BUSYBOX_START_STOP_DAEMON if CONNMAN_STARTSCRIPT && > RUNTIME > + select ROOTFS_VAR_LIB if RUNTIME > prompt "connman " > help > The Connection Manager project provides a daemon for > diff --git a/rules/connman.make b/rules/connman.make > index 2f671f7d4..313098d1c 100644 > --- a/rules/connman.make > +++ b/rules/connman.make > @@ -167,6 +167,7 @@ ifdef PTXCONF_CONNMAN_POLKIT > endif > > # # ship settings which enable wired interfaces per default > + @$(call install_copy, connman, 0, 0, 0755, /var/lib/connman) > @$(call install_alternative, connman, 0, 0, 0600, \ > /var/lib/connman/settings) > > -- > 2.20.1 > > > _______________________________________________ > ptxdist mailing list > [email protected] -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list [email protected]
