[Buildroot] RFC: systemd and service files cleanup

[Buildroot] RFC: systemd and service files cleanup

[Mike Williams]

All,

Currently, package service files for systemd are installed by buildroot in a variety of locations: /etc, /lib, and /usr/lib. For systemd, split /lib is deprecated and only checked if a split /lib and /usr/lib is enabled. /etc is meant for local customizations to the default service files. It is my opinion that buildroot should install all upstream package and buildroot-provided service files in /usr/lib, and this series cleans up our packages to do exactly that. Enabling them by default by linking them to /etc/systemd/system/multi-user.target.wants has been preserved.

Upstream systemd's standard installation directory for its binaries is /usr/lib. It appears that historically, buildroot's systemd installation location has been switched between /lib and /usr/lib multiple times. I couldn't find any particular reason for this, so I've moved it back to /usr/lib along with its service files.

Anyway, please send comments :).

Mike Williams

[Buildroot] [PATCH 01/15] ntp: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/ntp/ntp.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/ntp/ntp.mk b/package/ntp/ntp.mk
index 7d5644a..0e42f15 100644
--- a/package/ntp/ntp.mk
+++ b/package/ntp/ntp.mk
@@ -71,9 +71,9 @@ define NTP_INSTALL_INIT_SYSV
 endef
 
 define NTP_INSTALL_INIT_SYSTEMD
-	$(INSTALL) -D -m 644 package/ntp/ntpd.service $(TARGET_DIR)/etc/systemd/system/ntpd.service
+	$(INSTALL) -D -m 644 package/ntp/ntpd.service $(TARGET_DIR)/usr/lib/systemd/system/ntpd.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
+	ln -fs /usr/lib/systemd/system/ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
 endef
 endif
 
-- 
2.1.0

[Buildroot] [PATCH 01/15] ntp: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/ntp/ntp.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/ntp/ntp.mk b/package/ntp/ntp.mk
> index 7d5644a..0e42f15 100644
> --- a/package/ntp/ntp.mk
> +++ b/package/ntp/ntp.mk
> @@ -71,9 +71,9 @@ define NTP_INSTALL_INIT_SYSV
>  endef
>
>  define NTP_INSTALL_INIT_SYSTEMD
> -       $(INSTALL) -D -m 644 package/ntp/ntpd.service $(TARGET_DIR)/etc/systemd/system/ntpd.service
> +       $(INSTALL) -D -m 644 package/ntp/ntpd.service $(TARGET_DIR)/usr/lib/systemd/system/ntpd.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
> +       ln -fs /usr/lib/systemd/system/ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
>  endef
>  endif
>

[Buildroot] [PATCH 01/15] ntp: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:53 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 02/15] openvmtools: move systemd service to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/openvmtools/openvmtools.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
index e0ec6d6..84d9a81 100644
--- a/package/openvmtools/openvmtools.mk
+++ b/package/openvmtools/openvmtools.mk
@@ -67,9 +67,9 @@ endef
 
 define OPENVMTOOLS_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/openvmtools/vmtoolsd.service \
-		$(TARGET_DIR)/etc/systemd/system/vmtoolsd.service
+		$(TARGET_DIR)/usr/lib/systemd/system/vmtoolsd.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../vmtoolsd.service \
+	ln -fs /usr/lib/systemd/system/vmtoolsd.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/vmtoolsd.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 02/15] openvmtools: move systemd service to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/openvmtools/openvmtools.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
> index e0ec6d6..84d9a81 100644
> --- a/package/openvmtools/openvmtools.mk
> +++ b/package/openvmtools/openvmtools.mk
> @@ -67,9 +67,9 @@ endef
>
>  define OPENVMTOOLS_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/openvmtools/vmtoolsd.service \
> -               $(TARGET_DIR)/etc/systemd/system/vmtoolsd.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/vmtoolsd.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../vmtoolsd.service \
> +       ln -fs /usr/lib/systemd/system/vmtoolsd.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/vmtoolsd.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 02/15] openvmtools: move systemd service to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:54 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 03/15] psplash: move systemd service files to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/psplash/psplash.mk | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/package/psplash/psplash.mk b/package/psplash/psplash.mk
index 5901547..866aa9f 100644
--- a/package/psplash/psplash.mk
+++ b/package/psplash/psplash.mk
@@ -11,15 +11,15 @@ PSPLASH_AUTORECONF = YES
 
 define PSPLASH_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/psplash/psplash-start.service \
-		$(TARGET_DIR)/etc/systemd/system/psplash-start.service
-	$(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants
-	ln -sf  ../psplash-start.service \
+		$(TARGET_DIR)/usr/lib/systemd/system/psplash-start.service
+	$(INSTALL) -d $(TARGET_DIR)/usr/lib/systemd/system/sysinit.target.wants
+	ln -sf  /usr/lib/systemd/system/psplash-start.service \
 		 $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants/
 
 	$(INSTALL) -D -m 644 package/psplash/psplash-quit.service \
-		$(TARGET_DIR)/etc/systemd/system/psplash-quit.service
+		$(TARGET_DIR)/usr/lib/systemd/system/psplash-quit.service
 	$(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -sf  ../psplash-quit.service \
+	ln -sf  /usr/lib/systemd/system/psplash-quit.service \
 		 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 03/15] psplash: move systemd service files to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/psplash/psplash.mk | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/package/psplash/psplash.mk b/package/psplash/psplash.mk
> index 5901547..866aa9f 100644
> --- a/package/psplash/psplash.mk
> +++ b/package/psplash/psplash.mk
> @@ -11,15 +11,15 @@ PSPLASH_AUTORECONF = YES
>
>  define PSPLASH_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/psplash/psplash-start.service \
> -               $(TARGET_DIR)/etc/systemd/system/psplash-start.service
> -       $(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants
> -       ln -sf  ../psplash-start.service \
> +               $(TARGET_DIR)/usr/lib/systemd/system/psplash-start.service
> +       $(INSTALL) -d $(TARGET_DIR)/usr/lib/systemd/system/sysinit.target.wants
> +       ln -sf  /usr/lib/systemd/system/psplash-start.service \
>                  $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants/
>
>         $(INSTALL) -D -m 644 package/psplash/psplash-quit.service \
> -               $(TARGET_DIR)/etc/systemd/system/psplash-quit.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/psplash-quit.service
>         $(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -sf  ../psplash-quit.service \
> +       ln -sf  /usr/lib/systemd/system/psplash-quit.service \
>                  $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 03/15] psplash: move systemd service files to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:55 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 04/15] openntpd: move systemd service files to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/openntpd/openntpd.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk
index 60f025c..dcd7c7f 100644
--- a/package/openntpd/openntpd.mk
+++ b/package/openntpd/openntpd.mk
@@ -11,9 +11,9 @@ OPENNTPD_LICENSE_FILES = COPYING
 
 define OPENNTPD_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 0644 package/openntpd/ntpd.service \
-		$(TARGET_DIR)/etc/systemd/system/ntpd.service
+		$(TARGET_DIR)/usr/lib/systemd/system/ntpd.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../ntpd.service \
+	ln -fs /usr/lib/systemd/system/ntpd.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 04/15] openntpd: move systemd service files to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/openntpd/openntpd.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk
> index 60f025c..dcd7c7f 100644
> --- a/package/openntpd/openntpd.mk
> +++ b/package/openntpd/openntpd.mk
> @@ -11,9 +11,9 @@ OPENNTPD_LICENSE_FILES = COPYING
>
>  define OPENNTPD_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 0644 package/openntpd/ntpd.service \
> -               $(TARGET_DIR)/etc/systemd/system/ntpd.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/ntpd.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../ntpd.service \
> +       ln -fs /usr/lib/systemd/system/ntpd.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 04/15] openntpd: move systemd service files to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:56 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 05/15] dbus: move systemd service files to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/dbus/dbus.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/dbus/dbus.mk b/package/dbus/dbus.mk
index 200084f..c810800 100644
--- a/package/dbus/dbus.mk
+++ b/package/dbus/dbus.mk
@@ -54,7 +54,7 @@ endif
 ifeq ($(BR2_INIT_SYSTEMD),y)
 DBUS_CONF_OPTS += \
 	--enable-systemd \
