# HG changeset patch # User Matt Mackall # Date 1311280777 18000 # Node ID a934b924957446725cc9acd309a6cc863dc4d86c # Parent 95ced9f5bf2972bfb1d42fd963cbaf1e6f14df45 revert: restore check for uncommitted merge (issue2915) (BC) This will restore the pre-1.9 behavior. diff -r 95ced9f5bf29 -r a934b9249574 mercurial/commands.py --- a/mercurial/commands.py Tue Jul 19 13:43:53 2011 -0500 +++ b/mercurial/commands.py Thu Jul 21 15:39:37 2011 -0500 @@ -4154,9 +4154,11 @@ To cancel a merge (and lose your changes), use :hg:`update --clean .`. With no revision specified, revert the specified files or directories - to the state they had in the first parent of the working directory. + to the contents they had in the parent of the working directory. This restores the contents of files to an unmodified - state and unschedules adds, removes, copies, and renames. + state and unschedules adds, removes, copies, and renames. If the + working directory has two parents, you must explicitly specify a + revision. Using the -r/--rev or -d/--date options, revert the given files or directories to their states as of a specific revision. Because @@ -4179,6 +4181,11 @@ opts["rev"] = cmdutil.finddate(ui, repo, opts["date"]) parent, p2 = repo.dirstate.parents() + if not opts.get('rev') and p2 != nullid: + # revert after merge is a trap for new users (issue2915) + raise util.Abort(_('uncommitted merge with no revision specified'), + hint=_('use "hg update" or see "hg help revert"')) + ctx = scmutil.revsingle(repo, opts.get('rev')) node = ctx.node() diff -r 95ced9f5bf29 -r a934b9249574 tests/test-confused-revert.t --- a/tests/test-confused-revert.t Tue Jul 19 13:43:53 2011 -0500 +++ b/tests/test-confused-revert.t Thu Jul 21 15:39:37 2011 -0500 @@ -59,8 +59,8 @@ Revert should fail: $ hg revert - abort: no files or directories specified - (uncommitted merge, use --all to discard all changes, or 'hg update -C .' to abort the merge) + abort: uncommitted merge with no revision specified + (use "hg update" or see "hg help revert") [255] Revert should be ok now: