[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r19476 - trunk/subversion/svn

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2006-05-05 20:13:58 CEST

On Fri, May 05, 2006 at 02:06:59PM -0400, Daniel Berlin wrote:
> On Fri, 2006-05-05 at 09:51 -0700, Justin Erenkrantz wrote:
> > On 4/29/06, brane@tigris.org <brane@tigris.org> wrote:
> > > Flush stdout after every line of status output. This is useful when
> > > piping svn output into post-processor tools, e.g. to colorize the
> > > output, or simply into tee.
> >
> > Once again, I think this is an incredibly bad change. This patch
> > promotes bad behavior by lazy developers who want to treat Subversion
> > as CVS by building dumb wrapper scripts.
>
> I also think this is a very bad change.
> Relying on flushing after every line, anywhere, is just really really
> bad design.
>

I don't think anyone is *relying* on flushing, it's just that they'd
like to see progress 'as it happens' if they run something like:

$ svn st | tee foo

rather than getting the output in chunks, or potentially not until the
process completes.

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 5 20:14:35 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.