-	--with-systemdsystemunitdir=/lib/systemd/system
+	--with-systemdsystemunitdir=/usr/lib/systemd/system
 DBUS_DEPENDENCIES += systemd
 else
 DBUS_CONF_OPTS += --disable-systemd
-- 
2.1.0

[Buildroot] [PATCH 05/15] dbus: move systemd service files to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/dbus/dbus.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/dbus/dbus.mk b/package/dbus/dbus.mk
> index 200084f..c810800 100644
> --- a/package/dbus/dbus.mk
> +++ b/package/dbus/dbus.mk
> @@ -54,7 +54,7 @@ endif
>  ifeq ($(BR2_INIT_SYSTEMD),y)
>  DBUS_CONF_OPTS += \
>         --enable-systemd \
> -       --with-systemdsystemunitdir=/lib/systemd/system
> +       --with-systemdsystemunitdir=/usr/lib/systemd/system
>  DBUS_DEPENDENCIES += systemd
>  else
>  DBUS_CONF_OPTS += --disable-systemd
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 05/15] dbus: move systemd service files to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:56 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 06/15] avahi: systemd cleanups

[Mike Williams]

* Move service files to /usr/lib/systemd/system/
* Only disable systemd support on non-systemd systems

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/avahi/0002-disable-systemd-support.patch | 22 ----------------------
 package/avahi/avahi.mk                           |  8 ++++++--
 2 files changed, 6 insertions(+), 24 deletions(-)
 delete mode 100644 package/avahi/0002-disable-systemd-support.patch

diff --git a/package/avahi/0002-disable-systemd-support.patch b/package/avahi/0002-disable-systemd-support.patch
deleted file mode 100644
index 909795a..0000000
--- a/package/avahi/0002-disable-systemd-support.patch
+++ /dev/null
@@ -1,22 +0,0 @@
-[PATCH] avahi-daemon: disable systemd support
-
-Disable systemd support as it isn't needed for BR, and uses SOCK_CLOEXEC
-which isn't available on uClibc.
-
-Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
----
- avahi-daemon/sd-daemon.h |    1 +
- 1 file changed, 1 insertion(+)
-
-Index: avahi-0.6.27/avahi-daemon/sd-daemon.h
-===================================================================
---- avahi-0.6.27.orig/avahi-daemon/sd-daemon.h
-+++ avahi-0.6.27/avahi-daemon/sd-daemon.h
-@@ -66,6 +66,7 @@ extern "C" {
- 
-   See sd-daemon(7) for more information.
- */
-+#define DISABLE_SYSTEMD /* no systemd support in BR */
- 
- #if __GNUC__ >= 4
- #define _sd_printf_attr_(a,b) __attribute__ ((format (printf, a, b)))
diff --git a/package/avahi/avahi.mk b/package/avahi/avahi.mk
index 10c9915..3f0cbe9 100644
--- a/package/avahi/avahi.mk
+++ b/package/avahi/avahi.mk
@@ -88,6 +88,10 @@ AVAHI_DEPENDENCIES = \
 	$(if $(BR2_NEEDS_GETTEXT_IF_LOCALE),gettext) host-intltool \
 	host-pkgconf host-gettext
 
+ifneq ($(BR2_PACKAGE_SYSTEMD),y)
+AVAHI_EXTRA_CFLAGS += -DDISABLE_SYSTEMD
+endif
+
 ifneq ($(BR2_PACKAGE_AVAHI_DAEMON)$(BR2_PACKAGE_AVAHI_AUTOIPD),)
 AVAHI_DEPENDENCIES += libdaemon
 else
@@ -185,10 +189,10 @@ ifeq ($(BR2_PACKAGE_AVAHI_DAEMON),y)
 define AVAHI_INSTALL_INIT_SYSTEMD
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
 
-	ln -fs /lib/systemd/system/avahi-daemon.service \
+	ln -fs /usr/lib/systemd/system/avahi-daemon.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/avahi-daemon.service
 
-	ln -fs /lib/systemd/system/avahi-dnsconfd.service \
+	ln -fs /usr/lib/systemd/system/avahi-dnsconfd.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/avahi-dnsconfd.service
 
 	$(INSTALL) -D -m 644 package/avahi/avahi_tmpfiles.conf \
-- 
2.1.0

[Buildroot] [PATCH 06/15] avahi: systemd cleanups

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> * Move service files to /usr/lib/systemd/system/
> * Only disable systemd support on non-systemd systems
>
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/avahi/0002-disable-systemd-support.patch | 22 ----------------------
>  package/avahi/avahi.mk                           |  8 ++++++--
>  2 files changed, 6 insertions(+), 24 deletions(-)
>  delete mode 100644 package/avahi/0002-disable-systemd-support.patch
>
> diff --git a/package/avahi/0002-disable-systemd-support.patch b/package/avahi/0002-disable-systemd-support.patch
> deleted file mode 100644
> index 909795a..0000000
> --- a/package/avahi/0002-disable-systemd-support.patch
> +++ /dev/null
> @@ -1,22 +0,0 @@
> -[PATCH] avahi-daemon: disable systemd support
> -
> -Disable systemd support as it isn't needed for BR, and uses SOCK_CLOEXEC
> -which isn't available on uClibc.
> -
> -Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> ----
> - avahi-daemon/sd-daemon.h |    1 +
> - 1 file changed, 1 insertion(+)
> -
> -Index: avahi-0.6.27/avahi-daemon/sd-daemon.h
> -===================================================================
> ---- avahi-0.6.27.orig/avahi-daemon/sd-daemon.h
> -+++ avahi-0.6.27/avahi-daemon/sd-daemon.h
> -@@ -66,6 +66,7 @@ extern "C" {
> -
> -   See sd-daemon(7) for more information.
> - */
> -+#define DISABLE_SYSTEMD /* no systemd support in BR */
> -
> - #if __GNUC__ >= 4
> - #define _sd_printf_attr_(a,b) __attribute__ ((format (printf, a, b)))
> diff --git a/package/avahi/avahi.mk b/package/avahi/avahi.mk
> index 10c9915..3f0cbe9 100644
> --- a/package/avahi/avahi.mk
> +++ b/package/avahi/avahi.mk
> @@ -88,6 +88,10 @@ AVAHI_DEPENDENCIES = \
>         $(if $(BR2_NEEDS_GETTEXT_IF_LOCALE),gettext) host-intltool \
>         host-pkgconf host-gettext
>
> +ifneq ($(BR2_PACKAGE_SYSTEMD),y)
> +AVAHI_EXTRA_CFLAGS += -DDISABLE_SYSTEMD
> +endif
> +
>  ifneq ($(BR2_PACKAGE_AVAHI_DAEMON)$(BR2_PACKAGE_AVAHI_AUTOIPD),)
>  AVAHI_DEPENDENCIES += libdaemon
>  else
> @@ -185,10 +189,10 @@ ifeq ($(BR2_PACKAGE_AVAHI_DAEMON),y)
>  define AVAHI_INSTALL_INIT_SYSTEMD
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>
> -       ln -fs /lib/systemd/system/avahi-daemon.service \
> +       ln -fs /usr/lib/systemd/system/avahi-daemon.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/avahi-daemon.service
>
> -       ln -fs /lib/systemd/system/avahi-dnsconfd.service \
> +       ln -fs /usr/lib/systemd/system/avahi-dnsconfd.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/avahi-dnsconfd.service
>
>         $(INSTALL) -D -m 644 package/avahi/avahi_tmpfiles.conf \
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 06/15] avahi: systemd cleanups

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:57 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> * Move service files to /usr/lib/systemd/system/
>> * Only disable systemd support on non-systemd systems
>>
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 06/15] avahi: systemd cleanups

