Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] RFC: systemd and service files cleanup
Date: Fri, 20 Mar 2015 14:20:55 +0100	[thread overview]
Message-ID: <20150320142055.5ba1af4c@free-electrons.com> (raw)
In-Reply-To: <1426787807-29510-1-git-send-email-mike@mikebwilliams.com>

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

  parent reply	other threads:[~2015-03-20 13:20 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-19 17:56 [Buildroot] RFC: systemd and service files cleanup Mike Williams
2015-03-19 17:56 ` [Buildroot] [PATCH 01/15] ntp: move systemd service file to /usr/lib Mike Williams
2015-03-19 19:53   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 02/15] openvmtools: move systemd service " Mike Williams
2015-03-19 19:54   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 03/15] psplash: move systemd service files " Mike Williams
2015-03-19 19:55   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 04/15] openntpd: " Mike Williams
2015-03-19 19:56   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 05/15] dbus: " Mike Williams
2015-03-19 19:56   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 06/15] avahi: systemd cleanups Mike Williams
2015-03-19 19:57   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-20 13:15   ` Thomas Petazzoni
2015-03-20 13:25     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 07/15] irqbalance: move systemd service file to /usr/lib Mike Williams
2015-03-19 19:58   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 08/15] openssh: move systemd service files " Mike Williams
2015-03-19 19:58   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 09/15] rsyslog: move systemd service file " Mike Williams
2015-03-19 19:59   ` Steven Noonan
2015-03-20 10:28   ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 10/15] dropbear: " Mike Williams
2015-03-19 19:59   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 11/15] postgresql: " Mike Williams
2015-03-19 19:59   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 12/15] libiio: " Mike Williams
2015-03-19 20:00   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 13/15] dhcp: " Mike Williams
2015-03-19 20:00   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 14/15] kodi: " Mike Williams
2015-03-19 20:00   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 17:56 ` [Buildroot] [PATCH 15/15] systemd: change install path " Mike Williams
2015-03-19 20:02   ` Steven Noonan
2015-03-20 13:07     ` Samuel Martin
2015-03-19 19:31 ` [Buildroot] RFC: systemd and service files cleanup Steven Noonan
2015-03-19 19:43   ` Mike Williams
2015-03-19 19:50     ` Steven Noonan
2015-03-19 21:44       ` Arnout Vandecappelle
2015-03-20 16:21         ` Mike Williams
2015-03-20 13:18 ` Samuel Martin
2015-03-20 13:20 ` Thomas Petazzoni [this message]
2015-03-20 13:27   ` Mike Williams
2015-03-20 13:35     ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150320142055.5ba1af4c@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox