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

Re: changelist removal on commit

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-11-03 21:42:59 CET

David Glasser wrote:
> On 11/3/07, Stefan Küng <tortoisesvn@gmail.com> wrote:
>> Hi,
>>
>> There's one small issue I have with how changelists are kept/removed on
>> commits: currently, the changelist is only removed on a commit if the
>> commit is actually done on that particular changelist (and if the flag
>> is not set to keep the changelist).
>>
>> But I think it would be better if after the commit, all affected files
>> have their changelists removed, no matter if the commit was done by
>> specifying a changelist or not.
>>
>> This would make it more consistent. The '--keep-changelist' argument
>> wouldn't be dependent on the '--changelist ARG' parameter.
>>
>> I'm having the following workflow in mind:
>> * user starts working on an issue, modifies files
>> * adds files to changelist
>> * tests issue, changes some more code, and one or more other files
>> * user forgets to add those to the changlist for that issue
>> * user commits files belonging to the issue, forgets about the
>> changelist set
>>
>> now, the files (even though they're committed and now with status
>> 'normal') still belong to the changelist the user set before. The next
>> time one of those files get modified, they will show up in the 'svn st
>> -v' with a changelist name the user might have completely forgotten by then.
>>
>>
>> And of course there's another reason which has to do on how TSVN made up
>> the UI for the changelists and how it does commits: The commits are
>> independent of changelist names - everything the user has selected gets
>> committed. TSVN only helps the user to group items into changelists
>> (i.e., add them to a changelist) and select the items by changelist. But
>> they're still committed without a changelist name.
>> I've changed TSVN now to include the changelist name if all selected
>> items belong to the same changelist. But if even one item is checked for
>> commit which doesn't belong to that changelist, the commit is done the
>> normal way and the changelist doesn't get removed after the commit.
>> We already had some users complain about this, because they just assume
>> that the changelists get removed from those files even if some other
>> files are committed too. They have a hard time understanding why that
>> doesn't happen.
>
> I'm not sure what the best answer to the actual question is, but...
> couldn't TSVN have a submenu or something like "Commit changelist ->
> my-changelist"?

That would be possible, but it wouldn't be good UI.
There's already the possibility to select all entries which are grouped
in a changelist, but I don't like the idea of preventing the user from
selecting other items from other changelists (or which have no
changelist assigned).
The problem arises every time the user selects one or more items which
are not from a changelist, but other items are.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Nov 3 21:43:35 2007

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.