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

Re: svn commit: r13123 - in trunk/subversion: clients/cmdline include libsvn_client

From: John Szakmeister <john_at_szakmeister.net>
Date: 2005-02-23 04:46:57 CET

On Tuesday 22 February 2005 22:40, Ben Collins-Sussman wrote:
> On Feb 22, 2005, at 9:30 PM, John Szakmeister wrote:
> > This change broke switch tests 11 and 13 (at least on my build). In
> > these
> > 2 test cases, svn info was being run on a directory, and hence, the
> > entry->name was "". Therefore, the "Name:" was never printed before.
> > With this change, it's always printing the "Name:" line.
> >
> > The attached patch works against ra_local. I haven't tested it
> > against the other layers.
>
> Why such a complex patch to the switch code? Why not just make the new
> 'svn info' printing code behave like the old code? Why not just have
> the code not print an empty basename?

I was so confused by your comment... then I realized I sent the wrong
patch! The switch patch was something I did a while ago to prevent
switching files to dirs and vice versa.

Here's the real patch... all 1 line of it! :-)

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Wed Feb 23 05:50:38 2005

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.