bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
| 
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  |