[Thomas Petazzoni]

Dear Mike Williams,

On Thu, 19 Mar 2015 13:56:38 -0400, Mike Williams wrote:

> +ifneq ($(BR2_PACKAGE_SYSTEMD),y)
> +AVAHI_EXTRA_CFLAGS += -DDISABLE_SYSTEMD
> +endif

This cannot work, since AVAHI_EXTRA_CFLAGS is not used by anything.

I'm surprised you got a Reviewed-by from Samuel and Steven with this
big issue.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 06/15] avahi: systemd cleanups

[Samuel Martin]

On Fri, Mar 20, 2015 at 2:15 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Mike Williams,
>
> On Thu, 19 Mar 2015 13:56:38 -0400, Mike Williams wrote:
>
>> +ifneq ($(BR2_PACKAGE_SYSTEMD),y)
>> +AVAHI_EXTRA_CFLAGS += -DDISABLE_SYSTEMD
>> +endif
>
> This cannot work, since AVAHI_EXTRA_CFLAGS is not used by anything.
>
> I'm surprised you got a Reviewed-by from Samuel and Steven with this
> big issue.

argh! right! I missed that :-s


-- 
Samuel

[Buildroot] [PATCH 07/15] irqbalance: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/irqbalance/irqbalance.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/irqbalance/irqbalance.mk b/package/irqbalance/irqbalance.mk
index e3ef0bb..536d0e0 100644
--- a/package/irqbalance/irqbalance.mk
+++ b/package/irqbalance/irqbalance.mk
@@ -26,9 +26,9 @@ endef
 
 define IRQBALANCE_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/irqbalance/irqbalance.service \
-		$(TARGET_DIR)/etc/systemd/system/irqbalance.service
+		$(TARGET_DIR)/usr/lib/systemd/system/irqbalance.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../irqbalance.service \
+	ln -fs /usr/lib/systemd/system/irqbalance.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/irqbalance.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 07/15] irqbalance: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/irqbalance/irqbalance.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/irqbalance/irqbalance.mk b/package/irqbalance/irqbalance.mk
> index e3ef0bb..536d0e0 100644
> --- a/package/irqbalance/irqbalance.mk
> +++ b/package/irqbalance/irqbalance.mk
> @@ -26,9 +26,9 @@ endef
>
>  define IRQBALANCE_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/irqbalance/irqbalance.service \
> -               $(TARGET_DIR)/etc/systemd/system/irqbalance.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/irqbalance.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../irqbalance.service \
> +       ln -fs /usr/lib/systemd/system/irqbalance.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/irqbalance.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 07/15] irqbalance: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:58 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 08/15] openssh: move systemd service files to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/openssh/openssh.mk | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/package/openssh/openssh.mk b/package/openssh/openssh.mk
index 08f3fa0..eccfad1 100644
--- a/package/openssh/openssh.mk
+++ b/package/openssh/openssh.mk
@@ -36,10 +36,10 @@ endif
 
 define OPENSSH_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/openssh/sshd.service \
-		$(TARGET_DIR)/etc/systemd/system/sshd.service
+		$(TARGET_DIR)/usr/lib/systemd/system/sshd.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../sshd.service \
-		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/sshd.service
+	ln -fs /usr/lib/systemd/system/sshd.service \
+		$(TARGET_DIR)/usr/lib/systemd/system/multi-user.target.wants/sshd.service
 endef
 
 define OPENSSH_INSTALL_INIT_SYSV
-- 
2.1.0

[Buildroot] [PATCH 08/15] openssh: move systemd service files to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/openssh/openssh.mk | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/package/openssh/openssh.mk b/package/openssh/openssh.mk
> index 08f3fa0..eccfad1 100644
> --- a/package/openssh/openssh.mk
> +++ b/package/openssh/openssh.mk
> @@ -36,10 +36,10 @@ endif
>
>  define OPENSSH_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/openssh/sshd.service \
> -               $(TARGET_DIR)/etc/systemd/system/sshd.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/sshd.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../sshd.service \
> -               $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/sshd.service
> +       ln -fs /usr/lib/systemd/system/sshd.service \
> +               $(TARGET_DIR)/usr/lib/systemd/system/multi-user.target.wants/sshd.service
>  endef
>
>  define OPENSSH_INSTALL_INIT_SYSV
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 08/15] openssh: move systemd service files to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:58 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,


