From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Beno=c3=aet_Th=c3=a9baudeau?= Date: Wed, 14 Oct 2015 11:17:32 +0200 Subject: [Buildroot] [PATCH v2 1/8] package/dhcp: fix SysV init scripts option passing In-Reply-To: References: <1444728929-46246-1-git-send-email-benoit@wsystem.com> Message-ID: <561E1DAC.6000605@wsystem.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hi Maxime, On 14/10/2015 09:21, Maxime Hadjinlian wrote: > On Tue, Oct 13, 2015 at 11:35 AM, Beno?t Th?baudeau > wrote: > > The SysV init scripts have configuration variables like INTERFACES whose > contents have to be passed to the daemon. These variables are > initialized as empty strings, but some of them are not allowed to be > empty and there was no means of filling them apart from creating a root > FS overlay to overwrite these scripts. > > This commit adds support for files under /etc/default/ to set these > configuration variables. Such light files can now be added to the root > FS skeleton or overlays without having to duplicate most of the SysV > init scripts. > > Signed-off-by: Beno?t Th?baudeau > > > > Thanks a lot for your work, I am afraid I am going to add to your work and maybe be a bit of a kill joy: > > What about systemd ? > > There's already a service file in the dhcp package, so any changes made to the init scripts should also be made to the .service to keep the whole package coherent. > > If you need any help/tips, do ask. I don't have any experience with systemd. I see the basics from the online documentation. In order to switch an existing SysV + BusyBox defconfig to systemd, is there anything else to tweak than setting the init system to systemd in the configuration? Looking at the files, most of this series seems to already be fine with systemd. The SysV init scripts have not been kept consistent when systemd was added. But there are a few things to do for systemd though. For 1/8, would it be OK to change "EnvironmentFile=/etc/default/dhcpd.conf" to "EnvironmentFile=/etc/default/dhcpd" for systemd in order to share the filename with SysV? If I look into the /etc/default/ folder on my Ubuntu 15.04 (which uses systemd), none of the files present has a .conf extension. The purpose of these files is to set some environment variables, so they're more scripts than configuration files. When a change would affect both SysV and systemd, should there be a single patch common to both, or one patch per init system? Best regards, Beno?t