[PATCH] innfeed does not reopen (rotated) log file

Julien ÉLIE julien at trigofacile.com
Mon Jan 7 19:16:34 UTC 2013


Hi Russ,

>> In innfeed/main.c, there is an openLogFile() function that does a
>> freopen() of the innfeed log file.
>
>> Couldn't we just add a call to openLogFile() in the existing sighup()
>> function in main.c?  This sighup() function currently reloads
>> innfeed.conf and reconfigure hosts.
>
> Ah, yes, that sounds right to me.  That would indeed fix the problem, and
> I don't see any drawbacks to adding it to the existing sighup handler.
> That's quite a bit simpler than I was expecting!

The only issue I could think of is that the current sighup() function 
reloads innfeed.conf.  This file would therefore be reloaded everyday, 
which might cause a problem if it has changed and the news administrator 
still has not wanted to reload it.  An automatic reload of innfeed.conf 
would then have innfeed shut down (if for instance the syntax of the new 
innfeed.conf file is wrong).

-- 
Julien ÉLIE

« Ex nihilo nihil. » (Perse)


More information about the inn-workers mailing list