[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: Ben Collins-Sussman <sussman_at_collab.net>
Date: 2005-02-23 04:40:43 CET

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?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Feb 23 04:41:55 2005

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