chiark / gitweb /
conf-parser: let's explicitly deprecate .include in unit files
authorLennart Poettering <lennart@poettering.net>
Fri, 23 Feb 2018 19:46:38 +0000 (20:46 +0100)
committerSven Eden <yamakuzure@gmx.net>
Fri, 24 Aug 2018 14:47:08 +0000 (16:47 +0200)
commit3b387a06cf8205c9938a1000ca8a14b98116502c
tree0c39d4b881569f1e0ed397285472ce32bb8a8c07
parent7cba4007b6b434d222b3f6aa8fe66f0a07db1d2e
conf-parser: let's explicitly deprecate .include in unit files

.include lines are already deprecated somewhat, and for example
explicitly not mentioned in the documentation for this reason. Let's get
one step further and generatea warning when we encounter them (but still
process them).

Why are they deprecated? Because they are semantically awful — they
complicate stat() based mtime checks for configuration files and they
allow arbitrary loops we currently have zero protection against and
really shouldn't have to have.
src/shared/conf-parser.c