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

Re: svn commit: r19782 - in branches/changelist-feature/subversion: include libsvn_client svn

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: 2006-05-23 20:04:02 CEST

Ben Collins-Sussman wrote:
> On 5/23/06, David James <djames@collab.net> wrote:
>
>> If I want to look for files from changelist foo in the whole wc, I'll
>> specify the path to the root of the WC like so:
>> svn status --changelist foo ~/wc/svn-trunk
>>
>> Maybe this is just my bias as a Subversion developer, but that's how I
>> expect Subversion to work.
>
>
> Your example violates our policy about all arguments being 'global',
> and being able to appear in any order. Anything which isn't an
> argument to a switch (--switch blah) is assumed to be a path or url
> target. In other words, our parser sees no difference between
>
> svn command --changelist foo /some/path
> svn command /some/path --changelist foo
>
> And, in both of these cases, the final target list is [/some/path,
> [members of changelist]].

David's example doesn't violate anything of the sort. I think he's simply
misunderstanding (or proposing an alternative to) what you want --changelist
to mean.

David seems to be thinking of it as a subtractive filtering mechanism, as in
"Narrow the list of targets of this operation down to only those which are
in the changelist FOO". You are proposing it as additive: "Add to the list
of explicitly mentioned targets those items which are in the changelist FOO."

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on Tue May 23 20:03:11 2006

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