Patchwork [2,of,6] backout: remove unnecessary frobbing of addremove option

mail settings
Submitter Kevin Bullock
Date Feb. 13, 2013, 7:47 p.m.
Message ID <>
Download mbox | patch
Permalink /patch/981/
State Accepted
Commit 0bca4d31f647a76696e94c6e0fae77a7b50eb30a
Headers show


Kevin Bullock - Feb. 13, 2013, 7:47 p.m.
# HG changeset patch
# User Kevin Bullock <>
# Date 1360684050 0
# Node ID 67fff6f7f7bbe643f8cb21567d05d1533589c94d
# Parent  a6300d15b1e82121a9b646126ba35809068f340e
backout: remove unnecessary frobbing of addremove option

There's no way for addremove to show up in backout's opts dictionary. It
was being set manually because cmdutil.commit expected it to be there
(and would throw an exception if it wasn't). This was fixed waaaaaaay
back in:

changeset:   5829:784073457a0f
user:        Kirill Smelkov <>
date:        Thu Jan 10 12:07:18 2008 +0300
summary:     cmdutil.commit: extract 'addremove' from opts carefully


diff --git a/mercurial/ b/mercurial/
--- a/mercurial/
+++ b/mercurial/
@@ -468,7 +468,6 @@  def backout(ui, repo, node=None, rev=Non
                 ui.setconfig('ui', 'forcemerge', '')
         commit_opts = opts.copy()
-        commit_opts['addremove'] = False
         if not commit_opts['message'] and not commit_opts['logfile']:
             # we don't translate commit messages
             commit_opts['message'] = "Backed out changeset %s" % short(node)