On Tue, 20.05.14 15:43, [email protected] ([email protected]) 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 _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
