Re: [PATCH] Print progress message to stderr, not stdout

From: Marco Costalba <mcostalba@gmail.com>
Date: Dec 4, 2006, 18:34 UTC
Message-ID: <e5bfff550612041034g727a1bebg464f7c523c0fac7f@mail.gmail.com>
In-Reply-To: <20061204153705.GA8644@diana.vm.bytemark.co.uk>

On 12/4/06, Karl Hasselström <kha@treskal.com> wrote:

Show 22 quoted lines

> On 2006-12-04 09:17:16 +0000, Catalin Marinas wrote:
>
> > On 02/12/06, Marco Costalba <mcostalba@gmail.com> wrote:
> >
> > > On 11/11/06, Karl Hasselström <kha@treskal.com> wrote:
> > >
> > > > Printing progress messages to stdout causes them to get mixed up
> > > > with the actual output of the program. Using stderr is much
> > > > better, since the user can then redirect the two components
> > > > separately.
> > >
> > > This patch breaks qgit.
> >
> > Since there are other tools relying on a clean stderr, I think I
> > would revert it and add a verbose flag and/or config option. Karl,
> > any thoughts on this (since you sent the patch)?
>
> I introduced this since I wanted to divert the output to a file, and
> the progress message had no business being written to that file. But a
> command line option to suppress progress messages would work just as
> well if that's what git does.
>

If you don't mind I would prefer a command line option to _enable_ progress messages, something like -v or --verbose so to keep back compatibility with current versions of tools that do not expect stderr messages.

Thanks