diff --git a/mercurial/cmdutil.py b/mercurial/cmdutil.py --- a/mercurial/cmdutil.py +++ b/mercurial/cmdutil.py @@ -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): diff --git a/tests/test-uncommit.t b/tests/test-uncommit.t --- a/tests/test-uncommit.t +++ b/tests/test-uncommit.t @@ -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