unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: "Tomáš Čech" <tcech@suse.cz>
To: 20137@debbugs.gnu.org
Subject: bug#20137: number of generation doesn't always rise monotonically
Date: Mon, 23 Mar 2015 00:40:51 +0100	[thread overview]
Message-ID: <20150322234051.GE3826@venom> (raw)
In-Reply-To: <20150318173550.GE525@venom.suse.cz>

[-- Attachment #1: Type: text/plain, Size: 2100 bytes --]

Sorry, for some reason I didn't get any e-mail and noticed only
closing the bug. I'll subscribe to bug-guix ML to prevent it.


> That’s expected, yes.  What makes you think it’s a problem?

First, it was conflicting experience with what you were saying on IRC.
User experience was a bit confusing because the behaviour seemed
unexpected.

>When implementing that, there were several possible choices:
>
>  1. Upon rollback to N, remove all generations above N.  Rejected
>       because it gratuitously prevents useful use cases.
>
>  2. Upon rollback from P to N, keep all the generations, but use P+1
>     for the next generation number.  Doesn’t work, because rolling back
>     from P+1 would bring you back to P instead of N.

I can't understand this reason. It doesn't look like valid reason to
me, but only consequence of current implementation. It seems that you
don't store information about what was the previous generation and
that is whole point.

>  3. The current behavior.
>
>See <http://lists.gnu.org/archive/html/bug-guix/2013-01/msg00325.html>
>for part of the discussion.

Thanks for looking that up!



>Perhaps we can eventually move to an actual tree structure where the
>nodes can be named whatever.  Until now I thought that's how generations
>work, and are just named after integers for identification purposes.

Yes, I had this in mind as well.


>I agree this would be the right thing for a real undo mechanism.

Exactly!

>But how useful would it be?  I’ve never been in a situation where
>rollback/switch-generation would be insufficient or inappropriate.

I haven't met such situation either (yet).

>I’m concerned that this would add both code and user interface
>complexity for mostly hypothetical use cases.  WDYT?

Yes, it would surely add some more code and would be demanding for
creating good visual represantation for users, but it could also be
much closer to behavior user would expect. And that is something which
makes tools to be natural to use.

Thank you all for your comments.

S_W

[-- Attachment #2: Type: application/pgp-signature, Size: 181 bytes --]

  parent reply	other threads:[~2015-03-22 23:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-18 17:35 bug#20137: number of generation doesn't always rise monotonically Tomáš Čech
2015-03-18 20:36 ` Ludovic Courtès
2015-03-18 20:47   ` Taylan Ulrich Bayırlı/Kammer
2015-03-18 20:52     ` Thompson, David
2015-03-19  8:47     ` Ludovic Courtès
2015-03-22 22:48     ` Ludovic Courtès
2015-03-22 23:40 ` Tomáš Čech [this message]
2015-03-23  9:13   ` Ludovic Courtès
2015-03-23  9:50     ` Tomáš Čech

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150322234051.GE3826@venom \
    --to=tcech@suse.cz \
    --cc=20137@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).