Code review comment for lp:~thekorn/zeitgeist/fix-660415-improve-zeitgeist-daemon

Revision history for this message
Seif Lotfy (seif) wrote :

On Wed, Oct 20, 2010 at 12:27 PM, Markus Korn <email address hidden> wrote:

> Ok, I've removed the logging statements you both mentioned.
>
> > review approve
> >
> > I second Siegfrieds comment though. And please make sure the man page is
> > clear on the subject of debug logging.
>
> man zeitgeist-daemon says:
>
> --log-level=LOG_LEVEL
> Specifies how much information should be printed to the
> standard output. Possible values are: DEBUG,
> INFO, WARNING, ERROR, CRITICAL.
>
> is it what you want, or is sth. missing?
> --
>
> https://code.launchpad.net/~thekorn/zeitgeist/fix-660415-improve-zeitgeist-daemon/+merge/38914
> You are subscribed to branch lp:zeitgeist.
>

Good Job! Approved!

« Back to merge proposal