C. Michael Pilato wrote:
> Greg Stein wrote:
>> On Tue, Dec 16, 2008 at 14:18, Eric Gillespie <epg_at_pretzelnet.org> wrote:
>>> Ben Collins-Sussman <sussman_at_red-bean.com> writes:
>>>
>>>> That said, your use-case #2 never occurred to me, nor have I ever
>>>> heard anyone ask for it. I think it might be interesting to have a
>>> It's pretty common. p4 has '-c default' and we implemented '-c
>>> none' for gvn, which I found immensely useful. Maybe Shachar
>>> would like to take a crack at it? :)
>> There have been quite a few times that I've wanted to refer to "files
>> not in a changelist" with svn. I have had a couple changes pending,
>> marked as a changelist, but then do something *else* and want to
>> "check that other stuff, not on a changelist". How to easily refer to
>> those? If I do an "svn commit", will it do files *not* in a
>> changelist, or does it grab all files? I'd also like to do "svn diff"
>> on the not-changelist files.
>>
>> It would lso be nice to say "add not-changelist files to <this> changelist".
>>
>> I've figured that --changelist "" would be nice to refer to the
>> default, but it actually means a changelist named "".
>
> That's almost certainly not an intended consequence. I was going to, in
> fact, recommend that at the API level, we let the empty changelist name mean
> "not in a changelist", and at the UI level we add --without-changelist (as
> mutually exclusive with --changelist). I think we could make that change
> today without revving a bunch of APIs. Let's just call the current behavior
> a bug, because I can assure you that I didn't knowingly add that behavior
> when I rewrote this feature, and I betcha Ben didn't choose to allow that on
> purpose on his first crack at this, either.
Agreed. And even if the --without-changelist stuff doesn't get implemented in
time for 1.6, it'd probably be useful to put a simple check in the client
libraries to prevent unintended use of the empty changelist going forward.
-Hyrum
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=985760
Received on 2008-12-17 16:06:51 CET