[systemd-devel] [PATCH] man: fix description of sysctl.d order

[Kay Sievers]

On Thu, Sep 12, 2013 at 9:49 AM, Colin Guthrie <gmane at colin.guthr.ie> wrote:

>> Avoiding conflicting entries seems the best approach here, and the
>> first-one-wins, the later ones complain sounds like the best approach.
>
> Hmmm, fair enough, but I don't see why these should be different to
> sysctl.d here. Personally, (as you mentioned above) the
> later-override-earlier to  me feels more natural, but I'll admit that I
> don't fully grok how tmpfiles and binfmt should follow a first-one-wins
> approach whereas sysctl.d should follow a later-override-earlier...

See the later mail, binfmt is doing the opposite of what the man page
said, tmpfiles requires uniqueness, everything else is an error.

>>> Is there a reason why sysctl.conf cannot just be symlinked as
>>> sysctl.d/0-sysctl.conf rather than 99-sysctl.conf and keep the ordering
>>> consistent with the other tools?
>>
>> Where is that defined? /etc/sysctl.conf should not exist, in the ideal case.
>
> I was just referring to your NEWS item message in
> 04bf3c1a60d82791e0320381e9268f727708f776 which Mantas referenced where
> you said:
>
>         * The systemd-sysctl tool does no longer natively read the
>           file /etc/sysctl.conf. If desired, the file should be
>           symlinked from /etc/sysctl.d/99-sysctl.conf. Apart from
>           providing legacy support by a symlink rather than built-in
>           code, it also makes the otherwise hidden order of application
>           of the different files visible.
>
> If it was symlinked from as 0-sysctl.conf and the ordering had remained
> unchanged it would have achieved the same result no?

Yes, you can now control the policy with the file name instead of
built-in logic. Fedora will use 99- to let the "admin's local machine
config" which is usually empty always win.

> (side note: the version number in the changes is now duplicated - there
> are two 206's. I guess the top one should be 207)

That's intentional, distcheck will not work with NEWS and configure.ac
in sync. We can change that only directly before release.

Kay