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

Re: [PATCH] Fix for issue 3787

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Wed, 16 Mar 2011 09:10:33 +0000

On Mon, 2011-03-14 at 16:02 +0100, Stefan Sperling wrote:
> On Thu, Mar 10, 2011 at 04:49:32PM +0530, Noorul Islam K M wrote:
> >
> > >From issue tracker
> > (http://subversion.tigris.org/issues/show_bug.cgi?id=3787)
> >
> > It would be ever-so-helpful to folks looking to capture and replicate a working
> > copy sparse checkouts configuration if 'svn info -R' would show exclude items,
> > if only optionally, and even then if only just with minimal info:
> >
> > Path: some/excluded-path
> > Depth: exclude
> >
> > Today you cannot look at 'svn info -R' output and get a complete picture of what
> > is and isn't in the sparse configuration.
> > --------------------------------------------------------------------------------
> >
> > I added new option "--show-exclude" to 'svn info' subcommand. When this
> > option is passed to info command along with '-R', it will include nodes
> > with depth 'exclude' also for displaying information. This issue is
> > related to 3792 which is already fixed.
>
> I don't think the new --show-exclude option is necessary.
> We should always show information about excluded items in the output
> svn info -R.

Agreed. I'm not aware of any reason or any precedent for making this
optional.

- Julian

> Can you rework the patch accordingly? That will also make it smaller.
>
> Thanks!
Received on 2011-03-16 10:11:09 CET

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