[patch] make the logging header customisable
Andrew Bartlett
abartlet at samba.org
Wed May 22 21:53:04 MDT 2013
On Wed, 2013-05-22 at 09:59 -0700, Matthieu Patou wrote:
> On 05/19/2013 10:06 AM, Matthieu Patou wrote:
> > Hello,
> >
> > Find attached an updated version of this, feedback and review welcome.
> >
> >
> > On 06/12/2012 01:27 AM, Matthieu Patou wrote:
> >> Hello Volker & all,
> >>
> >> A couple of months ago we briefly discussed on the possibility to
> >> have a header printed even if we log on the stdout because with more
> >> than 1 process logging it can be a bit tedious sometime to understand
> >> who print what.
> >> My understanding was that you were more or less ok with this idea but
> >> that instead of adding a simple boolean to say "add headers also when
> >> logging to stdout" you proposed me (at least that was my
> >> understanding) to be able to specify the header that we want to have.
> >> I've been working on this lately and come with the following patch
> >> for the 3.5.x branch, it seems to work ok as if the "debug header
> >> template" parameter is not specified the header is as it was (which
> >> means no header in case of logging to stdout) but if you specify it
> >> then it will be as specified.
> >>
> >> For the moment the patch is for 3.5.14 as it's what we use at my work
> >> and it helps us during the QA and dev debug, it didn't apply yet to
> >> master but I'll do the job of front port if the idea of the patch is
> >> ok (and if it's also at least mostly ok).
> >>
> >> Matthieu.
> Any body on this patch ? should I push it directly to autobuild ?
G'Day Matthieu,
Sorry for not getting back to you on this one. The main thought I had
about this, at the time and repeated now is: We hit DEBUG a lot, do we
need the extra complexity?
What is the connection between logging with a header to stdout (which
was roundly welcomed, except that it caused some odd failure and Jeremy
never got the patch in) and this more complex proposal?
Thanks,
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
More information about the samba-technical
mailing list