[darcs-devel] [issue2050] UI improvement in selecting tags in darcs rollback

Duncan Coutts bugs at darcs.net
Sun Mar 6 19:45:50 UTC 2011


New submission from Duncan Coutts <duncan.coutts at googlemail.com>:

When doing darcs rollback and the user intendes to rollback a rather
patch that is behind a tag the UI is rather confusing.

The UI misleadingly indicates that the tags will be rolled back too.
This is not what the user wants to do. The user wants to roll back
just the old patch and to leave all the tags alone.

In fact the tags are not rolled back, it is just the UI that gives
this misleading impression.

Example:  (this is in the Cabal repo)

$ darcs-2.7.3 rollback -p '(now in separate repos)'
Sun Oct 10 16:50:50 BST 2010  Duncan Coutts <duncan at haskell.org>
  tagged 1.10 branch forked
Shall I rollback this patch? (1/13)  [ynW...], or ? for more options: n
Will not ask whether to rollback 12 already decided patches.
No patches selected!

So I select a patch with -p that I want to rollback, and it offers me
a tag. I don't want to rollback any tags so I say 'n'. darcs then
tells me that there are no patches, selected, ie the patch that I want
to rollback depend on me also rolling back all these tags. That's not
what I want to do!

I try it anyway:

$ darcs-2.7.3 rollback -p '(now in separate repos)'
Sun Oct 10 16:50:50 BST 2010  Duncan Coutts <duncan at haskell.org>
  tagged 1.10 branch forked
Shall I rollback this patch? (1/13)  [ynW...], or ? for more options: w
[... repeat 12 times ...]
Thu Apr  5 20:47:29 BST 2007  Ian Lynagh <igloo at earth.li>
  * Remove cabal-{builder,install,setup,upload} (now in separate repos)
Shall I rollback this patch? (13/13)  [ynW...], or ? for more options: y
[...snip...]
Shall I rollback this change? (1/110)  [ynW...], or ? for more options: a
What is the patch name? Don't delete cabal-install from the cabal repo
Do you want to add a long comment? [yn]y

Now the message in the editor for the long comment lists all these
tags, again giving the impression that they are being rolled back:

Sun Mar  6 18:19:42 GMT 2011  Duncan Coutts <duncan at community.haskell.org>
  * Don't delete cabal-install from the cabal repo
  This is in preparation to merging cabal and cabal-install back into one repo

  rolling back:

  Thu Apr  5 20:47:29 BST 2007  Ian Lynagh <igloo at earth.li>
    * Remove cabal-{builder,install,setup,upload} (now in separate repos)

      R ./cabal-install/
      [..snip..]

  Fri Jun 29 12:25:45 BST 2007  Ian Lynagh <igloo at earth.li>
    tagged 2007-06-29

  [..etc another 10 tags that look like they're rolled back...]

  Sun Oct 10 16:50:50 BST 2010  Duncan Coutts <duncan at haskell.org>
    tagged 1.10 branch forked


So I think I've messed things up by rolling back all these tags, but
now I run darcs show tags and lo..

$ darcs show tags
1.10 branch forked
2009-06-25
1.6.0.1
1.6.0.0
6.10 branch has been forked
1.5.5
2008-05-28
1.5.1
1.3.9
2007-09-06
1.2.0
2007-06-29

They're all there!

So actually despite my dire expectations, this operation has done
exactly what I wanted!

So, how could the UI be improved here. Do we need to prompt users for
tags at all? Can tags even be rolled back? From my point of view as a
naive user I would not expect to be prompted to rollback any tags that
depend on the patch I want to roll back and I would not expect them to
be listed in the commit message.

I understand why when doing unrecord say, we have to also select all
the patches that depend on the one I want. But for rollback, we're
making a new patch on the top of the pile so I don't see how tag
dependencies come into it.

Duncan

----------
messages: 13771
nosy: duncan.coutts
status: unknown
title: UI improvement in selecting tags in darcs rollback

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/issue2050>
__________________________________


More information about the darcs-devel mailing list