D4379: resolve: add config to make hg resolve not re-merge by default
valentin.gatienbaron (Valentin Gatien-Baron)
phabricator at mercurial-scm.org
Wed Aug 29 16:28:42 EDT 2018
valentin.gatienbaron updated this revision to Diff 10659.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D4379?vs=10577&id=10659
REVISION DETAIL
https://phab.mercurial-scm.org/D4379
AFFECTED FILES
mercurial/commands.py
mercurial/configitems.py
mercurial/help/config.txt
tests/test-resolve.t
CHANGE DETAILS
diff --git a/tests/test-resolve.t b/tests/test-resolve.t
--- a/tests/test-resolve.t
+++ b/tests/test-resolve.t
@@ -448,12 +448,22 @@
U file1
R file2
$ hg resolve --mark --re-merge
- abort: too many options specified
+ abort: too many actions specified
[255]
$ hg resolve --re-merge --all
merging file1
warning: conflicts while merging file1! (edit, then use 'hg resolve --mark')
[1]
+Explicit re-merge
+ $ hg resolve --unmark file1
+ $ hg resolve --config commands.resolve.explicit-re-merge=1 --all
+ abort: no action specified
+ (use --mark, --unmark, --list or --re-merge)
+ [255]
+ $ hg resolve --config commands.resolve.explicit-re-merge=1 --re-merge --all
+ merging file1
+ warning: conflicts while merging file1! (edit, then use 'hg resolve --mark')
+ [1]
$ cd ..
diff --git a/mercurial/help/config.txt b/mercurial/help/config.txt
--- a/mercurial/help/config.txt
+++ b/mercurial/help/config.txt
@@ -442,6 +442,11 @@
Confirm before performing action if no filename is passed.
(default: False)
+``resolve.explicit-re-merge``
+ Require uses of ``hg resolve`` to specify which action it should perform,
+ instead of re-merging files by default.
+ (default: False)
+
``resolve.mark-check``
Determines what level of checking :hg:`resolve --mark` will perform before
marking files as resolved. Valid values are ``none`, ``warn``, and
diff --git a/mercurial/configitems.py b/mercurial/configitems.py
--- a/mercurial/configitems.py
+++ b/mercurial/configitems.py
@@ -193,6 +193,9 @@
coreconfigitem('commands', 'resolve.confirm',
default=False,
)
+coreconfigitem('commands', 'resolve.explicit-re-merge',
+ default=False,
+)
coreconfigitem('commands', 'resolve.mark-check',
default='none',
)
diff --git a/mercurial/commands.py b/mercurial/commands.py
--- a/mercurial/commands.py
+++ b/mercurial/commands.py
@@ -4559,8 +4559,13 @@
all, mark, unmark, show, nostatus, remerge = \
[opts.get(o) for o in flaglist]
- if len(list(filter(None, [show, mark, unmark, remerge]))) > 1:
- raise error.Abort(_("too many options specified"))
+ actioncount = len(list(filter(None, [show, mark, unmark, remerge])))
+ if actioncount > 1:
+ raise error.Abort(_("too many actions specified"))
+ elif (actioncount == 0
+ and ui.config('commands', 'resolve.explicit-re-merge')):
+ hint = _('use --mark, --unmark, --list or --re-merge')
+ raise error.Abort(_('no action specified'), hint=hint)
if pats and all:
raise error.Abort(_("can't specify --all and patterns"))
if not (all or pats or show or mark or unmark):
To: valentin.gatienbaron, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list