-- 
Samuel

[Buildroot] [PATCH 09/15] rsyslog: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/rsyslog/rsyslog.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/rsyslog/rsyslog.mk b/package/rsyslog/rsyslog.mk
index c6758eb..7bc6a67 100644
--- a/package/rsyslog/rsyslog.mk
+++ b/package/rsyslog/rsyslog.mk
@@ -65,7 +65,7 @@ define RSYSLOG_INSTALL_INIT_SYSV
 endef
 
 define RSYSLOG_INSTALL_INIT_SYSTEMD
-	ln -sf /lib/systemd/system/rsyslog.service \
+	ln -sf /usr/lib/systemd/system/rsyslog.service \
 		$(TARGET_DIR)/etc/systemd/system/syslog.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
 	ln -sf ../syslog.service \
-- 
2.1.0

[Buildroot] [PATCH 09/15] rsyslog: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
> ---
>  package/rsyslog/rsyslog.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/rsyslog/rsyslog.mk b/package/rsyslog/rsyslog.mk
> index c6758eb..7bc6a67 100644
> --- a/package/rsyslog/rsyslog.mk
> +++ b/package/rsyslog/rsyslog.mk
> @@ -65,7 +65,7 @@ define RSYSLOG_INSTALL_INIT_SYSV
>  endef
>
>  define RSYSLOG_INSTALL_INIT_SYSTEMD
> -       ln -sf /lib/systemd/system/rsyslog.service \
> +       ln -sf /usr/lib/systemd/system/rsyslog.service \
>                 $(TARGET_DIR)/etc/systemd/system/syslog.service

Nothing needs to change in rsyslog to make it install the unit to this location?

>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>         ln -sf ../syslog.service \
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 09/15] rsyslog: move systemd service file to /usr/lib

[Samuel Martin]

Hi Mike,

On Thu, Mar 19, 2015 at 6:56 PM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
> ---
>  package/rsyslog/rsyslog.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/rsyslog/rsyslog.mk b/package/rsyslog/rsyslog.mk
> index c6758eb..7bc6a67 100644
> --- a/package/rsyslog/rsyslog.mk
> +++ b/package/rsyslog/rsyslog.mk
> @@ -65,7 +65,7 @@ define RSYSLOG_INSTALL_INIT_SYSV
>  endef
>
>  define RSYSLOG_INSTALL_INIT_SYSTEMD
> -       ln -sf /lib/systemd/system/rsyslog.service \
> +       ln -sf /usr/lib/systemd/system/rsyslog.service \
>                 $(TARGET_DIR)/etc/systemd/system/syslog.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>         ln -sf ../syslog.service \
You also need to update the symlink target here.

Regards,

-- 
Samuel

[Buildroot] [PATCH 10/15] dropbear: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/dropbear/dropbear.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
index f2ae403..760fff2 100644
--- a/package/dropbear/dropbear.mk
+++ b/package/dropbear/dropbear.mk
@@ -53,9 +53,9 @@ endef
 
 define DROPBEAR_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/dropbear/dropbear.service \
-		$(TARGET_DIR)/lib/systemd/system/dropbear.service
+		$(TARGET_DIR)/usr/lib/systemd/system/dropbear.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs /lib/systemd/system/dropbear.service \
+	ln -fs /usr/lib/systemd/system/dropbear.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/dropbear.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 10/15] dropbear: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/dropbear/dropbear.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
> index f2ae403..760fff2 100644
> --- a/package/dropbear/dropbear.mk
> +++ b/package/dropbear/dropbear.mk
> @@ -53,9 +53,9 @@ endef
>
>  define DROPBEAR_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/dropbear/dropbear.service \
> -               $(TARGET_DIR)/lib/systemd/system/dropbear.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/dropbear.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs /lib/systemd/system/dropbear.service \
> +       ln -fs /usr/lib/systemd/system/dropbear.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/dropbear.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 10/15] dropbear: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:59 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,

-- 
Samuel

[Buildroot] [PATCH 11/15] postgresql: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/postgresql/postgresql.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/postgresql/postgresql.mk b/package/postgresql/postgresql.mk
index 7dc3f0b..e904760 100644
--- a/package/postgresql/postgresql.mk
+++ b/package/postgresql/postgresql.mk
@@ -85,9 +85,9 @@ endef
 
 define POSTGRESQL_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/postgresql/postgresql.service \
-		$(TARGET_DIR)/etc/systemd/system/postgresql.service
+		$(TARGET_DIR)/usr/lib/systemd/system/postgresql.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../postgresql.service \
+	ln -fs /usr/lib/systemd/system/postgresql.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/postgresql.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 11/15] postgresql: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/postgresql/postgresql.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/postgresql/postgresql.mk b/package/postgresql/postgresql.mk
> index 7dc3f0b..e904760 100644
> --- a/package/postgresql/postgresql.mk
> +++ b/package/postgresql/postgresql.mk
> @@ -85,9 +85,9 @@ endef
>
>  define POSTGRESQL_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/postgresql/postgresql.service \
> -               $(TARGET_DIR)/etc/systemd/system/postgresql.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/postgresql.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../postgresql.service \
> +       ln -fs /usr/lib/systemd/system/postgresql.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/postgresql.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 11/15] postgresql: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 8:59 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,


-- 
Samuel

[Buildroot] [PATCH 12/15] libiio: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/libiio/libiio.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/libiio/libiio.mk b/package/libiio/libiio.mk
index abcddba..a821892 100644
--- a/package/libiio/libiio.mk
+++ b/package/libiio/libiio.mk
@@ -41,8 +41,8 @@ endef
 define LIBIIO_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
 	$(INSTALL) -D -m 0644 $(@D)/debian/iiod.service \
-		$(TARGET_DIR)/lib/systemd/system/iiod.service
-	ln -fs /lib/systemd/system/iiod.service \
+		$(TARGET_DIR)/usr/lib/systemd/system/iiod.service
+	ln -fs /usr/lib/systemd/system/iiod.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/iiod.service
 endef
 endif
-- 
2.1.0

