[OE-core] [PATCH 1/1] systemd: make /etc/sysctl.conf have real effect

[Koen Kooi]

> Op 6 nov. 2014, om 08:59 heeft ChenQi <Qi.Chen at windriver.com> het volgende geschreven:
> 
> On 11/06/2014 03:48 PM, Koen Kooi wrote:
>> Op 6 nov. 2014, om 08:32 heeft Chen Qi <Qi.Chen at windriver.com> het volgende geschreven:
>> 
>>> In systemd, /etc/sysctl.conf is actually ignored by systemd-sysctl,
>>> because this command only examine *.conf files under a bunch of directories
>>> like /etc/sysctl.d or /usr/lib/sysctl.d.
>>> 
>>> The problem is we are used to configuring kernel parameters in /etc/sysctl.conf,
>>> so it would be really strange if the configuration in that file doesn't have any
>>> effect.
>>> 
>>> This patch reference Fedora's solution to this problem, creating a symlink to
>>> /etc/sysctl.conf under /etc/sysctl.d/.
>> Shouldn't this be done in procps instead?
>> 
> 
> Actually, the problem is not about `sysctl' command.
> procps provides `sysctl', but busybox also provides this command.
> It's very possible that on our generated image, procps is not installed but `sysctl' command is available.
> Both busybox's and procps's `sysctl' command takes /etc/sysctl.conf into consideration.

Right, but only procps installs that file.

> Now, systemd provides a similar utility called `systemd-sysctl' which is executed at boot time via systemd-sysctl.service.
> 
> So our actually problem is that systemd-sysctl ignores /etc/sysctl.conf, which makes it somewhat strange, especially to users who are used to configuring parameters in sysctl.conf.
> And this patch solves this problem by adding a symlink under /etc/sysctl.d/.
> 
> That's why I think we should put this in systemd.

You're adding a symlink to a file which only exists if you install procps, which isn't in RDEPENDS.