On Wed, 2008-12-17 at 09:46 -0500, 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.
+1 that it's a bug if the empty string can be used as a changelist name.
- Julian
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=985750
Received on 2008-12-17 15:58:53 CET