[Buildroot] [PATCH 12/15] libiio: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/libiio/libiio.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/libiio/libiio.mk b/package/libiio/libiio.mk
> index abcddba..a821892 100644
> --- a/package/libiio/libiio.mk
> +++ b/package/libiio/libiio.mk
> @@ -41,8 +41,8 @@ endef
>  define LIBIIO_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -d $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>         $(INSTALL) -D -m 0644 $(@D)/debian/iiod.service \
> -               $(TARGET_DIR)/lib/systemd/system/iiod.service
> -       ln -fs /lib/systemd/system/iiod.service \
> +               $(TARGET_DIR)/usr/lib/systemd/system/iiod.service
> +       ln -fs /usr/lib/systemd/system/iiod.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/iiod.service
>  endef
>  endif
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 12/15] libiio: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 9:00 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,


-- 
Samuel

[Buildroot] [PATCH 13/15] dhcp: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/dhcp/dhcp.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/dhcp/dhcp.mk b/package/dhcp/dhcp.mk
index f01a5fa..85acfbf 100644
--- a/package/dhcp/dhcp.mk
+++ b/package/dhcp/dhcp.mk
@@ -78,11 +78,11 @@ endef
 ifeq ($(BR2_PACKAGE_DHCP_SERVER),y)
 define DHCP_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/dhcp/dhcpd.service \
-		$(TARGET_DIR)/lib/systemd/system/dhcpd.service
+		$(TARGET_DIR)/usr/lib/systemd/system/dhcpd.service
 
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
 
-	ln -sf ../../../../lib/systemd/system/dhcpd.service \
+	ln -sf /usr/lib/systemd/system/dhcpd.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/dhcpd.service
 
 	echo "d /var/lib/dhcp 0755 - - - -" > \
-- 
2.1.0

[Buildroot] [PATCH 13/15] dhcp: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/dhcp/dhcp.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/dhcp/dhcp.mk b/package/dhcp/dhcp.mk
> index f01a5fa..85acfbf 100644
> --- a/package/dhcp/dhcp.mk
> +++ b/package/dhcp/dhcp.mk
> @@ -78,11 +78,11 @@ endef
>  ifeq ($(BR2_PACKAGE_DHCP_SERVER),y)
>  define DHCP_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/dhcp/dhcpd.service \
> -               $(TARGET_DIR)/lib/systemd/system/dhcpd.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/dhcpd.service
>
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>
> -       ln -sf ../../../../lib/systemd/system/dhcpd.service \
> +       ln -sf /usr/lib/systemd/system/dhcpd.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/dhcpd.service
>
>         echo "d /var/lib/dhcp 0755 - - - -" > \
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 13/15] dhcp: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 9:00 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,


-- 
Samuel

[Buildroot] [PATCH 14/15] kodi: move systemd service file to /usr/lib

[Mike Williams]

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/kodi/kodi.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
index f40fc31..7ad4988 100644
--- a/package/kodi/kodi.mk
+++ b/package/kodi/kodi.mk
@@ -256,11 +256,11 @@ endef
 
 define KODI_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/kodi/kodi.service \
-		$(TARGET_DIR)/etc/systemd/system/kodi.service
+		$(TARGET_DIR)/usr/lib/systemd/system/kodi.service
 
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
 
-	ln -fs ../kodi.service \
+	ln -fs /usr/lib/systemd/system/kodi.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/kodi.service
 endef
 
-- 
2.1.0

[Buildroot] [PATCH 14/15] kodi: move systemd service file to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/kodi/kodi.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
> index f40fc31..7ad4988 100644
> --- a/package/kodi/kodi.mk
> +++ b/package/kodi/kodi.mk
> @@ -256,11 +256,11 @@ endef
>
>  define KODI_INSTALL_INIT_SYSTEMD
>         $(INSTALL) -D -m 644 package/kodi/kodi.service \
> -               $(TARGET_DIR)/etc/systemd/system/kodi.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/kodi.service
>
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
>
> -       ln -fs ../kodi.service \
> +       ln -fs /usr/lib/systemd/system/kodi.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/kodi.service
>  endef
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 14/15] kodi: move systemd service file to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 9:00 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,



-- 
Samuel

[Buildroot] [PATCH 15/15] systemd: change install path to /usr/lib

[Mike Williams]

/usr/lib is the default path for upstream systemd.

Signed-off-by: Mike Williams <mike@mikebwilliams.com>
---
 package/systemd/systemd.mk | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
index 98bda02..3ec124c 100644
--- a/package/systemd/systemd.mk
+++ b/package/systemd/systemd.mk
@@ -27,8 +27,6 @@ SYSTEMD_DEPENDENCIES += busybox
 endif
 
 SYSTEMD_CONF_OPTS += \
-	--with-rootprefix= \
-	--with-rootlibdir=/lib \
 	--enable-static=no \
 	--disable-manpages \
 	--disable-selinux \
@@ -100,16 +98,16 @@ endif
 ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
 SYSTEMD_CONF_OPTS += --enable-networkd
 define SYSTEMD_INSTALL_RESOLVCONF_HOOK
-	ln -sf ../run/systemd/resolve/resolv.conf \
+	ln -sf /run/systemd/resolve/resolv.conf \
 		$(TARGET_DIR)/etc/resolv.conf
 endef
 else
 SYSTEMD_CONF_OPTS += --disable-networkd
 define SYSTEMD_INSTALL_SERVICE_NETWORK
 	$(INSTALL) -D -m 644 package/systemd/network.service \
-		$(TARGET_DIR)/etc/systemd/system/network.service
+		$(TARGET_DIR)/usr/lib/systemd/system/network.service
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
-	ln -fs ../network.service \
+	ln -fs /usr/lib/systemd/system/network.service \
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/network.service
 endef
 endif
@@ -118,7 +116,7 @@ ifeq ($(BR2_PACKAGE_SYSTEMD_TIMESYNCD),y)
 SYSTEMD_CONF_OPTS += --enable-timesyncd
 define SYSTEMD_INSTALL_SERVICE_TIMESYNC
 	mkdir -p $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants
-	ln -sf ../../../../lib/systemd/system/systemd-timesyncd.service \
+	ln -sf /usr/lib/systemd/system/systemd-timesyncd.service \
 		$(TARGET_DIR)/etc/systemd/system/sysinit.target.wants/systemd-timesyncd.service
 endef
 else
@@ -144,12 +142,12 @@ SYSTEMD_MAKE_OPTS += LIBS=-lrt
 SYSTEMD_MAKE_OPTS += LDFLAGS+=-ldl
 
 define SYSTEMD_INSTALL_INIT_HOOK
-	ln -fs ../lib/systemd/systemd $(TARGET_DIR)/sbin/init
-	ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/halt
-	ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/poweroff
-	ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/reboot
+	ln -fs /usr/lib/systemd/systemd $(TARGET_DIR)/sbin/init
+	ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/halt
+	ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/poweroff
+	ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/reboot
 
