817
by Martin Pool
- Deferred patch that uses Python ndiff format. |
1 |
*** modified file 'bzrlib/commands.py' |
2 |
--- bzrlib/commands.py
|
|
3 |
+++ bzrlib/commands.py
|
|
4 |
@@ -721,10 +721,11 @@
|
|
5 |
""" |
|
6 |
||
7 |
takes_args = ['file*'] |
|
8 |
- takes_options = ['revision', 'diff-options']
|
|
9 |
+ takes_options = ['revision', 'diff-options', 'ndiff']
|
|
10 |
aliases = ['di', 'dif'] |
|
11 |
||
12 |
- def run(self, revision=None, file_list=None, diff_options=None):
|
|
13 |
+ def run(self, revision=None, file_list=None, diff_options=None,
|
|
14 |
+ ndiff=None):
|
|
15 |
from bzrlib.diff import show_diff |
|
16 |
||
17 |
if file_list: |
|
18 |
@@ -735,9 +736,15 @@
|
|
19 |
file_list = None |
|
20 |
else: |
|
21 |
b = find_branch('.') |
|
22 |
-
|
|
23 |
+
|
|
24 |
+ if ndiff:
|
|
25 |
+ format = 'ndiff'
|
|
26 |
+ else:
|
|
27 |
+ format = None
|
|
28 |
+
|
|
29 |
show_diff(b, revision, specific_files=file_list, |
|
30 |
- external_diff_options=diff_options)
|
|
31 |
+ external_diff_options=diff_options,
|
|
32 |
+ format=format)
|
|
33 |
||
34 |
||
35 |
||
36 |
@@ -1344,6 +1351,7 @@
|
|
37 |
'format': unicode, |
|
38 |
'forward': None, |
|
39 |
'message': unicode, |
|
40 |
+ 'ndiff': None,
|
|
41 |
'no-recurse': None, |
|
42 |
'profile': None, |
|
43 |
'revision': _parse_revision_str, |
|
44 |
||
45 |
*** modified file 'bzrlib/diff.py' |
|
46 |
--- bzrlib/diff.py
|
|
47 |
+++ bzrlib/diff.py
|
|
48 |
@@ -70,6 +70,13 @@
|
|
49 |
print >>to_file |
|
50 |
||
51 |
||
52 |
+
|
|
53 |
+def internal_ndiff(old_label, oldlines,
|
|
54 |
+ new_label, newlines,
|
|
55 |
+ to_file):
|
|
56 |
+ """Show diff in python-specific ndiff format."""
|
|
57 |
+ from difflib import ndiff
|
|
58 |
+ to_file.writelines(ndiff(oldlines, newlines))
|
|
59 |
||
60 |
||
61 |
def external_diff(old_label, oldlines, new_label, newlines, to_file, |
|
62 |
@@ -152,7 +159,8 @@
|
|
63 |
||
64 |
||
65 |
||
66 |
-def show_diff(b, revision, specific_files, external_diff_options=None):
|
|
67 |
+def show_diff(b, revision, specific_files, external_diff_options=None,
|
|
68 |
+ format=None):
|
|
69 |
"""Shortcut for showing the diff to the working tree. |
|
70 |
||
71 |
b |
|
72 |
@@ -160,6 +168,9 @@
|
|
73 |
||
74 |
revision |
|
75 |
None for each, or otherwise the old revision to compare against. |
|
76 |
+
|
|
77 |
+ format
|
|
78 |
+ 'unified', 'context', 'ndiff', 'external'
|
|
79 |
||
80 |
The more general form is show_diff_trees(), where the caller |
|
81 |
supplies any two trees. |
|
82 |
@@ -174,12 +185,13 @@
|
|
83 |
new_tree = b.working_tree() |
|
84 |
||
85 |
show_diff_trees(old_tree, new_tree, sys.stdout, specific_files, |
|
86 |
- external_diff_options)
|
|
87 |
+ external_diff_options, format)
|
|
88 |
||
89 |
||
90 |
||
91 |
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None, |
|
92 |
- external_diff_options=None):
|
|
93 |
+ external_diff_options=None,
|
|
94 |
+ format=None):
|
|
95 |
"""Show in text form the changes from one tree to another. |
|
96 |
||
97 |
to_files |
|
98 |
@@ -204,10 +216,12 @@
|
|
99 |
if external_diff_options: |
|
100 |
assert isinstance(external_diff_options, basestring) |
|
101 |
opts = external_diff_options.split() |
|
102 |
- def diff_file(olab, olines, nlab, nlines, to_file):
|
|
103 |
+ def diff_fn(olab, olines, nlab, nlines, to_file):
|
|
104 |
external_diff(olab, olines, nlab, nlines, to_file, opts) |
|
105 |
+ elif format == 'ndiff':
|
|
106 |
+ diff_fn = internal_ndiff
|
|
107 |
else: |
|
108 |
- diff_file = internal_diff
|
|
109 |
+ diff_fn = internal_diff
|
|
110 |
||
111 |
||
112 |
delta = compare_trees(old_tree, new_tree, want_unchanged=False, |
|
113 |
@@ -216,7 +230,7 @@
|
|
114 |
for path, file_id, kind in delta.removed: |
|
115 |
print >>to_file, '*** removed %s %r' % (kind, path) |
|
116 |
if kind == 'file': |
|
117 |
- diff_file(old_label + path,
|
|
118 |
+ diff_fn(old_label + path,
|
|
119 |
old_tree.get_file(file_id).readlines(), |
|
120 |
DEVNULL, |
|
121 |
[], |
|
122 |
@@ -225,7 +239,7 @@
|
|
123 |
for path, file_id, kind in delta.added: |
|
124 |
print >>to_file, '*** added %s %r' % (kind, path) |
|
125 |
if kind == 'file': |
|
126 |
- diff_file(DEVNULL,
|
|
127 |
+ diff_fn(DEVNULL,
|
|
128 |
[], |
|
129 |
new_label + path, |
|
130 |
new_tree.get_file(file_id).readlines(), |
|
131 |
@@ -234,7 +248,7 @@
|
|
132 |
for old_path, new_path, file_id, kind, text_modified in delta.renamed: |
|
133 |
print >>to_file, '*** renamed %s %r => %r' % (kind, old_path, new_path) |
|
134 |
if text_modified: |
|
135 |
- diff_file(old_label + old_path,
|
|
136 |
+ diff_fn(old_label + old_path,
|
|
137 |
old_tree.get_file(file_id).readlines(), |
|
138 |
new_label + new_path, |
|
139 |
new_tree.get_file(file_id).readlines(), |
|
140 |
@@ -243,7 +257,7 @@
|
|
141 |
for path, file_id, kind in delta.modified: |
|
142 |
print >>to_file, '*** modified %s %r' % (kind, path) |
|
143 |
if kind == 'file': |
|
144 |
- diff_file(old_label + path,
|
|
145 |
+ diff_fn(old_label + path,
|
|
146 |
old_tree.get_file(file_id).readlines(), |
|
147 |
new_label + path, |
|
148 |
new_tree.get_file(file_id).readlines(), |
|
149 |