David Glasser wrote:
>> FWIW, it sounds like you both are saying it should work the same way.
>> The part that I suspect Julian was missing was that before you made
>> this change many of the subcommands were essentially adding what was
>> in the changelist to what was in CWD which really does not make any
>> sense. And clearly I think a user expects:
>>
>> svn info --changelist foo /path
>>
>> To show the info for the items in /path that belong to changelist foo.
>> That is what Julian says it should do and that is what Mike just made
>> it do with these changes.
>
> No, it doesn't now, because Mike's changes made this mean "show info
> about everything that 'svn info /path' would show which is on
> changelist foo"; since info is by default depth=empty, that would be
> nothing.
Hence, my suggestion that the presence of --changelist implies '--depth
infinity' (unless --depth is also explicitly provided). I *think* that will
please the majority of folks, and I'm sorry I didn't think of it before.
Are there objections to my making exactly this change? It should be an easy
tweak to svn/main.c.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2008-01-27 00:02:14 CET