[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:10:06 CEST

On Fri, May 05, 2006 at 09:51:29AM -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.
>

Once again: Subversion is scriptable by design (it's even mentioned on
our front page).

I honestly don't have much CVS experience, so I don't know specifically
what problems you're referring to above.

Could you explain a bit more about what kinds of problems 'dumb wrapper
scripts' have caused for CVS (or users of CVS)? At the moment, I'm
struggling to see why you're against this change.

> That said, this commit adds dead code - see the snippet below. Please fix.
>

That does need fixing, though.

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:10:31 2006

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