uncommit: make -D/--date and -U/--user mutually exclusive
authorMatt Harbison <matt_harbison@yahoo.com>
Tue, 10 Sep 2019 22:52:04 -0400
changeset 42921 4690902850df
parent 42920 a50661567f83
child 42922 8d9322b6e687
uncommit: make -D/--date and -U/--user mutually exclusive This is how amend and graft work (but not MQ). I'm not sure why this didn't work for me when I first tried it. Differential Revision: https://phab.mercurial-scm.org/D6842
mercurial/cmdutil.py
tests/test-uncommit.t
--- a/mercurial/cmdutil.py	Tue Sep 10 22:04:22 2019 -0400
+++ b/mercurial/cmdutil.py	Tue Sep 10 22:52:04 2019 -0400
@@ -185,10 +185,17 @@
 def resolvecommitoptions(ui, opts):
     """modify commit options dict to handle related options
     """
+    if opts.get('date') and opts.get('currentdate'):
+        raise error.Abort(_('--date and --currentdate are mutually '
+                            'exclusive'))
+    if opts.get(b'user') and opts.get(b'currentuser'):
+        raise error.Abort(_('--user and --currentuser are mutually '
+                            'exclusive'))
+
     # N.B. this is extremely similar to setupheaderopts() in mq.py
-    if not opts.get(b'date') and opts.get(b'currentdate'):
+    if opts.get(b'currentdate'):
         opts[b'date'] = b'%d %d' % dateutil.makedate()
-    if not opts.get(b'user') and opts.get(b'currentuser'):
+    if opts.get(b'currentuser'):
         opts[b'user'] = ui.username()
 
 def ishunk(x):
--- a/tests/test-uncommit.t	Tue Sep 10 22:04:22 2019 -0400
+++ b/tests/test-uncommit.t	Tue Sep 10 22:52:04 2019 -0400
@@ -549,10 +549,18 @@
   date:        Mon Jun 30 12:12:12 1980 +0000
   summary:     uncommit with message
   
+Bad option combinations
+
+  $ hg rollback -q --config ui.rollback=True
+  $ hg uncommit -U --user 'user'
+  abort: --user and --currentuser are mutually exclusive
+  [255]
+  $ hg uncommit -D --date today
+  abort: --date and --currentdate are mutually exclusive
+  [255]
 
 `uncommit <dir>` and `cd <dir> && uncommit .` behave the same...
 
-  $ hg rollback -q --config ui.rollback=True
   $ echo 2 > dir/file2.txt
   $ hg ci -Aqm 'add file2 in directory'
   $ hg uncommit dir