[systemd-devel] logind.conf.d?

[Lennart Poettering]

On Tue, 20.05.14 15:43, josh at joshtriplett.org (josh at joshtriplett.org) wrote:

> > Please try to add this generically to conf-parser.c, but make it
> > optional, so that can disable this for example for unit files, where we
> > actually want to keep track of the drop-ins (and already do that) and
> > thus wouldnt use the generic version anyway... Or in other words, if we
> > add this for logind, we need to add it for journald, timesyncd,
> > ... too...
> 
> As far as I can tell, systemd already has a standard idiom for parsing
> configuration .d directories: call conf_files_list_nulstr (if you have
> multiple directories) or conf_files_list/conf_files_list_strv (if you
> just have one), and loop over the resulting configuration files.  I'd
> intended to make logind do the same thing, in manager_parse_config_file,
> changing the single parse call to a simple loop.  I'd expect the code
> patch to be extremely short, and the documentation patch to be larger
> than the code.
> 
> Are you looking for a generic function similar to config_parse, such as
> config_parse_many, which combines a call to conf_files_list_* and a loop
> calling config_parse on each result?

Yeah, something like that. I just want something generic, that works in
all our daemons the same way, without having to have more than one line
in each of them for this...

Lennart

-- 
Lennart Poettering, Red Hat