Re: [PATCH] git-daemon: fix for rotating logs

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <git@...>
Date: Monday, April 28, 2008 - 3:28 pm

Miklos Vajna wrote:

No.


It could, but that's the reason one uses the openlog(3) interface to
syslog, to centralise logfilemanagement and *not* having to deal with
the intricacies of logfile rotation and the like.  So it doesn't need
to.  When you need logfile rotation, you're better off with using
openlog(3), and doing that the proper way means *NOT* closing and
reopening that everytime.


They don't.  Your patch fixes the wrong problem.  Please don't fix
something that wasn't broken in the first place.



N.B. I've never had to close and reopen the openlog(3) syslog interface
in any of the daemons I've written.
The example you refer to of icald is where it directly writes to a
logfile, which is *not* what this patch was about, so please do not use
it as justification.
-- 
Sincerely,                                                          srb@cuci.nl
           Stephen R. van den Berg.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH] git-daemon: fix for rotating logs, Junio C Hamano, (Mon Apr 28, 12:29 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 2:08 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Miklos Vajna, (Mon Apr 28, 2:21 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Mike Hommey, (Mon Apr 28, 2:29 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Andreas Ericsson, (Tue Apr 29, 2:17 am)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Tue Apr 29, 6:54 am)
Re: [PATCH] git-daemon: fix for rotating logs, Andreas Ericsson, (Tue Apr 29, 7:00 am)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Tue Apr 29, 11:16 am)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Tue Apr 29, 1:58 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 2:37 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Miklos Vajna, (Mon Apr 28, 3:00 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Mon Apr 28, 3:28 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 3:53 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Mon Apr 28, 4:43 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 4:53 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Mon Apr 28, 5:00 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 5:09 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Mon Apr 28, 5:13 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 5:23 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Stephen R. van den Berg, (Mon Apr 28, 7:16 pm)
Re: [PATCH] git-daemon: fix for rotating logs, Johannes Schindelin, (Mon Apr 28, 3:24 pm)