Hello Kaarle, Thanks for the cleaning work. Some comment inside. On Mon, 2017-02-13 at 11:47 +0200, Kaarle Ritvanen wrote: > Signed-off-by: Kaarle Ritvanen <kaarle.ritvanen@xxxxxxxxxxxxx> > --- > ulogd.conf.in | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/ulogd.conf.in b/ulogd.conf.in > index 2fcf39a..6e7bdf4 100644 > --- a/ulogd.conf.in > +++ b/ulogd.conf.in > @@ -183,15 +183,15 @@ nlgroup=1 > socket_path="/tmp/nuauth_ulogd2.sock" > > [emu1] > -file="/var/log/ulogd_syslogemu.log" > +file="/var/log/ulogd.syslogemu" I really like the idea of getting an harmonized naming for the log files but I think we should do it reverse for values that are not commented in the configuration file. Most distributions and install are shipping with a copy of default configuration. Most users won't have changed the values that are uncommented. So they will have on disk the log file with name defined in the configuration file. And they will benefit from a new logrotate file. Thus we should use the file name defined in the conf as value of the default file name in code. Could you resubmit a patchset ? BR, -- Eric Leblond <eric@xxxxxxxxx> Blog: https://home.regit.org/ -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
