darcs

Issue 1805 changepref idempotency should be a noop

Title changepref idempotency should be a noop
Priority feature Status needs-implementation
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, jaredj, kowey, twb
Assigned To
Topics UI

Created on 2010-03-31.03:45:02 by twb, last changed 2011-08-12.13:23:43 by kowey.

Messages
msg10584 (view) Author: twb Date: 2010-03-31.03:45:01
Attempting to set a preference to its current value should probably
become a noop, just as an add+remove or a move+move-back does.

    $ with-temp-dir
    with-temp-dir: entering directory `/tmp/with-temp-dir.JHJzB2'
    This directory will be deleted when you exit.
    $ darcs init
    $ darcs setpref boringfile x
    Changing value of boringfile from '' to 'x'
    $ darcs rec -lamx
    Finished recording patch 'x'
    $ darcs setpref boringfile x
    Changing value of boringfile from 'x' to 'x'
    $ darcs rec -lamx2
    Finished recording patch 'x2'
    $ darcs cha -v
    Wed Mar 31 14:39:55 EST 2010  Trent W. Buck <trentbuck@gmail.com>
      * x2
        changepref boringfile
        x
        x

    Wed Mar 31 14:39:52 EST 2010  Trent W. Buck <trentbuck@gmail.com>
      * x
        changepref boringfile

        x
msg10589 (view) Author: kowey Date: 2010-03-31.10:19:01
Our preferences system in needs a general overhaul in the long term. 
Searching for 'pref' in BTS may be interesting.
History
Date User Action Args
2010-03-31 03:45:02twbcreate
2010-03-31 10:19:03koweysetpriority: feature
status: unknown -> needs-implementation
topic: + ProbablyEasy, UI
messages: + msg10589
nosy: + kowey, darcs-devel, jaredj
2011-08-12 13:23:43koweysettopic: - ProbablyEasy