1658.1.9
by Martin Pool
Give an error for bzr diff on an nonexistent file (Malone #3619) |
1 |
# Copyright (C) 2004, 2005, 2006 Canonical Ltd.
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
2 |
#
|
1
by mbp at sourcefrog
import from baz patch-364 |
3 |
# This program is free software; you can redistribute it and/or modify
|
4 |
# it under the terms of the GNU General Public License as published by
|
|
5 |
# the Free Software Foundation; either version 2 of the License, or
|
|
6 |
# (at your option) any later version.
|
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
7 |
#
|
1
by mbp at sourcefrog
import from baz patch-364 |
8 |
# This program is distributed in the hope that it will be useful,
|
9 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
10 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
11 |
# GNU General Public License for more details.
|
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
12 |
#
|
1
by mbp at sourcefrog
import from baz patch-364 |
13 |
# You should have received a copy of the GNU General Public License
|
14 |
# along with this program; if not, write to the Free Software
|
|
15 |
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
|
|
16 |
||
2520.4.140
by Aaron Bentley
Use matching blocks from mpdiff for knit delta creation |
17 |
import difflib |
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
18 |
import os |
1899.1.5
by John Arbash Meinel
Always buffer the output of diff, so we can check if retcode==2 is because of Binary files |
19 |
import re |
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
20 |
import sys |
21 |
||
22 |
from bzrlib.lazy_import import lazy_import |
|
23 |
lazy_import(globals(), """ |
|
24 |
import errno
|
|
1692.8.7
by James Henstridge
changes suggested by John Meinel |
25 |
import subprocess
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
26 |
import tempfile
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
27 |
import time
|
28 |
||
1955.2.10
by John Arbash Meinel
Unset a few other LANG type variables when spawning diff |
29 |
from bzrlib import (
|
30 |
errors,
|
|
31 |
osutils,
|
|
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
32 |
patiencediff,
|
33 |
textfile,
|
|
1551.12.29
by Aaron Bentley
Copy and extend patch date formatting code, add patch-date parsing |
34 |
timestamp,
|
1955.2.10
by John Arbash Meinel
Unset a few other LANG type variables when spawning diff |
35 |
)
|
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
36 |
""") |
37 |
||
1852.10.3
by Robert Collins
Remove all uses of compare_trees and replace with Tree.changes_from throughout bzrlib. |
38 |
# compatability - plugins import compare_trees from diff!!!
|
39 |
# deprecated as of 0.10
|
|
1534.4.35
by Robert Collins
Give branch its own basis tree and last_revision methods; deprecated branch.working_tree() |
40 |
from bzrlib.delta import compare_trees |
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
41 |
from bzrlib.symbol_versioning import ( |
42 |
deprecated_function, |
|
43 |
zero_eight, |
|
44 |
)
|
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
45 |
from bzrlib.trace import mutter, warning |
1
by mbp at sourcefrog
import from baz patch-364 |
46 |
|
1711.2.24
by John Arbash Meinel
Late bind to PatienceSequenceMatcher to allow plugin to override. |
47 |
|
767
by Martin Pool
- files are only reported as modified if their name or parent has changed, |
48 |
# TODO: Rather than building a changeset object, we should probably
|
49 |
# invoke callbacks on an object. That object can either accumulate a
|
|
50 |
# list, write them out directly, etc etc.
|
|
51 |
||
2520.4.140
by Aaron Bentley
Use matching blocks from mpdiff for knit delta creation |
52 |
|
53 |
class _PrematchedMatcher(difflib.SequenceMatcher): |
|
54 |
"""Allow SequenceMatcher operations to use predetermined blocks"""
|
|
55 |
||
56 |
def __init__(self, matching_blocks): |
|
57 |
difflib.SequenceMatcher(self, None, None) |
|
58 |
self.matching_blocks = matching_blocks |
|
59 |
self.opcodes = None |
|
60 |
||
61 |
||
1558.15.11
by Aaron Bentley
Apply merge review suggestions |
62 |
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file, |
1711.2.30
by John Arbash Meinel
Fix bug in internal_diff handling of unicode paths |
63 |
allow_binary=False, sequence_matcher=None, |
64 |
path_encoding='utf8'): |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
65 |
# FIXME: difflib is wrong if there is no trailing newline.
|
66 |
# The syntax used by patch seems to be "\ No newline at
|
|
67 |
# end of file" following the last diff line from that
|
|
68 |
# file. This is not trivial to insert into the
|
|
69 |
# unified_diff output and it might be better to just fix
|
|
70 |
# or replace that function.
|
|
71 |
||
72 |
# In the meantime we at least make sure the patch isn't
|
|
73 |
# mangled.
|
|
74 |
||
75 |
||
76 |
# Special workaround for Python2.3, where difflib fails if
|
|
77 |
# both sequences are empty.
|
|
78 |
if not oldlines and not newlines: |
|
79 |
return
|
|
1558.15.2
by Aaron Bentley
Implemented binary file handling for diff |
80 |
|
1558.15.11
by Aaron Bentley
Apply merge review suggestions |
81 |
if allow_binary is False: |
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
82 |
textfile.check_text_lines(oldlines) |
83 |
textfile.check_text_lines(newlines) |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
84 |
|
1185.81.8
by John Arbash Meinel
Updating unified_diff to take a factory, using the new diff algorithm in the code. |
85 |
if sequence_matcher is None: |
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
86 |
sequence_matcher = patiencediff.PatienceSequenceMatcher |
87 |
ud = patiencediff.unified_diff(oldlines, newlines, |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
88 |
fromfile=old_filename.encode(path_encoding), |
89 |
tofile=new_filename.encode(path_encoding), |
|
1185.81.8
by John Arbash Meinel
Updating unified_diff to take a factory, using the new diff algorithm in the code. |
90 |
sequencematcher=sequence_matcher) |
475
by Martin Pool
- rewrite diff using compare_trees() |
91 |
|
1092.1.50
by Robert Collins
make diff lsdiff/filterdiff friendly |
92 |
ud = list(ud) |
475
by Martin Pool
- rewrite diff using compare_trees() |
93 |
# work-around for difflib being too smart for its own good
|
94 |
# if /dev/null is "1,0", patch won't recognize it as /dev/null
|
|
95 |
if not oldlines: |
|
96 |
ud[2] = ud[2].replace('-1,0', '-0,0') |
|
97 |
elif not newlines: |
|
98 |
ud[2] = ud[2].replace('+1,0', '+0,0') |
|
1092.1.50
by Robert Collins
make diff lsdiff/filterdiff friendly |
99 |
# work around for difflib emitting random spaces after the label
|
100 |
ud[0] = ud[0][:-2] + '\n' |
|
101 |
ud[1] = ud[1][:-2] + '\n' |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
102 |
|
804
by Martin Pool
Patch from John: |
103 |
for line in ud: |
104 |
to_file.write(line) |
|
974.1.5
by Aaron Bentley
Fixed handling of missing newlines in udiffs |
105 |
if not line.endswith('\n'): |
106 |
to_file.write("\n\\ No newline at end of file\n") |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
107 |
print >>to_file |
108 |
||
109 |
||
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
110 |
def _spawn_external_diff(diffcmd, capture_errors=True): |
111 |
"""Spawn the externall diff process, and return the child handle.
|
|
112 |
||
113 |
:param diffcmd: The command list to spawn
|
|
2138.1.1
by Wouter van Heyst
Robuster external diff output handling. |
114 |
:param capture_errors: Capture stderr as well as setting LANG=C
|
115 |
and LC_ALL=C. This lets us read and understand the output of diff,
|
|
116 |
and respond to any errors.
|
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
117 |
:return: A Popen object.
|
118 |
"""
|
|
119 |
if capture_errors: |
|
2321.2.2
by Alexander Belchenko
win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) |
120 |
# construct minimal environment
|
121 |
env = {} |
|
122 |
path = os.environ.get('PATH') |
|
123 |
if path is not None: |
|
124 |
env['PATH'] = path |
|
2321.2.5
by Alexander Belchenko
external diff: no need for special code path for win32 (suggested by John Meinel) |
125 |
env['LANGUAGE'] = 'C' # on win32 only LANGUAGE has effect |
126 |
env['LANG'] = 'C' |
|
127 |
env['LC_ALL'] = 'C' |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
128 |
stderr = subprocess.PIPE |
129 |
else: |
|
2321.2.2
by Alexander Belchenko
win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) |
130 |
env = None |
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
131 |
stderr = None |
132 |
||
133 |
try: |
|
134 |
pipe = subprocess.Popen(diffcmd, |
|
135 |
stdin=subprocess.PIPE, |
|
136 |
stdout=subprocess.PIPE, |
|
137 |
stderr=stderr, |
|
2321.2.2
by Alexander Belchenko
win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) |
138 |
env=env) |
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
139 |
except OSError, e: |
140 |
if e.errno == errno.ENOENT: |
|
141 |
raise errors.NoDiff(str(e)) |
|
142 |
raise
|
|
143 |
||
144 |
return pipe |
|
145 |
||
146 |
||
1185.35.29
by Aaron Bentley
Support whitespace in diff filenames |
147 |
def external_diff(old_filename, oldlines, new_filename, newlines, to_file, |
571
by Martin Pool
- new --diff-options to pass options through to external |
148 |
diff_opts): |
568
by Martin Pool
- start adding support for showing diffs by calling out to |
149 |
"""Display a diff by calling out to the external diff program."""
|
581
by Martin Pool
- make sure any bzr output is flushed before |
150 |
# make sure our own output is properly ordered before the diff
|
151 |
to_file.flush() |
|
152 |
||
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
153 |
oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-') |
154 |
newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-') |
|
155 |
oldtmpf = os.fdopen(oldtmp_fd, 'wb') |
|
156 |
newtmpf = os.fdopen(newtmp_fd, 'wb') |
|
568
by Martin Pool
- start adding support for showing diffs by calling out to |
157 |
|
158 |
try: |
|
159 |
# TODO: perhaps a special case for comparing to or from the empty
|
|
160 |
# sequence; can just use /dev/null on Unix
|
|
161 |
||
162 |
# TODO: if either of the files being compared already exists as a
|
|
163 |
# regular named file (e.g. in the working directory) then we can
|
|
164 |
# compare directly to that, rather than copying it.
|
|
165 |
||
166 |
oldtmpf.writelines(oldlines) |
|
167 |
newtmpf.writelines(newlines) |
|
168 |
||
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
169 |
oldtmpf.close() |
170 |
newtmpf.close() |
|
568
by Martin Pool
- start adding support for showing diffs by calling out to |
171 |
|
571
by Martin Pool
- new --diff-options to pass options through to external |
172 |
if not diff_opts: |
173 |
diff_opts = [] |
|
174 |
diffcmd = ['diff', |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
175 |
'--label', old_filename, |
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
176 |
old_abspath, |
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
177 |
'--label', new_filename, |
1711.2.56
by John Arbash Meinel
Raise NoDiff if 'diff' not present. |
178 |
new_abspath, |
179 |
'--binary', |
|
180 |
]
|
|
571
by Martin Pool
- new --diff-options to pass options through to external |
181 |
|
182 |
# diff only allows one style to be specified; they don't override.
|
|
183 |
# note that some of these take optargs, and the optargs can be
|
|
184 |
# directly appended to the options.
|
|
185 |
# this is only an approximate parser; it doesn't properly understand
|
|
186 |
# the grammar.
|
|
187 |
for s in ['-c', '-u', '-C', '-U', |
|
188 |
'-e', '--ed', |
|
189 |
'-q', '--brief', |
|
190 |
'--normal', |
|
191 |
'-n', '--rcs', |
|
192 |
'-y', '--side-by-side', |
|
193 |
'-D', '--ifdef']: |
|
194 |
for j in diff_opts: |
|
195 |
if j.startswith(s): |
|
196 |
break
|
|
197 |
else: |
|
198 |
continue
|
|
199 |
break
|
|
200 |
else: |
|
201 |
diffcmd.append('-u') |
|
202 |
||
203 |
if diff_opts: |
|
204 |
diffcmd.extend(diff_opts) |
|
205 |
||
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
206 |
pipe = _spawn_external_diff(diffcmd, capture_errors=True) |
207 |
out,err = pipe.communicate() |
|
208 |
rc = pipe.returncode |
|
571
by Martin Pool
- new --diff-options to pass options through to external |
209 |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
210 |
# internal_diff() adds a trailing newline, add one here for consistency
|
211 |
out += '\n' |
|
1899.1.5
by John Arbash Meinel
Always buffer the output of diff, so we can check if retcode==2 is because of Binary files |
212 |
if rc == 2: |
213 |
# 'diff' gives retcode == 2 for all sorts of errors
|
|
214 |
# one of those is 'Binary files differ'.
|
|
215 |
# Bad options could also be the problem.
|
|
1904.1.4
by Marien Zwart
Make external diff in binary mode work with recent versions of diffutils. |
216 |
# 'Binary files' is not a real error, so we suppress that error.
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
217 |
lang_c_out = out |
218 |
||
219 |
# Since we got here, we want to make sure to give an i18n error
|
|
220 |
pipe = _spawn_external_diff(diffcmd, capture_errors=False) |
|
221 |
out, err = pipe.communicate() |
|
222 |
||
223 |
# Write out the new i18n diff response
|
|
224 |
to_file.write(out+'\n') |
|
225 |
if pipe.returncode != 2: |
|
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
226 |
raise errors.BzrError( |
227 |
'external diff failed with exit code 2'
|
|
2138.1.1
by Wouter van Heyst
Robuster external diff output handling. |
228 |
' when run with LANG=C and LC_ALL=C,'
|
229 |
' but not when run natively: %r' % (diffcmd,)) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
230 |
|
231 |
first_line = lang_c_out.split('\n', 1)[0] |
|
1904.1.4
by Marien Zwart
Make external diff in binary mode work with recent versions of diffutils. |
232 |
# Starting with diffutils 2.8.4 the word "binary" was dropped.
|
233 |
m = re.match('^(binary )?files.*differ$', first_line, re.I) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
234 |
if m is None: |
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
235 |
raise errors.BzrError('external diff failed with exit code 2;' |
236 |
' command: %r' % (diffcmd,)) |
|
1920.1.1
by John Arbash Meinel
fix bug #56307, handle binary files even when LANG is not english |
237 |
else: |
238 |
# Binary files differ, just return
|
|
239 |
return
|
|
240 |
||
241 |
# If we got to here, we haven't written out the output of diff
|
|
242 |
# do so now
|
|
243 |
to_file.write(out) |
|
244 |
if rc not in (0, 1): |
|
571
by Martin Pool
- new --diff-options to pass options through to external |
245 |
# returns 1 if files differ; that's OK
|
246 |
if rc < 0: |
|
247 |
msg = 'signal %d' % (-rc) |
|
248 |
else: |
|
249 |
msg = 'exit code %d' % rc |
|
250 |
||
1996.3.9
by John Arbash Meinel
lazy_import diff.py |
251 |
raise errors.BzrError('external diff failed with %s; command: %r' |
252 |
% (rc, diffcmd)) |
|
1899.1.6
by John Arbash Meinel
internal_diff always adds a trailing \n, make sure external_diff does too |
253 |
|
254 |
||
568
by Martin Pool
- start adding support for showing diffs by calling out to |
255 |
finally: |
256 |
oldtmpf.close() # and delete |
|
257 |
newtmpf.close() |
|
1711.2.54
by John Arbash Meinel
Use mkstemp instead of NamedTemporary file for external diff. |
258 |
# Clean up. Warn in case the files couldn't be deleted
|
259 |
# (in case windows still holds the file open, but not
|
|
260 |
# if the files have already been deleted)
|
|
261 |
try: |
|
262 |
os.remove(old_abspath) |
|
263 |
except OSError, e: |
|
264 |
if e.errno not in (errno.ENOENT,): |
|
265 |
warning('Failed to delete temporary file: %s %s', |
|
266 |
old_abspath, e) |
|
267 |
try: |
|
268 |
os.remove(new_abspath) |
|
269 |
except OSError: |
|
270 |
if e.errno not in (errno.ENOENT,): |
|
271 |
warning('Failed to delete temporary file: %s %s', |
|
272 |
new_abspath, e) |
|
568
by Martin Pool
- start adding support for showing diffs by calling out to |
273 |
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
274 |
|
275 |
@deprecated_function(zero_eight) |
|
1432
by Robert Collins
branch: namespace |
276 |
def show_diff(b, from_spec, specific_files, external_diff_options=None, |
1185.35.28
by Aaron Bentley
Support diff with two branches as input. |
277 |
revision2=None, output=None, b2=None): |
619
by Martin Pool
doc |
278 |
"""Shortcut for showing the diff to the working tree.
|
279 |
||
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
280 |
Please use show_diff_trees instead.
|
281 |
||
619
by Martin Pool
doc |
282 |
b
|
283 |
Branch.
|
|
284 |
||
285 |
revision
|
|
1432
by Robert Collins
branch: namespace |
286 |
None for 'basis tree', or otherwise the old revision to compare against.
|
619
by Martin Pool
doc |
287 |
|
288 |
The more general form is show_diff_trees(), where the caller
|
|
289 |
supplies any two trees.
|
|
290 |
"""
|
|
1092.1.47
by Robert Collins
make show_diff redirectable |
291 |
if output is None: |
292 |
output = sys.stdout |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
293 |
|
1432
by Robert Collins
branch: namespace |
294 |
if from_spec is None: |
1508.1.19
by Robert Collins
Give format3 working trees their own last-revision marker. |
295 |
old_tree = b.bzrdir.open_workingtree() |
1185.35.28
by Aaron Bentley
Support diff with two branches as input. |
296 |
if b2 is None: |
1534.4.35
by Robert Collins
Give branch its own basis tree and last_revision methods; deprecated branch.working_tree() |
297 |
old_tree = old_tree = old_tree.basis_tree() |
329
by Martin Pool
- refactor command functions into command classes |
298 |
else: |
1185.67.2
by Aaron Bentley
Renamed Branch.storage to Branch.repository |
299 |
old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id) |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
300 |
|
1185.11.5
by John Arbash Meinel
Merged up-to-date against mainline, still broken. |
301 |
if revision2 is None: |
1185.35.28
by Aaron Bentley
Support diff with two branches as input. |
302 |
if b2 is None: |
1508.1.19
by Robert Collins
Give format3 working trees their own last-revision marker. |
303 |
new_tree = b.bzrdir.open_workingtree() |
1185.35.28
by Aaron Bentley
Support diff with two branches as input. |
304 |
else: |
1508.1.19
by Robert Collins
Give format3 working trees their own last-revision marker. |
305 |
new_tree = b2.bzrdir.open_workingtree() |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
306 |
else: |
1185.67.2
by Aaron Bentley
Renamed Branch.storage to Branch.repository |
307 |
new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id) |
329
by Martin Pool
- refactor command functions into command classes |
308 |
|
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
309 |
return show_diff_trees(old_tree, new_tree, output, specific_files, |
310 |
external_diff_options) |
|
571
by Martin Pool
- new --diff-options to pass options through to external |
311 |
|
312 |
||
1551.2.15
by Aaron Bentley
Rename cmd_show_diff to diff_cmd_helper |
313 |
def diff_cmd_helper(tree, specific_files, external_diff_options, |
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
314 |
old_revision_spec=None, new_revision_spec=None, |
2197.2.1
by Martin Pool
Refactor cmd_diff |
315 |
revision_specs=None, |
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
316 |
old_label='a/', new_label='b/'): |
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
317 |
"""Helper for cmd_diff.
|
318 |
||
2197.2.1
by Martin Pool
Refactor cmd_diff |
319 |
:param tree:
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
320 |
A WorkingTree
|
321 |
||
2197.2.1
by Martin Pool
Refactor cmd_diff |
322 |
:param specific_files:
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
323 |
The specific files to compare, or None
|
324 |
||
2197.2.1
by Martin Pool
Refactor cmd_diff |
325 |
:param external_diff_options:
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
326 |
If non-None, run an external diff, and pass it these options
|
327 |
||
2197.2.1
by Martin Pool
Refactor cmd_diff |
328 |
:param old_revision_spec:
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
329 |
If None, use basis tree as old revision, otherwise use the tree for
|
330 |
the specified revision.
|
|
331 |
||
2197.2.1
by Martin Pool
Refactor cmd_diff |
332 |
:param new_revision_spec:
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
333 |
If None, use working tree as new revision, otherwise use the tree for
|
334 |
the specified revision.
|
|
335 |
|
|
2197.2.1
by Martin Pool
Refactor cmd_diff |
336 |
:param revision_specs:
|
337 |
Zero, one or two RevisionSpecs from the command line, saying what revisions
|
|
338 |
to compare. This can be passed as an alternative to the old_revision_spec
|
|
339 |
and new_revision_spec parameters.
|
|
340 |
||
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
341 |
The more general form is show_diff_trees(), where the caller
|
342 |
supplies any two trees.
|
|
343 |
"""
|
|
2197.2.2
by Martin Pool
merge bzr.dev, reconcile with option changes |
344 |
|
345 |
# TODO: perhaps remove the old parameters old_revision_spec and
|
|
346 |
# new_revision_spec, since this is only really for use from cmd_diff and
|
|
347 |
# it now always passes through a sequence of revision_specs -- mbp
|
|
348 |
# 20061221
|
|
349 |
||
1551.2.14
by Aaron Bentley
Updated argument names, DRY fixes. |
350 |
def spec_tree(spec): |
1732.3.1
by Matthieu Moy
Implementation of -r revno:N:/path/to/branch |
351 |
if tree: |
352 |
revision = spec.in_store(tree.branch) |
|
353 |
else: |
|
354 |
revision = spec.in_store(None) |
|
355 |
revision_id = revision.rev_id |
|
356 |
branch = revision.branch |
|
357 |
return branch.repository.revision_tree(revision_id) |
|
2197.2.1
by Martin Pool
Refactor cmd_diff |
358 |
|
359 |
if revision_specs is not None: |
|
360 |
assert (old_revision_spec is None |
|
361 |
and new_revision_spec is None) |
|
362 |
if len(revision_specs) > 0: |
|
363 |
old_revision_spec = revision_specs[0] |
|
364 |
if len(revision_specs) > 1: |
|
365 |
new_revision_spec = revision_specs[1] |
|
366 |
||
1551.2.14
by Aaron Bentley
Updated argument names, DRY fixes. |
367 |
if old_revision_spec is None: |
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
368 |
old_tree = tree.basis_tree() |
369 |
else: |
|
1551.2.14
by Aaron Bentley
Updated argument names, DRY fixes. |
370 |
old_tree = spec_tree(old_revision_spec) |
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
371 |
|
2197.2.1
by Martin Pool
Refactor cmd_diff |
372 |
if (new_revision_spec is None |
373 |
or new_revision_spec.spec is None): |
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
374 |
new_tree = tree |
375 |
else: |
|
1551.2.14
by Aaron Bentley
Updated argument names, DRY fixes. |
376 |
new_tree = spec_tree(new_revision_spec) |
2197.2.1
by Martin Pool
Refactor cmd_diff |
377 |
|
1551.7.19
by Aaron Bentley
Always include working tree when calculating file ids for diff |
378 |
if new_tree is not tree: |
379 |
extra_trees = (tree,) |
|
380 |
else: |
|
381 |
extra_trees = None |
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
382 |
|
1551.2.14
by Aaron Bentley
Updated argument names, DRY fixes. |
383 |
return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files, |
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
384 |
external_diff_options, |
1551.7.19
by Aaron Bentley
Always include working tree when calculating file ids for diff |
385 |
old_label=old_label, new_label=new_label, |
386 |
extra_trees=extra_trees) |
|
1551.2.13
by Aaron Bentley
Got diff working properly with checkouts |
387 |
|
571
by Martin Pool
- new --diff-options to pass options through to external |
388 |
|
389 |
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None, |
|
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
390 |
external_diff_options=None, |
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
391 |
old_label='a/', new_label='b/', |
392 |
extra_trees=None): |
|
550
by Martin Pool
- Refactor diff code into one that works purely on |
393 |
"""Show in text form the changes from one tree to another.
|
394 |
||
395 |
to_files
|
|
396 |
If set, include only changes to these files.
|
|
571
by Martin Pool
- new --diff-options to pass options through to external |
397 |
|
398 |
external_diff_options
|
|
399 |
If set, use an external GNU diff and pass these options.
|
|
1551.7.18
by Aaron Bentley
Indentation and documentation fixes |
400 |
|
401 |
extra_trees
|
|
402 |
If set, more Trees to use for looking up file ids
|
|
550
by Martin Pool
- Refactor diff code into one that works purely on |
403 |
"""
|
1543.1.1
by Denys Duchier
lock operations for trees - use them for diff |
404 |
old_tree.lock_read() |
405 |
try: |
|
2255.7.38
by John Arbash Meinel
show_diff_trees() should lock any extra trees it is passed. |
406 |
if extra_trees is not None: |
407 |
for tree in extra_trees: |
|
408 |
tree.lock_read() |
|
1543.1.1
by Denys Duchier
lock operations for trees - use them for diff |
409 |
new_tree.lock_read() |
410 |
try: |
|
411 |
return _show_diff_trees(old_tree, new_tree, to_file, |
|
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
412 |
specific_files, external_diff_options, |
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
413 |
old_label=old_label, new_label=new_label, |
414 |
extra_trees=extra_trees) |
|
1543.1.1
by Denys Duchier
lock operations for trees - use them for diff |
415 |
finally: |
416 |
new_tree.unlock() |
|
2255.7.38
by John Arbash Meinel
show_diff_trees() should lock any extra trees it is passed. |
417 |
if extra_trees is not None: |
418 |
for tree in extra_trees: |
|
419 |
tree.unlock() |
|
1543.1.1
by Denys Duchier
lock operations for trees - use them for diff |
420 |
finally: |
421 |
old_tree.unlock() |
|
422 |
||
423 |
||
424 |
def _show_diff_trees(old_tree, new_tree, to_file, |
|
1684.1.6
by Martin Pool
(patch) --diff-prefix option (goffredo, alexander) |
425 |
specific_files, external_diff_options, |
1551.7.17
by Aaron Bentley
Switch to PathsNotVersioned, accept extra_trees |
426 |
old_label='a/', new_label='b/', extra_trees=None): |
329
by Martin Pool
- refactor command functions into command classes |
427 |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
428 |
# GNU Patch uses the epoch date to detect files that are being added
|
429 |
# or removed in a diff.
|
|
430 |
EPOCH_DATE = '1970-01-01 00:00:00 +0000' |
|
1706.1.2
by James Henstridge
Simplify logic a little |
431 |
|
329
by Martin Pool
- refactor command functions into command classes |
432 |
# TODO: Generation of pseudo-diffs for added/deleted files could
|
433 |
# be usefully made into a much faster special case.
|
|
434 |
||
571
by Martin Pool
- new --diff-options to pass options through to external |
435 |
if external_diff_options: |
436 |
assert isinstance(external_diff_options, basestring) |
|
437 |
opts = external_diff_options.split() |
|
438 |
def diff_file(olab, olines, nlab, nlines, to_file): |
|
439 |
external_diff(olab, olines, nlab, nlines, to_file, opts) |
|
440 |
else: |
|
441 |
diff_file = internal_diff |
|
442 |
||
1852.10.3
by Robert Collins
Remove all uses of compare_trees and replace with Tree.changes_from throughout bzrlib. |
443 |
delta = new_tree.changes_from(old_tree, |
444 |
specific_files=specific_files, |
|
445 |
extra_trees=extra_trees, require_versioned=True) |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
446 |
|
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
447 |
has_changes = 0 |
475
by Martin Pool
- rewrite diff using compare_trees() |
448 |
for path, file_id, kind in delta.removed: |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
449 |
has_changes = 1 |
1732.1.7
by John Arbash Meinel
Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. |
450 |
print >>to_file, '=== removed %s %r' % (kind, path.encode('utf8')) |
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
451 |
old_name = '%s%s\t%s' % (old_label, path, |
452 |
_patch_header_date(old_tree, file_id, path)) |
|
453 |
new_name = '%s%s\t%s' % (new_label, path, EPOCH_DATE) |
|
454 |
old_tree.inventory[file_id].diff(diff_file, old_name, old_tree, |
|
455 |
new_name, None, None, to_file) |
|
475
by Martin Pool
- rewrite diff using compare_trees() |
456 |
for path, file_id, kind in delta.added: |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
457 |
has_changes = 1 |
1732.1.7
by John Arbash Meinel
Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. |
458 |
print >>to_file, '=== added %s %r' % (kind, path.encode('utf8')) |
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
459 |
old_name = '%s%s\t%s' % (old_label, path, EPOCH_DATE) |
460 |
new_name = '%s%s\t%s' % (new_label, path, |
|
461 |
_patch_header_date(new_tree, file_id, path)) |
|
462 |
new_tree.inventory[file_id].diff(diff_file, new_name, new_tree, |
|
463 |
old_name, None, None, to_file, |
|
1399.1.4
by Robert Collins
move diff and symlink conditionals into inventory.py from diff.py |
464 |
reverse=True) |
1398
by Robert Collins
integrate in Gustavos x-bit patch |
465 |
for (old_path, new_path, file_id, kind, |
466 |
text_modified, meta_modified) in delta.renamed: |
|
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
467 |
has_changes = 1 |
1398
by Robert Collins
integrate in Gustavos x-bit patch |
468 |
prop_str = get_prop_change(meta_modified) |
469 |
print >>to_file, '=== renamed %s %r => %r%s' % ( |
|
1711.2.28
by John Arbash Meinel
Fix up bzrlib.diff now that compare_trees is returning unicode paths. |
470 |
kind, old_path.encode('utf8'), |
471 |
new_path.encode('utf8'), prop_str) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
472 |
old_name = '%s%s\t%s' % (old_label, old_path, |
473 |
_patch_header_date(old_tree, file_id, |
|
474 |
old_path)) |
|
475 |
new_name = '%s%s\t%s' % (new_label, new_path, |
|
476 |
_patch_header_date(new_tree, file_id, |
|
477 |
new_path)) |
|
478 |
_maybe_diff_file_or_symlink(old_name, old_tree, file_id, |
|
479 |
new_name, new_tree, |
|
1092.2.9
by Robert Collins
bugfix _maybe_diff, the test was not catching the error |
480 |
text_modified, kind, to_file, diff_file) |
1398
by Robert Collins
integrate in Gustavos x-bit patch |
481 |
for path, file_id, kind, text_modified, meta_modified in delta.modified: |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
482 |
has_changes = 1 |
1398
by Robert Collins
integrate in Gustavos x-bit patch |
483 |
prop_str = get_prop_change(meta_modified) |
1732.1.7
by John Arbash Meinel
Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. |
484 |
print >>to_file, '=== modified %s %r%s' % (kind, path.encode('utf8'), prop_str) |
2405.1.3
by John Arbash Meinel
Do a better fix, which recognizes that we should pass the correct old path. |
485 |
# The file may be in a different location in the old tree (because
|
486 |
# the containing dir was renamed, but the file itself was not)
|
|
487 |
old_path = old_tree.id2path(file_id) |
|
488 |
old_name = '%s%s\t%s' % (old_label, old_path, |
|
489 |
_patch_header_date(old_tree, file_id, old_path)) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
490 |
new_name = '%s%s\t%s' % (new_label, path, |
491 |
_patch_header_date(new_tree, file_id, path)) |
|
1398
by Robert Collins
integrate in Gustavos x-bit patch |
492 |
if text_modified: |
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
493 |
_maybe_diff_file_or_symlink(old_name, old_tree, file_id, |
494 |
new_name, new_tree, |
|
1398
by Robert Collins
integrate in Gustavos x-bit patch |
495 |
True, kind, to_file, diff_file) |
1658.1.9
by Martin Pool
Give an error for bzr diff on an nonexistent file (Malone #3619) |
496 |
|
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
497 |
return has_changes |
1658.1.9
by Martin Pool
Give an error for bzr diff on an nonexistent file (Malone #3619) |
498 |
|
499 |
||
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
500 |
def _patch_header_date(tree, file_id, path): |
501 |
"""Returns a timestamp suitable for use in a patch header."""
|
|
2405.1.2
by John Arbash Meinel
Fix bug #103870 by passing None instead of a (sometimes wrong) path |
502 |
mtime = tree.get_file_mtime(file_id, path) |
503 |
assert mtime is not None, \ |
|
504 |
"got an mtime of None for file-id %s, path %s in tree %s" % ( |
|
505 |
file_id, path, tree) |
|
506 |
return timestamp.format_patch_date(mtime) |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
507 |
|
508 |
||
1662.1.9
by Martin Pool
Give a clear error for bzr status of an unversioned, nonexistent file. (Malone #3619) |
509 |
def _raise_if_nonexistent(paths, old_tree, new_tree): |
510 |
"""Complain if paths are not in either inventory or tree.
|
|
511 |
||
512 |
It's OK with the files exist in either tree's inventory, or
|
|
513 |
if they exist in the tree but are not versioned.
|
|
514 |
|
|
515 |
This can be used by operations such as bzr status that can accept
|
|
516 |
unknown or ignored files.
|
|
517 |
"""
|
|
518 |
mutter("check paths: %r", paths) |
|
519 |
if not paths: |
|
520 |
return
|
|
521 |
s = old_tree.filter_unversioned_files(paths) |
|
522 |
s = new_tree.filter_unversioned_files(s) |
|
523 |
s = [path for path in s if not new_tree.has_filename(path)] |
|
524 |
if s: |
|
525 |
raise errors.PathsDoNotExist(sorted(s)) |
|
526 |
||
527 |
||
1398
by Robert Collins
integrate in Gustavos x-bit patch |
528 |
def get_prop_change(meta_modified): |
529 |
if meta_modified: |
|
530 |
return " (properties changed)" |
|
531 |
else: |
|
532 |
return "" |
|
533 |
||
534 |
||
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
535 |
def _maybe_diff_file_or_symlink(old_path, old_tree, file_id, |
536 |
new_path, new_tree, text_modified, |
|
1092.2.9
by Robert Collins
bugfix _maybe_diff, the test was not catching the error |
537 |
kind, to_file, diff_file): |
1092.2.6
by Robert Collins
symlink support updated to work |
538 |
if text_modified: |
1399.1.4
by Robert Collins
move diff and symlink conditionals into inventory.py from diff.py |
539 |
new_entry = new_tree.inventory[file_id] |
540 |
old_tree.inventory[file_id].diff(diff_file, |
|
1740.2.5
by Aaron Bentley
Merge from bzr.dev |
541 |
old_path, old_tree, |
542 |
new_path, new_entry, |
|
1399.1.4
by Robert Collins
move diff and symlink conditionals into inventory.py from diff.py |
543 |
new_tree, to_file) |