[PATCH 7 of 7] record: don't honor format-changing diffopts (issue4459)

Siddharth Agarwal sid0 at fb.com
Tue Nov 25 21:54:02 CST 2014


# HG changeset patch
# User Siddharth Agarwal <sid0 at fb.com>
# Date 1416369637 28800
#      Tue Nov 18 20:00:37 2014 -0800
# Node ID 716d9ab5c1424496bea3802cef94283f752d7122
# Parent  7451bf7e463e2932817a08d6b7114ecd363e0cfd
record: don't honor format-changing diffopts (issue4459)

record does support the whitespace options, so honor those. In upcoming patches
we'll replace all the other uses.

diff --git hgext/record.py hgext/record.py
--- hgext/record.py
+++ hgext/record.py
@@ -519,10 +519,9 @@
                                '(use "hg commit" instead)'))
 
         status = repo.status(match=match)
-        diffopts = opts.copy()
-        diffopts['nodates'] = True
-        diffopts['git'] = True
-        diffopts = patch.diffopts(ui, opts=diffopts)
+        diffopts = patch.difffeatureopts(ui, opts=opts, whitespace=True)
+        diffopts.nodates = True
+        diffopts.git = True
         chunks = patch.diff(repo, changes=status, opts=diffopts)
         fp = cStringIO.StringIO()
         fp.write(''.join(chunks))
diff --git tests/test-record.t tests/test-record.t
--- tests/test-record.t
+++ tests/test-record.t
@@ -559,9 +559,9 @@
   >   echo $i >> plain
   > done
 
-Record beginning, middle
+Record beginning, middle, and test that format-breaking diffopts are ignored
 
-  $ hg record -d '14 0' -m middle-only plain <<EOF
+  $ hg record --config diff.noprefix=True -d '14 0' -m middle-only plain <<EOF
   > y
   > y
   > y


More information about the Mercurial-devel mailing list