-	ln -fs ../../../lib/systemd/system/multi-user.target $(TARGET_DIR)/etc/systemd/system/default.target
+	ln -fs /usr/lib/systemd/system/multi-user.target $(TARGET_DIR)/etc/systemd/system/default.target
 endef
 
 define SYSTEMD_INSTALL_MACHINEID_HOOK
@@ -157,14 +155,14 @@ define SYSTEMD_INSTALL_MACHINEID_HOOK
 endef
 
 define SYSTEMD_SANITIZE_PATH_IN_UNITS
-	find $(TARGET_DIR)/lib/systemd/system -name '*.service' \
+	find $(TARGET_DIR)/usr/lib/systemd/system -name '*.service' \
 		-exec $(SED) 's,$(HOST_DIR),,g' {} \;
 endef
 
 # Disable ldconfig.service, as /sbin/ldconfig is not available when the
 # target is built with a glibc-based toolchain.
 define SYSTEMD_DISABLE_LDCONFIG_SERVICE_HOOK
-	rm -f $(TARGET_DIR)/lib/systemd/system/sysinit.target.wants/ldconfig.service
+	rm -f $(TARGET_DIR)/usr/lib/systemd/system/sysinit.target.wants/ldconfig.service
 endef
 
 SYSTEMD_POST_INSTALL_TARGET_HOOKS += \
@@ -199,7 +197,7 @@ define SYSTEMD_INSTALL_SERVICE_TTY
 	else \
 		SERVICE="serial-getty"; \
 	fi; \
-	ln -fs ../../../../lib/systemd/system/$${SERVICE}@.service \
+	ln -fs /usr/lib/systemd/system/$${SERVICE}@.service \
 		$(TARGET_DIR)/etc/systemd/system/getty.target.wants/$${SERVICE}@$(BR2_TARGET_GENERIC_GETTY_PORT).service
 endef
 endif
-- 
2.1.0

[Buildroot] [PATCH 15/15] systemd: change install path to /usr/lib

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> /usr/lib is the default path for upstream systemd.
>
> Signed-off-by: Mike Williams <mike@mikebwilliams.com>

Reviewed-by: Steven Noonan <steven@uplinklabs.net>

> ---
>  package/systemd/systemd.mk | 26 ++++++++++++--------------
>  1 file changed, 12 insertions(+), 14 deletions(-)
>
> diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
> index 98bda02..3ec124c 100644
> --- a/package/systemd/systemd.mk
> +++ b/package/systemd/systemd.mk
> @@ -27,8 +27,6 @@ SYSTEMD_DEPENDENCIES += busybox
>  endif
>
>  SYSTEMD_CONF_OPTS += \
> -       --with-rootprefix= \
> -       --with-rootlibdir=/lib \
>         --enable-static=no \
>         --disable-manpages \
>         --disable-selinux \
> @@ -100,16 +98,16 @@ endif
>  ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
>  SYSTEMD_CONF_OPTS += --enable-networkd
>  define SYSTEMD_INSTALL_RESOLVCONF_HOOK
> -       ln -sf ../run/systemd/resolve/resolv.conf \
> +       ln -sf /run/systemd/resolve/resolv.conf \
>                 $(TARGET_DIR)/etc/resolv.conf
>  endef
>  else
>  SYSTEMD_CONF_OPTS += --disable-networkd
>  define SYSTEMD_INSTALL_SERVICE_NETWORK
>         $(INSTALL) -D -m 644 package/systemd/network.service \
> -               $(TARGET_DIR)/etc/systemd/system/network.service
> +               $(TARGET_DIR)/usr/lib/systemd/system/network.service
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> -       ln -fs ../network.service \
> +       ln -fs /usr/lib/systemd/system/network.service \
>                 $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/network.service
>  endef
>  endif
> @@ -118,7 +116,7 @@ ifeq ($(BR2_PACKAGE_SYSTEMD_TIMESYNCD),y)
>  SYSTEMD_CONF_OPTS += --enable-timesyncd
>  define SYSTEMD_INSTALL_SERVICE_TIMESYNC
>         mkdir -p $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants
> -       ln -sf ../../../../lib/systemd/system/systemd-timesyncd.service \
> +       ln -sf /usr/lib/systemd/system/systemd-timesyncd.service \
>                 $(TARGET_DIR)/etc/systemd/system/sysinit.target.wants/systemd-timesyncd.service
>  endef
>  else
> @@ -144,12 +142,12 @@ SYSTEMD_MAKE_OPTS += LIBS=-lrt
>  SYSTEMD_MAKE_OPTS += LDFLAGS+=-ldl
>
>  define SYSTEMD_INSTALL_INIT_HOOK
> -       ln -fs ../lib/systemd/systemd $(TARGET_DIR)/sbin/init
> -       ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/halt
> -       ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/poweroff
> -       ln -fs ../bin/systemctl $(TARGET_DIR)/sbin/reboot
> +       ln -fs /usr/lib/systemd/systemd $(TARGET_DIR)/sbin/init
> +       ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/halt
> +       ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/poweroff
> +       ln -fs /usr/bin/systemctl $(TARGET_DIR)/sbin/reboot
>
> -       ln -fs ../../../lib/systemd/system/multi-user.target $(TARGET_DIR)/etc/systemd/system/default.target
> +       ln -fs /usr/lib/systemd/system/multi-user.target $(TARGET_DIR)/etc/systemd/system/default.target
>  endef
>
>  define SYSTEMD_INSTALL_MACHINEID_HOOK
> @@ -157,14 +155,14 @@ define SYSTEMD_INSTALL_MACHINEID_HOOK
>  endef
>
>  define SYSTEMD_SANITIZE_PATH_IN_UNITS
> -       find $(TARGET_DIR)/lib/systemd/system -name '*.service' \
> +       find $(TARGET_DIR)/usr/lib/systemd/system -name '*.service' \
>                 -exec $(SED) 's,$(HOST_DIR),,g' {} \;
>  endef
>
>  # Disable ldconfig.service, as /sbin/ldconfig is not available when the
>  # target is built with a glibc-based toolchain.
>  define SYSTEMD_DISABLE_LDCONFIG_SERVICE_HOOK
> -       rm -f $(TARGET_DIR)/lib/systemd/system/sysinit.target.wants/ldconfig.service
> +       rm -f $(TARGET_DIR)/usr/lib/systemd/system/sysinit.target.wants/ldconfig.service
>  endef
>
>  SYSTEMD_POST_INSTALL_TARGET_HOOKS += \
> @@ -199,7 +197,7 @@ define SYSTEMD_INSTALL_SERVICE_TTY
>         else \
>                 SERVICE="serial-getty"; \
>         fi; \
> -       ln -fs ../../../../lib/systemd/system/$${SERVICE}@.service \
> +       ln -fs /usr/lib/systemd/system/$${SERVICE}@.service \
>                 $(TARGET_DIR)/etc/systemd/system/getty.target.wants/$${SERVICE}@$(BR2_TARGET_GENERIC_GETTY_PORT).service
>  endef
>  endif
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[Buildroot] [PATCH 15/15] systemd: change install path to /usr/lib

[Samuel Martin]

On Thu, Mar 19, 2015 at 9:02 PM, Steven Noonan <steven@uplinklabs.net> wrote:
> On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
>> /usr/lib is the default path for upstream systemd.
>>
>> Signed-off-by: Mike Williams <mike@mikebwilliams.com>
>
> Reviewed-by: Steven Noonan <steven@uplinklabs.net>
Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,


-- 
Samuel

[Buildroot] RFC: systemd and service files cleanup

[Steven Noonan]

On Thu, Mar 19, 2015 at 10:56 AM, Mike Williams <mike@mikebwilliams.com> wrote:
> All,
>
> Currently, package service files for systemd are installed by buildroot in a variety of locations: /etc, /lib, and /usr/lib. For systemd, split /lib is deprecated and only checked if a split /lib and /usr/lib is enabled. /etc is meant for local customizations to the default service files. It is my opinion that buildroot should install all upstream package and buildroot-provided service files in /usr/lib, and this series cleans up our packages to do exactly that. Enabling them by default by linking them to /etc/systemd/system/multi-user.target.wants has been preserved.
>
> Upstream systemd's standard installation directory for its binaries is /usr/lib. It appears that historically, buildroot's systemd installation location has been switched between /lib and /usr/lib multiple times. I couldn't find any particular reason for this, so I've moved it back to /usr/lib along with its service files.
>
> Anyway, please send comments :).
>

Nice. Basically any systemd units installed by Buildroot should be in
/usr/lib/systemd/system and nothing from Buildroot should be installed
to /etc/systemd/system, so I agree with this series as a whole in
terms of direction.

Also, I have a related patch set, but it has only had very light
testing on my end:

http://git.uplinklabs.net/snoonan/projects/buildroot.git/log/?h=unified-bin

I come from the Arch Linux world, so I made a new skeleton that
matches the Arch base filesystem layout: with /bin, /sbin, /usr/bin,
and /usr/sbin all unified (likewise with /lib and /usr/lib). I haven't
convinced myself to post the series because I can't think of a
particularly good justification for it in the Buildroot context.

[Buildroot] RFC: systemd and service files cleanup

[Mike Williams]

> I come from the Arch Linux world, so I made a new skeleton that
> matches the Arch base filesystem layout: with /bin, /sbin, /usr/bin,
> and /usr/sbin all unified (likewise with /lib and /usr/lib). I haven't
> convinced myself to post the series because I can't think of a
> particularly good justification for it in the Buildroot context.

I'm working that way myself, I don't know what use cases in Buildroot
still require the split.

[Buildroot] RFC: systemd and service files cleanup

[Steven Noonan]

On Thu, Mar 19, 2015 at 12:43 PM, Mike Williams <mike@mikebwilliams.com> wrote:
>> I come from the Arch Linux world, so I made a new skeleton that
>> matches the Arch base filesystem layout: with /bin, /sbin, /usr/bin,
>> and /usr/sbin all unified (likewise with /lib and /usr/lib). I haven't
>> convinced myself to post the series because I can't think of a
>> particularly good justification for it in the Buildroot context.
>
> I'm working that way myself, I don't know what use cases in Buildroot
> still require the split.

We could consider offering it as an experimental alternative to the
default skeleton (just add another "choice" in kconfig, as I did in my
patch series). And open a discussion about unifying the directory
structure. It's certainly possible to support both use cases, though
it does require more testing.

[Buildroot] RFC: systemd and service files cleanup

[Arnout Vandecappelle]

On 19/03/15 20:50, Steven Noonan wrote:
> On Thu, Mar 19, 2015 at 12:43 PM, Mike Williams <mike@mikebwilliams.com> wrote:
>>> I come from the Arch Linux world, so I made a new skeleton that
>>> matches the Arch base filesystem layout: with /bin, /sbin, /usr/bin,
>>> and /usr/sbin all unified (likewise with /lib and /usr/lib). I haven't
>>> convinced myself to post the series because I can't think of a
>>> particularly good justification for it in the Buildroot context.
>>
>> I'm working that way myself, I don't know what use cases in Buildroot
>> still require the split.
> 
> We could consider offering it as an experimental alternative to the
> default skeleton (just add another "choice" in kconfig, as I did in my
> patch series). And open a discussion about unifying the directory
> structure. It's certainly possible to support both use cases, though
> it does require more testing.

 I've also started using a unified /bin /sbin /usr/bin /usr/sbin and unified
/lib /usr/lib /lib64 /usr/lib64, and it really simplifies things a lot. I yet
have to find a disadvantage.

 Make it it a choice sounds like a great idea! Please Cc me on the patch :-)


 Regards,
 Arnout


-- 
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:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] RFC: systemd and service files cleanup

[Mike Williams]

All,

On Thu, Mar 19, 2015 at 5:44 PM, Arnout Vandecappelle <arnout@mind.be> wrote:
> On 19/03/15 20:50, Steven Noonan wrote:
>> On Thu, Mar 19, 2015 at 12:43 PM, Mike Williams <mike@mikebwilliams.com> wrote:
>>>> I come from the Arch Linux world, so I made a new skeleton that
>>>> matches the Arch base filesystem layout: with /bin, /sbin, /usr/bin,
>>>> and /usr/sbin all unified (likewise with /lib and /usr/lib). I haven't
>>>> convinced myself to post the series because I can't think of a
>>>> particularly good justification for it in the Buildroot context.
>>>
>>> I'm working that way myself, I don't know what use cases in Buildroot
>>> still require the split.
>>
>> We could consider offering it as an experimental alternative to the
>> default skeleton (just add another "choice" in kconfig, as I did in my
>> patch series). And open a discussion about unifying the directory
>> structure. It's certainly possible to support both use cases, though
>> it does require more testing.
>
>  I've also started using a unified /bin /sbin /usr/bin /usr/sbin and unified
> /lib /usr/lib /lib64 /usr/lib64, and it really simplifies things a lot. I yet
> have to find a disadvantage.

Working through this systemd cleanup, I'm now finding that some
packages install udev rules to /lib/udev/rules.d by default. The udev
documentation states that:

"The udev rules are read from the files located in the system rules
directory /usr/lib/udev/rules.d, the volatile runtime directory
/run/udev/rules.d and the local administration directory
/etc/udev/rules.d." I can verify that systemd's udev does not check
/lib/udev/rules.d, even with --enable-split-usr (our current setup).
I'm not sure what eudev does.

https://www.kernel.org/pub/linux/utils/kernel/hotplug/udev/udev.html

It makes me wonder how many upstream developers are already using a
merged /usr system, and are not noticing the technically wrong
defaults for split systems. Likewise, apparently many of buildroot's
systemd users have merged /usr already, they won't notice this
breakage in their testing either.

Mike

[Buildroot] RFC: systemd and service files cleanup

[Samuel Martin]

Hi Mike, all,

On Thu, Mar 19, 2015 at 6:56 PM, Mike Williams <mike@mikebwilliams.com> wrote:
> All,
>
> Currently, package service files for systemd are installed by buildroot in a variety of locations: /etc, /lib, and /usr/lib. For systemd, split /lib is deprecated and only checked if a split /lib and /usr/lib is enabled. /etc is meant for local customizations to the default service files. It is my opinion that buildroot should install all upstream package and buildroot-provided service files in /usr/lib, and this series cleans up our packages to do exactly that. Enabling them by default by linking them to /etc/systemd/system/multi-user.target.wants has been preserved.
>
> Upstream systemd's standard installation directory for its binaries is /usr/lib. It appears that historically, buildroot's systemd installation location has been switched between /lib and /usr/lib multiple times. I couldn't find any particular reason for this, so I've moved it back to /usr/lib along with its service files.
>
> Anyway, please send comments :).

Thanks for helping improving this point. :)

Just one global comment:
In the series, you change all relative symlink with absolute one.
We use to use relative symlink to avoid confusing people looking at
the target tree because they are still right.

I don't have strong opinion about this, so let's see what others think about it.

Regards,

-- 
Samuel

[Buildroot] RFC: systemd and service files cleanup

[Thomas Petazzoni]

Dear Mike Williams,

Adding Steven in Cc.

On Thu, 19 Mar 2015 13:56:32 -0400, Mike Williams wrote:

> Currently, package service files for systemd are installed by
> buildroot in a variety of locations: /etc, /lib, and /usr/lib. For
> systemd, split /lib is deprecated and only checked if a split /lib
> and /usr/lib is enabled. /etc is meant for local customizations to
> the default service files. It is my opinion that buildroot should
> install all upstream package and buildroot-provided service files
> in /usr/lib, and this series cleans up our packages to do exactly
> that. Enabling them by default by linking them
> to /etc/systemd/system/multi-user.target.wants has been preserved.
> 
> Upstream systemd's standard installation directory for its binaries
> is /usr/lib. It appears that historically, buildroot's systemd
> installation location has been switched between /lib and /usr/lib
> multiple times. I couldn't find any particular reason for this, so
> I've moved it back to /usr/lib along with its service files.

I am generally fine with the proposed changes, especially since other
systemd users looked at them.

However, there's one thing I'm not sure about: you're changing all the
relative symbolic links to absolute symbolic links:

-	ln -fs ../ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
+	ln -fs /usr/lib/systemd/system/ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service

While it causes no problem on the target, it means that when looking at
output/target/ on your build machine, now all those symbolic links are
broken. I find it quite nice when relative symlinks are used, since it
allows to have things in output/target looking somewhat sensible.

Is there any strong reason for switching to absolute symbolic links?

Also, there are *lots* of systemd related patches sitting in patchwork.
Could you and Steven have a look at those pending patches, and let me
know which ones can be applied, which ones cannot?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] RFC: systemd and service files cleanup

[Mike Williams]

Thomas,

On Fri, Mar 20, 2015 at 9:20 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Mike Williams,
>
> Adding Steven in Cc.
>
> On Thu, 19 Mar 2015 13:56:32 -0400, Mike Williams wrote:
>
>> Currently, package service files for systemd are installed by
>> buildroot in a variety of locations: /etc, /lib, and /usr/lib. For
>> systemd, split /lib is deprecated and only checked if a split /lib
>> and /usr/lib is enabled. /etc is meant for local customizations to
>> the default service files. It is my opinion that buildroot should
>> install all upstream package and buildroot-provided service files
>> in /usr/lib, and this series cleans up our packages to do exactly
>> that. Enabling them by default by linking them
>> to /etc/systemd/system/multi-user.target.wants has been preserved.
>>
>> Upstream systemd's standard installation directory for its binaries
>> is /usr/lib. It appears that historically, buildroot's systemd
>> installation location has been switched between /lib and /usr/lib
>> multiple times. I couldn't find any particular reason for this, so
>> I've moved it back to /usr/lib along with its service files.
>
> I am generally fine with the proposed changes, especially since other
> systemd users looked at them.
>
> However, there's one thing I'm not sure about: you're changing all the
> relative symbolic links to absolute symbolic links:
>
> -       ln -fs ../ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
> +       ln -fs /usr/lib/systemd/system/ntpd.service $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/ntpd.service
>
> While it causes no problem on the target, it means that when looking at
> output/target/ on your build machine, now all those symbolic links are
> broken. I find it quite nice when relative symlinks are used, since it
> allows to have things in output/target looking somewhat sensible.
>
> Is there any strong reason for switching to absolute symbolic links?

No, they were just relative to the files that used to be in /etc and I
used absolute because it was easier for me to not screw them up when
changing them :). How about you or one of the other buildroot core
team tell me how you want it done, and I'll re-roll the patch series.

>
> Also, there are *lots* of systemd related patches sitting in patchwork.
> Could you and Steven have a look at those pending patches, and let me
> know which ones can be applied, which ones cannot?

Will do. I think after the next series of this patch, and a couple new
spins of other patches, we should be able to send you a large list of
patches to commit.

>
> Thanks,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com

[Buildroot] RFC: systemd and service files cleanup

[Thomas Petazzoni]

Dear Mike Williams,

On Fri, 20 Mar 2015 09:27:51 -0400, Mike Williams wrote:

> > Is there any strong reason for switching to absolute symbolic links?
> 
> No, they were just relative to the files that used to be in /etc and I
> used absolute because it was easier for me to not screw them up when
> changing them :). How about you or one of the other buildroot core
> team tell me how you want it done, and I'll re-roll the patch series.

Ok, then I believe staying with relative symlinks is probably the
safest option, since that's generally what we're doing in the rest of
Buildroot.

> Will do. I think after the next series of this patch, and a couple new
> spins of other patches, we should be able to send you a large list of
> patches to commit.

Ok, good. If some of the patches need a bit of rework, don't hesitate
to pick them up and resend an updated version as needed.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com