bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
| 
4763.2.4
by John Arbash Meinel
 merge bzr.2.1 in preparation for NEWS entry.  | 
1  | 
# Copyright (C) 2005-2010 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
 | 
|
| 
4183.7.1
by Sabin Iacob
 update FSF mailing address  | 
15  | 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 | 
| 
1
by mbp at sourcefrog
 import from baz patch-364  | 
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  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
20  | 
import shutil  | 
| 
4603.1.20
by Aaron Bentley
 Use string.Template substitution with @ as delimiter.  | 
21  | 
import string  | 
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
22  | 
import sys  | 
23  | 
||
24  | 
from bzrlib.lazy_import import lazy_import  | 
|
25  | 
lazy_import(globals(), """  | 
|
26  | 
import errno
 | 
|
| 
1692.8.7
by James Henstridge
 changes suggested by John Meinel  | 
27  | 
import subprocess
 | 
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
28  | 
import tempfile
 | 
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
29  | 
import time
 | 
30  | 
||
| 
1955.2.10
by John Arbash Meinel
 Unset a few other LANG type variables when spawning diff  | 
31  | 
from bzrlib import (
 | 
| 
1551.19.33
by Aaron Bentley
 Use as_revision_id for diff  | 
32  | 
    branch as _mod_branch,
 | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
33  | 
    bzrdir,
 | 
| 
4913.5.24
by Gordon Tyler
 Added cmdline.split function, which replaces commands.shlex_split_unicode.  | 
34  | 
    cmdline,
 | 
| 
1955.2.10
by John Arbash Meinel
 Unset a few other LANG type variables when spawning diff  | 
35  | 
    errors,
 | 
36  | 
    osutils,
 | 
|
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
37  | 
    patiencediff,
 | 
38  | 
    textfile,
 | 
|
| 
1551.12.29
by Aaron Bentley
 Copy and extend patch date formatting code, add patch-date parsing  | 
39  | 
    timestamp,
 | 
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
40  | 
    views,
 | 
| 
1955.2.10
by John Arbash Meinel
 Unset a few other LANG type variables when spawning diff  | 
41  | 
    )
 | 
| 
4845.2.1
by Gary van der Merwe
 When launching an external diff app, don't write temporary files for a working tree.  | 
42  | 
|
43  | 
from bzrlib.workingtree import WorkingTree
 | 
|
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
44  | 
""")  | 
45  | 
||
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
46  | 
from bzrlib.registry import (  | 
47  | 
Registry,  | 
|
48  | 
    )
 | 
|
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
49  | 
from bzrlib.symbol_versioning import (  | 
| 
3948.3.2
by Martin Pool
 Remove APIs deprecated up to and including 1.6  | 
50  | 
deprecated_function,  | 
51  | 
    )
 | 
|
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
52  | 
from bzrlib.trace import mutter, note, warning  | 
| 
1
by mbp at sourcefrog
 import from baz patch-364  | 
53  | 
|
| 
1711.2.24
by John Arbash Meinel
 Late bind to PatienceSequenceMatcher to allow plugin to override.  | 
54  | 
|
| 
4603.1.20
by Aaron Bentley
 Use string.Template substitution with @ as delimiter.  | 
55  | 
class AtTemplate(string.Template):  | 
56  | 
"""Templating class that uses @ instead of $."""  | 
|
57  | 
||
58  | 
delimiter = '@'  | 
|
59  | 
||
60  | 
||
| 
767
by Martin Pool
 - files are only reported as modified if their name or parent has changed,  | 
61  | 
# TODO: Rather than building a changeset object, we should probably
 | 
62  | 
# invoke callbacks on an object.  That object can either accumulate a
 | 
|
63  | 
# list, write them out directly, etc etc.
 | 
|
64  | 
||
| 
2520.4.140
by Aaron Bentley
 Use matching blocks from mpdiff for knit delta creation  | 
65  | 
|
66  | 
class _PrematchedMatcher(difflib.SequenceMatcher):  | 
|
67  | 
"""Allow SequenceMatcher operations to use predetermined blocks"""  | 
|
68  | 
||
69  | 
def __init__(self, matching_blocks):  | 
|
70  | 
difflib.SequenceMatcher(self, None, None)  | 
|
71  | 
self.matching_blocks = matching_blocks  | 
|
72  | 
self.opcodes = None  | 
|
73  | 
||
74  | 
||
| 
1558.15.11
by Aaron Bentley
 Apply merge review suggestions  | 
75  | 
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  | 
76  | 
allow_binary=False, sequence_matcher=None,  | 
77  | 
path_encoding='utf8'):  | 
|
| 
475
by Martin Pool
 - rewrite diff using compare_trees()  | 
78  | 
    # FIXME: difflib is wrong if there is no trailing newline.
 | 
79  | 
    # The syntax used by patch seems to be "\ No newline at
 | 
|
80  | 
    # end of file" following the last diff line from that
 | 
|
81  | 
    # file.  This is not trivial to insert into the
 | 
|
82  | 
    # unified_diff output and it might be better to just fix
 | 
|
83  | 
    # or replace that function.
 | 
|
84  | 
||
85  | 
    # In the meantime we at least make sure the patch isn't
 | 
|
86  | 
    # mangled.
 | 
|
87  | 
||
88  | 
||
89  | 
    # Special workaround for Python2.3, where difflib fails if
 | 
|
90  | 
    # both sequences are empty.
 | 
|
91  | 
if not oldlines and not newlines:  | 
|
92  | 
        return
 | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
93  | 
|
| 
1558.15.11
by Aaron Bentley
 Apply merge review suggestions  | 
94  | 
if allow_binary is False:  | 
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
95  | 
textfile.check_text_lines(oldlines)  | 
96  | 
textfile.check_text_lines(newlines)  | 
|
| 
475
by Martin Pool
 - rewrite diff using compare_trees()  | 
97  | 
|
| 
1185.81.8
by John Arbash Meinel
 Updating unified_diff to take a factory, using the new diff algorithm in the code.  | 
98  | 
if sequence_matcher is None:  | 
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
99  | 
sequence_matcher = patiencediff.PatienceSequenceMatcher  | 
100  | 
ud = patiencediff.unified_diff(oldlines, newlines,  | 
|
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
101  | 
fromfile=old_filename.encode(path_encoding),  | 
102  | 
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.  | 
103  | 
sequencematcher=sequence_matcher)  | 
| 
475
by Martin Pool
 - rewrite diff using compare_trees()  | 
104  | 
|
| 
1092.1.50
by Robert Collins
 make diff lsdiff/filterdiff friendly  | 
105  | 
ud = list(ud)  | 
| 
3085.1.1
by John Arbash Meinel
 Fix internal_diff to not fail when the texts are identical.  | 
106  | 
if len(ud) == 0: # Identical contents, nothing to do  | 
107  | 
        return
 | 
|
| 
475
by Martin Pool
 - rewrite diff using compare_trees()  | 
108  | 
    # work-around for difflib being too smart for its own good
 | 
109  | 
    # if /dev/null is "1,0", patch won't recognize it as /dev/null
 | 
|
110  | 
if not oldlines:  | 
|
111  | 
ud[2] = ud[2].replace('-1,0', '-0,0')  | 
|
112  | 
elif not newlines:  | 
|
113  | 
ud[2] = ud[2].replace('+1,0', '+0,0')  | 
|
114  | 
||
| 
804
by Martin Pool
 Patch from John:  | 
115  | 
for line in ud:  | 
116  | 
to_file.write(line)  | 
|
| 
974.1.5
by Aaron Bentley
 Fixed handling of missing newlines in udiffs  | 
117  | 
if not line.endswith('\n'):  | 
118  | 
to_file.write("\n\\ No newline at end of file\n")  | 
|
| 
2911.6.1
by Blake Winton
 Change 'print >> f,'s to 'f.write('s.  | 
119  | 
to_file.write('\n')  | 
| 
475
by Martin Pool
 - rewrite diff using compare_trees()  | 
120  | 
|
121  | 
||
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
122  | 
def _spawn_external_diff(diffcmd, capture_errors=True):  | 
123  | 
"""Spawn the externall diff process, and return the child handle.  | 
|
124  | 
||
125  | 
    :param diffcmd: The command list to spawn
 | 
|
| 
2138.1.1
by Wouter van Heyst
 Robuster external diff output handling.  | 
126  | 
    :param capture_errors: Capture stderr as well as setting LANG=C
 | 
127  | 
        and LC_ALL=C. This lets us read and understand the output of diff,
 | 
|
128  | 
        and respond to any errors.
 | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
129  | 
    :return: A Popen object.
 | 
130  | 
    """
 | 
|
131  | 
if capture_errors:  | 
|
| 
2321.2.2
by Alexander Belchenko
 win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE)  | 
132  | 
        # construct minimal environment
 | 
133  | 
env = {}  | 
|
134  | 
path = os.environ.get('PATH')  | 
|
135  | 
if path is not None:  | 
|
136  | 
env['PATH'] = path  | 
|
| 
2321.2.5
by Alexander Belchenko
 external diff: no need for special code path for win32 (suggested by John Meinel)  | 
137  | 
env['LANGUAGE'] = 'C' # on win32 only LANGUAGE has effect  | 
138  | 
env['LANG'] = 'C'  | 
|
139  | 
env['LC_ALL'] = 'C'  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
140  | 
stderr = subprocess.PIPE  | 
141  | 
else:  | 
|
| 
2321.2.2
by Alexander Belchenko
 win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE)  | 
142  | 
env = None  | 
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
143  | 
stderr = None  | 
144  | 
||
145  | 
try:  | 
|
146  | 
pipe = subprocess.Popen(diffcmd,  | 
|
147  | 
stdin=subprocess.PIPE,  | 
|
148  | 
stdout=subprocess.PIPE,  | 
|
149  | 
stderr=stderr,  | 
|
| 
2321.2.2
by Alexander Belchenko
 win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE)  | 
150  | 
env=env)  | 
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
151  | 
except OSError, e:  | 
152  | 
if e.errno == errno.ENOENT:  | 
|
153  | 
raise errors.NoDiff(str(e))  | 
|
154  | 
        raise
 | 
|
155  | 
||
156  | 
return pipe  | 
|
157  | 
||
158  | 
||
| 
1185.35.29
by Aaron Bentley
 Support whitespace in diff filenames  | 
159  | 
def external_diff(old_filename, oldlines, new_filename, newlines, to_file,  | 
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
160  | 
diff_opts):  | 
| 
568
by Martin Pool
 - start adding support for showing diffs by calling out to  | 
161  | 
"""Display a diff by calling out to the external diff program."""  | 
| 
581
by Martin Pool
 - make sure any bzr output is flushed before  | 
162  | 
    # make sure our own output is properly ordered before the diff
 | 
163  | 
to_file.flush()  | 
|
164  | 
||
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
165  | 
oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-')  | 
166  | 
newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-')  | 
|
167  | 
oldtmpf = os.fdopen(oldtmp_fd, 'wb')  | 
|
168  | 
newtmpf = os.fdopen(newtmp_fd, 'wb')  | 
|
| 
568
by Martin Pool
 - start adding support for showing diffs by calling out to  | 
169  | 
|
170  | 
try:  | 
|
171  | 
        # TODO: perhaps a special case for comparing to or from the empty
 | 
|
172  | 
        # sequence; can just use /dev/null on Unix
 | 
|
173  | 
||
174  | 
        # TODO: if either of the files being compared already exists as a
 | 
|
175  | 
        # regular named file (e.g. in the working directory) then we can
 | 
|
176  | 
        # compare directly to that, rather than copying it.
 | 
|
177  | 
||
178  | 
oldtmpf.writelines(oldlines)  | 
|
179  | 
newtmpf.writelines(newlines)  | 
|
180  | 
||
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
181  | 
oldtmpf.close()  | 
182  | 
newtmpf.close()  | 
|
| 
568
by Martin Pool
 - start adding support for showing diffs by calling out to  | 
183  | 
|
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
184  | 
if not diff_opts:  | 
185  | 
diff_opts = []  | 
|
| 
4422.1.1
by John Arbash Meinel
 Possibly fix for bug #382709 handling non-ascii external filenames.  | 
186  | 
if sys.platform == 'win32':  | 
187  | 
            # Popen doesn't do the proper encoding for external commands
 | 
|
188  | 
            # Since we are dealing with an ANSI api, use mbcs encoding
 | 
|
189  | 
old_filename = old_filename.encode('mbcs')  | 
|
| 
4422.1.2
by Martin
 Fix copy-and-paste error in previous change  | 
190  | 
new_filename = new_filename.encode('mbcs')  | 
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
191  | 
diffcmd = ['diff',  | 
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
192  | 
'--label', old_filename,  | 
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
193  | 
old_abspath,  | 
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
194  | 
'--label', new_filename,  | 
| 
1711.2.56
by John Arbash Meinel
 Raise NoDiff if 'diff' not present.  | 
195  | 
new_abspath,  | 
196  | 
'--binary',  | 
|
197  | 
                  ]
 | 
|
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
198  | 
|
199  | 
        # diff only allows one style to be specified; they don't override.
 | 
|
200  | 
        # note that some of these take optargs, and the optargs can be
 | 
|
201  | 
        # directly appended to the options.
 | 
|
202  | 
        # this is only an approximate parser; it doesn't properly understand
 | 
|
203  | 
        # the grammar.
 | 
|
204  | 
for s in ['-c', '-u', '-C', '-U',  | 
|
205  | 
'-e', '--ed',  | 
|
206  | 
'-q', '--brief',  | 
|
207  | 
'--normal',  | 
|
208  | 
'-n', '--rcs',  | 
|
209  | 
'-y', '--side-by-side',  | 
|
210  | 
'-D', '--ifdef']:  | 
|
211  | 
for j in diff_opts:  | 
|
212  | 
if j.startswith(s):  | 
|
213  | 
                    break
 | 
|
214  | 
else:  | 
|
215  | 
                continue
 | 
|
216  | 
            break
 | 
|
217  | 
else:  | 
|
218  | 
diffcmd.append('-u')  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
219  | 
|
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
220  | 
if diff_opts:  | 
221  | 
diffcmd.extend(diff_opts)  | 
|
222  | 
||
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
223  | 
pipe = _spawn_external_diff(diffcmd, capture_errors=True)  | 
224  | 
out,err = pipe.communicate()  | 
|
225  | 
rc = pipe.returncode  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
226  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
227  | 
        # internal_diff() adds a trailing newline, add one here for consistency
 | 
228  | 
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  | 
229  | 
if rc == 2:  | 
230  | 
            # 'diff' gives retcode == 2 for all sorts of errors
 | 
|
231  | 
            # one of those is 'Binary files differ'.
 | 
|
232  | 
            # 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.  | 
233  | 
            # '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  | 
234  | 
lang_c_out = out  | 
235  | 
||
236  | 
            # Since we got here, we want to make sure to give an i18n error
 | 
|
237  | 
pipe = _spawn_external_diff(diffcmd, capture_errors=False)  | 
|
238  | 
out, err = pipe.communicate()  | 
|
239  | 
||
240  | 
            # Write out the new i18n diff response
 | 
|
241  | 
to_file.write(out+'\n')  | 
|
242  | 
if pipe.returncode != 2:  | 
|
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
243  | 
raise errors.BzrError(  | 
244  | 
                               'external diff failed with exit code 2'
 | 
|
| 
2138.1.1
by Wouter van Heyst
 Robuster external diff output handling.  | 
245  | 
                               ' when run with LANG=C and LC_ALL=C,'
 | 
246  | 
' 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  | 
247  | 
|
248  | 
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.  | 
249  | 
            # Starting with diffutils 2.8.4 the word "binary" was dropped.
 | 
250  | 
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  | 
251  | 
if m is None:  | 
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
252  | 
raise errors.BzrError('external diff failed with exit code 2;'  | 
253  | 
' command: %r' % (diffcmd,))  | 
|
| 
1920.1.1
by John Arbash Meinel
 fix bug #56307, handle binary files even when LANG is not english  | 
254  | 
else:  | 
255  | 
                # Binary files differ, just return
 | 
|
256  | 
                return
 | 
|
257  | 
||
258  | 
        # If we got to here, we haven't written out the output of diff
 | 
|
259  | 
        # do so now
 | 
|
260  | 
to_file.write(out)  | 
|
261  | 
if rc not in (0, 1):  | 
|
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
262  | 
            # returns 1 if files differ; that's OK
 | 
263  | 
if rc < 0:  | 
|
264  | 
msg = 'signal %d' % (-rc)  | 
|
265  | 
else:  | 
|
266  | 
msg = 'exit code %d' % rc  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
267  | 
|
268  | 
raise errors.BzrError('external diff failed with %s; command: %r'  | 
|
| 
1996.3.9
by John Arbash Meinel
 lazy_import diff.py  | 
269  | 
% (rc, diffcmd))  | 
| 
1899.1.6
by John Arbash Meinel
 internal_diff always adds a trailing \n, make sure external_diff does too  | 
270  | 
|
271  | 
||
| 
568
by Martin Pool
 - start adding support for showing diffs by calling out to  | 
272  | 
finally:  | 
273  | 
oldtmpf.close() # and delete  | 
|
274  | 
newtmpf.close()  | 
|
| 
1711.2.54
by John Arbash Meinel
 Use mkstemp instead of NamedTemporary file for external diff.  | 
275  | 
        # Clean up. Warn in case the files couldn't be deleted
 | 
276  | 
        # (in case windows still holds the file open, but not
 | 
|
277  | 
        # if the files have already been deleted)
 | 
|
278  | 
try:  | 
|
279  | 
os.remove(old_abspath)  | 
|
280  | 
except OSError, e:  | 
|
281  | 
if e.errno not in (errno.ENOENT,):  | 
|
282  | 
warning('Failed to delete temporary file: %s %s',  | 
|
283  | 
old_abspath, e)  | 
|
284  | 
try:  | 
|
285  | 
os.remove(new_abspath)  | 
|
286  | 
except OSError:  | 
|
287  | 
if e.errno not in (errno.ENOENT,):  | 
|
288  | 
warning('Failed to delete temporary file: %s %s',  | 
|
289  | 
new_abspath, e)  | 
|
| 
568
by Martin Pool
 - start adding support for showing diffs by calling out to  | 
290  | 
|
| 
1551.2.13
by Aaron Bentley
 Got diff working properly with checkouts  | 
291  | 
|
| 
4705.1.1
by Gary van der Merwe
 Change _get_trees_to_diff to get_trees_and_branches_to_diff.  | 
292  | 
def get_trees_and_branches_to_diff(path_list, revision_specs, old_url, new_url,  | 
| 
4732.1.2
by Vincent Ladeuil
 (trivial) Fix some PEP8 issues  | 
293  | 
apply_view=True):  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
294  | 
"""Get the trees and specific files to diff given a list of paths.  | 
295  | 
||
296  | 
    This method works out the trees to be diff'ed and the files of
 | 
|
297  | 
    interest within those trees.
 | 
|
298  | 
||
299  | 
    :param path_list:
 | 
|
300  | 
        the list of arguments passed to the diff command
 | 
|
301  | 
    :param revision_specs:
 | 
|
302  | 
        Zero, one or two RevisionSpecs from the diff command line,
 | 
|
303  | 
        saying what revisions to compare.
 | 
|
304  | 
    :param old_url:
 | 
|
305  | 
        The url of the old branch or tree. If None, the tree to use is
 | 
|
306  | 
        taken from the first path, if any, or the current working tree.
 | 
|
307  | 
    :param new_url:
 | 
|
308  | 
        The url of the new branch or tree. If None, the tree to use is
 | 
|
309  | 
        taken from the first path, if any, or the current working tree.
 | 
|
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
310  | 
    :param apply_view:
 | 
311  | 
        if True and a view is set, apply the view or check that the paths
 | 
|
312  | 
        are within it
 | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
313  | 
    :returns:
 | 
| 
4739.3.1
by Jonathan Lange
 Fix the docstring for get_trees_and_branches_to_diff.  | 
314  | 
        a tuple of (old_tree, new_tree, old_branch, new_branch,
 | 
315  | 
        specific_files, extra_trees) where extra_trees is a sequence of
 | 
|
316  | 
        additional trees to search in for file-ids.
 | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
317  | 
    """
 | 
318  | 
    # Get the old and new revision specs
 | 
|
319  | 
old_revision_spec = None  | 
|
320  | 
new_revision_spec = None  | 
|
321  | 
if revision_specs is not None:  | 
|
322  | 
if len(revision_specs) > 0:  | 
|
323  | 
old_revision_spec = revision_specs[0]  | 
|
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
324  | 
if old_url is None:  | 
325  | 
old_url = old_revision_spec.get_branch()  | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
326  | 
if len(revision_specs) > 1:  | 
327  | 
new_revision_spec = revision_specs[1]  | 
|
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
328  | 
if new_url is None:  | 
329  | 
new_url = new_revision_spec.get_branch()  | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
330  | 
|
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
331  | 
other_paths = []  | 
332  | 
make_paths_wt_relative = True  | 
|
| 
3164.1.1
by Ian Clatworthy
 diff without arguments means the current tree, not the current directory  | 
333  | 
consider_relpath = True  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
334  | 
if path_list is None or len(path_list) == 0:  | 
| 
3164.1.1
by Ian Clatworthy
 diff without arguments means the current tree, not the current directory  | 
335  | 
        # If no path is given, the current working tree is used
 | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
336  | 
default_location = u'.'  | 
| 
3164.1.1
by Ian Clatworthy
 diff without arguments means the current tree, not the current directory  | 
337  | 
consider_relpath = False  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
338  | 
elif old_url is not None and new_url is not None:  | 
339  | 
other_paths = path_list  | 
|
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
340  | 
make_paths_wt_relative = False  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
341  | 
else:  | 
342  | 
default_location = path_list[0]  | 
|
343  | 
other_paths = path_list[1:]  | 
|
344  | 
||
345  | 
    # Get the old location
 | 
|
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
346  | 
specific_files = []  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
347  | 
if old_url is None:  | 
348  | 
old_url = default_location  | 
|
349  | 
working_tree, branch, relpath = \  | 
|
350  | 
bzrdir.BzrDir.open_containing_tree_or_branch(old_url)  | 
|
| 
3164.1.1
by Ian Clatworthy
 diff without arguments means the current tree, not the current directory  | 
351  | 
if consider_relpath and relpath != '':  | 
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
352  | 
if working_tree is not None and apply_view:  | 
| 
4032.4.1
by Eduardo Padoan
 Moved diff._check_path_in_view() to views.check_path_in_view()  | 
353  | 
views.check_path_in_view(working_tree, relpath)  | 
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
354  | 
specific_files.append(relpath)  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
355  | 
old_tree = _get_tree_to_diff(old_revision_spec, working_tree, branch)  | 
| 
4705.1.1
by Gary van der Merwe
 Change _get_trees_to_diff to get_trees_and_branches_to_diff.  | 
356  | 
old_branch = branch  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
357  | 
|
358  | 
    # Get the new location
 | 
|
359  | 
if new_url is None:  | 
|
360  | 
new_url = default_location  | 
|
361  | 
if new_url != old_url:  | 
|
362  | 
working_tree, branch, relpath = \  | 
|
363  | 
bzrdir.BzrDir.open_containing_tree_or_branch(new_url)  | 
|
| 
3164.1.1
by Ian Clatworthy
 diff without arguments means the current tree, not the current directory  | 
364  | 
if consider_relpath and relpath != '':  | 
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
365  | 
if working_tree is not None and apply_view:  | 
| 
4032.4.1
by Eduardo Padoan
 Moved diff._check_path_in_view() to views.check_path_in_view()  | 
366  | 
views.check_path_in_view(working_tree, relpath)  | 
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
367  | 
specific_files.append(relpath)  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
368  | 
new_tree = _get_tree_to_diff(new_revision_spec, working_tree, branch,  | 
369  | 
basis_is_default=working_tree is None)  | 
|
| 
4705.1.1
by Gary van der Merwe
 Change _get_trees_to_diff to get_trees_and_branches_to_diff.  | 
370  | 
new_branch = branch  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
371  | 
|
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
372  | 
    # Get the specific files (all files is None, no files is [])
 | 
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
373  | 
if make_paths_wt_relative and working_tree is not None:  | 
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
374  | 
try:  | 
375  | 
from bzrlib.builtins import safe_relpath_files  | 
|
376  | 
other_paths = safe_relpath_files(working_tree, other_paths,  | 
|
377  | 
apply_view=apply_view)  | 
|
378  | 
except errors.FileInWrongBranch:  | 
|
379  | 
raise errors.BzrCommandError("Files are in different branches")  | 
|
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
380  | 
specific_files.extend(other_paths)  | 
381  | 
if len(specific_files) == 0:  | 
|
382  | 
specific_files = None  | 
|
| 
3586.1.21
by Ian Clatworthy
 enhance diff to support views  | 
383  | 
if (working_tree is not None and working_tree.supports_views()  | 
384  | 
and apply_view):  | 
|
385  | 
view_files = working_tree.views.lookup_view()  | 
|
386  | 
if view_files:  | 
|
387  | 
specific_files = view_files  | 
|
388  | 
view_str = views.view_display_str(view_files)  | 
|
| 
4210.1.1
by Ian Clatworthy
 reword 'ignoring files outside view' message  | 
389  | 
note("*** Ignoring files outside view. View is %s" % view_str)  | 
| 
3072.1.2
by Ian Clatworthy
 Test various --old and --new combinations  | 
390  | 
|
391  | 
    # Get extra trees that ought to be searched for file-ids
 | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
392  | 
extra_trees = None  | 
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
393  | 
if working_tree is not None and working_tree not in (old_tree, new_tree):  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
394  | 
extra_trees = (working_tree,)  | 
| 
4705.1.1
by Gary van der Merwe
 Change _get_trees_to_diff to get_trees_and_branches_to_diff.  | 
395  | 
return old_tree, new_tree, old_branch, new_branch, specific_files, extra_trees  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
396  | 
|
| 
4739.3.1
by Jonathan Lange
 Fix the docstring for get_trees_and_branches_to_diff.  | 
397  | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
398  | 
def _get_tree_to_diff(spec, tree=None, branch=None, basis_is_default=True):  | 
399  | 
if branch is None and tree is not None:  | 
|
400  | 
branch = tree.branch  | 
|
401  | 
if spec is None or spec.spec is None:  | 
|
402  | 
if basis_is_default:  | 
|
| 
3072.1.5
by Ian Clatworthy
 more good ideas from abentley  | 
403  | 
if tree is not None:  | 
404  | 
return tree.basis_tree()  | 
|
405  | 
else:  | 
|
406  | 
return branch.basis_tree()  | 
|
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
407  | 
else:  | 
408  | 
return tree  | 
|
| 
3655.3.1
by Lukáš Lalinský
 Fix `bzr st -rbranch:PATH_TO_BRANCH`  | 
409  | 
return spec.as_tree(branch)  | 
| 
3072.1.1
by Ian Clatworthy
 Improved diff based on feedback from abentley  | 
410  | 
|
411  | 
||
| 
571
by Martin Pool
 - new --diff-options to pass options through to external  | 
412  | 
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)  | 
413  | 
external_diff_options=None,  | 
| 
1551.7.17
by Aaron Bentley
 Switch to PathsNotVersioned, accept extra_trees  | 
414  | 
old_label='a/', new_label='b/',  | 
| 
2598.6.12
by ghigo
 Move the encoding of the commit message at the command line level  | 
415  | 
extra_trees=None,  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
416  | 
path_encoding='utf8',  | 
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
417  | 
using=None,  | 
418  | 
format_cls=None):  | 
|
| 
550
by Martin Pool
 - Refactor diff code into one that works purely on  | 
419  | 
"""Show in text form the changes from one tree to another.  | 
420  | 
||
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
421  | 
    :param to_file: The output stream.
 | 
422  | 
    :param specific_files:Include only changes to these files - None for all
 | 
|
423  | 
        changes.
 | 
|
424  | 
    :param external_diff_options: If set, use an external GNU diff and pass 
 | 
|
425  | 
        these options.
 | 
|
426  | 
    :param extra_trees: If set, more Trees to use for looking up file ids
 | 
|
427  | 
    :param path_encoding: If set, the path will be encoded as specified, 
 | 
|
428  | 
        otherwise is supposed to be utf8
 | 
|
429  | 
    :param format_cls: Formatter class (DiffTree subclass)
 | 
|
| 
550
by Martin Pool
 - Refactor diff code into one that works purely on  | 
430  | 
    """
 | 
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
431  | 
if format_cls is None:  | 
432  | 
format_cls = DiffTree  | 
|
| 
1543.1.1
by Denys Duchier
 lock operations for trees - use them for diff  | 
433  | 
old_tree.lock_read()  | 
434  | 
try:  | 
|
| 
2255.7.38
by John Arbash Meinel
 show_diff_trees() should lock any extra trees it is passed.  | 
435  | 
if extra_trees is not None:  | 
436  | 
for tree in extra_trees:  | 
|
437  | 
tree.lock_read()  | 
|
| 
1543.1.1
by Denys Duchier
 lock operations for trees - use them for diff  | 
438  | 
new_tree.lock_read()  | 
439  | 
try:  | 
|
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
440  | 
differ = format_cls.from_trees_options(old_tree, new_tree, to_file,  | 
441  | 
path_encoding,  | 
|
442  | 
external_diff_options,  | 
|
443  | 
old_label, new_label, using)  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
444  | 
return differ.show_diff(specific_files, extra_trees)  | 
| 
1543.1.1
by Denys Duchier
 lock operations for trees - use them for diff  | 
445  | 
finally:  | 
446  | 
new_tree.unlock()  | 
|
| 
2255.7.38
by John Arbash Meinel
 show_diff_trees() should lock any extra trees it is passed.  | 
447  | 
if extra_trees is not None:  | 
448  | 
for tree in extra_trees:  | 
|
449  | 
tree.unlock()  | 
|
| 
1543.1.1
by Denys Duchier
 lock operations for trees - use them for diff  | 
450  | 
finally:  | 
451  | 
old_tree.unlock()  | 
|
452  | 
||
453  | 
||
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
454  | 
def _patch_header_date(tree, file_id, path):  | 
455  | 
"""Returns a timestamp suitable for use in a patch header."""  | 
|
| 
4976.1.3
by Jelmer Vernooij
 Cope with ghosts in 'bzr diff'  | 
456  | 
try:  | 
457  | 
mtime = tree.get_file_mtime(file_id, path)  | 
|
458  | 
except errors.FileTimestampUnavailable:  | 
|
459  | 
mtime = 0  | 
|
| 
2405.1.2
by John Arbash Meinel
 Fix bug #103870 by passing None instead of a (sometimes wrong) path  | 
460  | 
return timestamp.format_patch_date(mtime)  | 
| 
1740.2.5
by Aaron Bentley
 Merge from bzr.dev  | 
461  | 
|
462  | 
||
| 
3268.1.1
by C Miller
 Describe the property changes in diffs. Currently, this is the executable-bit  | 
463  | 
def get_executable_change(old_is_x, new_is_x):  | 
464  | 
descr = { True:"+x", False:"-x", None:"??" }  | 
|
465  | 
if old_is_x != new_is_x:  | 
|
466  | 
return ["%s to %s" % (descr[old_is_x], descr[new_is_x],)]  | 
|
467  | 
else:  | 
|
468  | 
return []  | 
|
469  | 
||
| 
1398
by Robert Collins
 integrate in Gustavos x-bit patch  | 
470  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
471  | 
class DiffPath(object):  | 
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
472  | 
"""Base type for command object that compare files"""  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
473  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
474  | 
    # The type or contents of the file were unsuitable for diffing
 | 
| 
3009.2.29
by Aaron Bentley
 Change constants to strings  | 
475  | 
CANNOT_DIFF = 'CANNOT_DIFF'  | 
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
476  | 
    # The file has changed in a semantic way
 | 
| 
3009.2.29
by Aaron Bentley
 Change constants to strings  | 
477  | 
CHANGED = 'CHANGED'  | 
478  | 
    # The file content may have changed, but there is no semantic change
 | 
|
479  | 
UNCHANGED = 'UNCHANGED'  | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
480  | 
|
| 
3009.2.13
by Aaron Bentley
 Refactor differ to support registering differ factories  | 
481  | 
def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8'):  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
482  | 
"""Constructor.  | 
483  | 
||
484  | 
        :param old_tree: The tree to show as the old tree in the comparison
 | 
|
485  | 
        :param new_tree: The tree to show as new in the comparison
 | 
|
486  | 
        :param to_file: The file to write comparison data to
 | 
|
487  | 
        :param path_encoding: The character encoding to write paths in
 | 
|
488  | 
        """
 | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
489  | 
self.old_tree = old_tree  | 
490  | 
self.new_tree = new_tree  | 
|
491  | 
self.to_file = to_file  | 
|
| 
3009.2.13
by Aaron Bentley
 Refactor differ to support registering differ factories  | 
492  | 
self.path_encoding = path_encoding  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
493  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
494  | 
def finish(self):  | 
495  | 
        pass
 | 
|
496  | 
||
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
497  | 
    @classmethod
 | 
498  | 
def from_diff_tree(klass, diff_tree):  | 
|
499  | 
return klass(diff_tree.old_tree, diff_tree.new_tree,  | 
|
500  | 
diff_tree.to_file, diff_tree.path_encoding)  | 
|
501  | 
||
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
502  | 
    @staticmethod
 | 
503  | 
def _diff_many(differs, file_id, old_path, new_path, old_kind, new_kind):  | 
|
504  | 
for file_differ in differs:  | 
|
505  | 
result = file_differ.diff(file_id, old_path, new_path, old_kind,  | 
|
506  | 
new_kind)  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
507  | 
if result is not DiffPath.CANNOT_DIFF:  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
508  | 
return result  | 
509  | 
else:  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
510  | 
return DiffPath.CANNOT_DIFF  | 
511  | 
||
512  | 
||
513  | 
class DiffKindChange(object):  | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
514  | 
"""Special differ for file kind changes.  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
515  | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
516  | 
    Represents kind change as deletion + creation.  Uses the other differs
 | 
517  | 
    to do this.
 | 
|
518  | 
    """
 | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
519  | 
def __init__(self, differs):  | 
520  | 
self.differs = differs  | 
|
521  | 
||
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
522  | 
def finish(self):  | 
523  | 
        pass
 | 
|
524  | 
||
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
525  | 
    @classmethod
 | 
526  | 
def from_diff_tree(klass, diff_tree):  | 
|
527  | 
return klass(diff_tree.differs)  | 
|
528  | 
||
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
529  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
530  | 
"""Perform comparison  | 
531  | 
||
532  | 
        :param file_id: The file_id of the file to compare
 | 
|
533  | 
        :param old_path: Path of the file in the old tree
 | 
|
534  | 
        :param new_path: Path of the file in the new tree
 | 
|
535  | 
        :param old_kind: Old file-kind of the file
 | 
|
536  | 
        :param new_kind: New file-kind of the file
 | 
|
537  | 
        """
 | 
|
| 
3009.2.18
by Aaron Bentley
 Change KindChangeDiffer's anti-recursion to avoid kind pairs with None  | 
538  | 
if None in (old_kind, new_kind):  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
539  | 
return DiffPath.CANNOT_DIFF  | 
540  | 
result = DiffPath._diff_many(self.differs, file_id, old_path,  | 
|
| 
3009.2.18
by Aaron Bentley
 Change KindChangeDiffer's anti-recursion to avoid kind pairs with None  | 
541  | 
new_path, old_kind, None)  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
542  | 
if result is DiffPath.CANNOT_DIFF:  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
543  | 
return result  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
544  | 
return DiffPath._diff_many(self.differs, file_id, old_path, new_path,  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
545  | 
None, new_kind)  | 
546  | 
||
547  | 
||
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
548  | 
class DiffDirectory(DiffPath):  | 
| 
3009.2.19
by Aaron Bentley
 Implement directory diffing  | 
549  | 
|
550  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
551  | 
"""Perform comparison between two directories. (dummy)  | 
|
552  | 
||
553  | 
        """
 | 
|
554  | 
if 'directory' not in (old_kind, new_kind):  | 
|
555  | 
return self.CANNOT_DIFF  | 
|
556  | 
if old_kind not in ('directory', None):  | 
|
557  | 
return self.CANNOT_DIFF  | 
|
558  | 
if new_kind not in ('directory', None):  | 
|
559  | 
return self.CANNOT_DIFF  | 
|
560  | 
return self.CHANGED  | 
|
561  | 
||
| 
3009.2.20
by Aaron Bentley
 PEP8  | 
562  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
563  | 
class DiffSymlink(DiffPath):  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
564  | 
|
565  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
566  | 
"""Perform comparison between two symlinks  | 
567  | 
||
568  | 
        :param file_id: The file_id of the file to compare
 | 
|
569  | 
        :param old_path: Path of the file in the old tree
 | 
|
570  | 
        :param new_path: Path of the file in the new tree
 | 
|
571  | 
        :param old_kind: Old file-kind of the file
 | 
|
572  | 
        :param new_kind: New file-kind of the file
 | 
|
573  | 
        """
 | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
574  | 
if 'symlink' not in (old_kind, new_kind):  | 
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
575  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
576  | 
if old_kind == 'symlink':  | 
577  | 
old_target = self.old_tree.get_symlink_target(file_id)  | 
|
578  | 
elif old_kind is None:  | 
|
579  | 
old_target = None  | 
|
580  | 
else:  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
581  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
582  | 
if new_kind == 'symlink':  | 
583  | 
new_target = self.new_tree.get_symlink_target(file_id)  | 
|
584  | 
elif new_kind is None:  | 
|
585  | 
new_target = None  | 
|
586  | 
else:  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
587  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
588  | 
return self.diff_symlink(old_target, new_target)  | 
589  | 
||
590  | 
def diff_symlink(self, old_target, new_target):  | 
|
591  | 
if old_target is None:  | 
|
592  | 
self.to_file.write('=== target is %r\n' % new_target)  | 
|
593  | 
elif new_target is None:  | 
|
594  | 
self.to_file.write('=== target was %r\n' % old_target)  | 
|
595  | 
else:  | 
|
596  | 
self.to_file.write('=== target changed %r => %r\n' %  | 
|
597  | 
(old_target, new_target))  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
598  | 
return self.CHANGED  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
599  | 
|
600  | 
||
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
601  | 
class DiffText(DiffPath):  | 
| 
3009.2.2
by Aaron Bentley
 Implement Differ object for abstracting diffing  | 
602  | 
|
| 
3009.2.7
by Aaron Bentley
 Move responsibility for generating diff labels into Differ.diff  | 
603  | 
    # GNU Patch uses the epoch date to detect files that are being added
 | 
604  | 
    # or removed in a diff.
 | 
|
605  | 
EPOCH_DATE = '1970-01-01 00:00:00 +0000'  | 
|
606  | 
||
| 
3009.2.13
by Aaron Bentley
 Refactor differ to support registering differ factories  | 
607  | 
def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8',  | 
608  | 
old_label='', new_label='', text_differ=internal_diff):  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
609  | 
DiffPath.__init__(self, old_tree, new_tree, to_file, path_encoding)  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
610  | 
self.text_differ = text_differ  | 
611  | 
self.old_label = old_label  | 
|
612  | 
self.new_label = new_label  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
613  | 
self.path_encoding = path_encoding  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
614  | 
|
615  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
616  | 
"""Compare two files in unified diff format  | 
617  | 
||
618  | 
        :param file_id: The file_id of the file to compare
 | 
|
619  | 
        :param old_path: Path of the file in the old tree
 | 
|
620  | 
        :param new_path: Path of the file in the new tree
 | 
|
621  | 
        :param old_kind: Old file-kind of the file
 | 
|
622  | 
        :param new_kind: New file-kind of the file
 | 
|
623  | 
        """
 | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
624  | 
if 'file' not in (old_kind, new_kind):  | 
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
625  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
626  | 
from_file_id = to_file_id = file_id  | 
627  | 
if old_kind == 'file':  | 
|
628  | 
old_date = _patch_header_date(self.old_tree, file_id, old_path)  | 
|
629  | 
elif old_kind is None:  | 
|
630  | 
old_date = self.EPOCH_DATE  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
631  | 
from_file_id = None  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
632  | 
else:  | 
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
633  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
634  | 
if new_kind == 'file':  | 
635  | 
new_date = _patch_header_date(self.new_tree, file_id, new_path)  | 
|
636  | 
elif new_kind is None:  | 
|
637  | 
new_date = self.EPOCH_DATE  | 
|
638  | 
to_file_id = None  | 
|
639  | 
else:  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
640  | 
return self.CANNOT_DIFF  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
641  | 
from_label = '%s%s\t%s' % (self.old_label, old_path, old_date)  | 
642  | 
to_label = '%s%s\t%s' % (self.new_label, new_path, new_date)  | 
|
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
643  | 
return self.diff_text(from_file_id, to_file_id, from_label, to_label,  | 
644  | 
old_path, new_path)  | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
645  | 
|
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
646  | 
def diff_text(self, from_file_id, to_file_id, from_label, to_label,  | 
647  | 
from_path=None, to_path=None):  | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
648  | 
"""Diff the content of given files in two trees  | 
649  | 
||
650  | 
        :param from_file_id: The id of the file in the from tree.  If None,
 | 
|
651  | 
            the file is not present in the from tree.
 | 
|
652  | 
        :param to_file_id: The id of the file in the to tree.  This may refer
 | 
|
653  | 
            to a different file from from_file_id.  If None,
 | 
|
654  | 
            the file is not present in the to tree.
 | 
|
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
655  | 
        :param from_path: The path in the from tree or None if unknown.
 | 
656  | 
        :param to_path: The path in the to tree or None if unknown.
 | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
657  | 
        """
 | 
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
658  | 
def _get_text(tree, file_id, path):  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
659  | 
if file_id is not None:  | 
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
660  | 
return tree.get_file(file_id, path).readlines()  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
661  | 
else:  | 
662  | 
return []  | 
|
663  | 
try:  | 
|
| 
4377.3.3
by Ian Clatworthy
 avoid unnecessary id2path calculation when diffing  | 
664  | 
from_text = _get_text(self.old_tree, from_file_id, from_path)  | 
665  | 
to_text = _get_text(self.new_tree, to_file_id, to_path)  | 
|
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
666  | 
self.text_differ(from_label, from_text, to_label, to_text,  | 
667  | 
self.to_file)  | 
|
668  | 
except errors.BinaryFile:  | 
|
669  | 
self.to_file.write(  | 
|
670  | 
("Binary files %s and %s differ\n" %  | 
|
671  | 
(from_label, to_label)).encode(self.path_encoding))  | 
|
| 
3009.2.14
by Aaron Bentley
 Update return type handling  | 
672  | 
return self.CHANGED  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
673  | 
|
674  | 
||
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
675  | 
class DiffFromTool(DiffPath):  | 
676  | 
||
677  | 
def __init__(self, command_template, old_tree, new_tree, to_file,  | 
|
678  | 
path_encoding='utf-8'):  | 
|
679  | 
DiffPath.__init__(self, old_tree, new_tree, to_file, path_encoding)  | 
|
680  | 
self.command_template = command_template  | 
|
| 
3638.3.2
by Vincent Ladeuil
 Fix all calls to tempfile.mkdtemp to osutils.mkdtemp.  | 
681  | 
self._root = osutils.mkdtemp(prefix='bzr-diff-')  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
682  | 
|
683  | 
    @classmethod
 | 
|
684  | 
def from_string(klass, command_string, old_tree, new_tree, to_file,  | 
|
685  | 
path_encoding='utf-8'):  | 
|
| 
4913.5.24
by Gordon Tyler
 Added cmdline.split function, which replaces commands.shlex_split_unicode.  | 
686  | 
command_template = cmdline.split(command_string)  | 
| 
4603.1.20
by Aaron Bentley
 Use string.Template substitution with @ as delimiter.  | 
687  | 
if '@' not in command_string:  | 
688  | 
command_template.extend(['@old_path', '@new_path'])  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
689  | 
return klass(command_template, old_tree, new_tree, to_file,  | 
690  | 
path_encoding)  | 
|
691  | 
||
692  | 
    @classmethod
 | 
|
693  | 
def make_from_diff_tree(klass, command_string):  | 
|
694  | 
def from_diff_tree(diff_tree):  | 
|
695  | 
return klass.from_string(command_string, diff_tree.old_tree,  | 
|
696  | 
diff_tree.new_tree, diff_tree.to_file)  | 
|
697  | 
return from_diff_tree  | 
|
698  | 
||
699  | 
def _get_command(self, old_path, new_path):  | 
|
700  | 
my_map = {'old_path': old_path, 'new_path': new_path}  | 
|
| 
4603.1.20
by Aaron Bentley
 Use string.Template substitution with @ as delimiter.  | 
701  | 
return [AtTemplate(t).substitute(my_map) for t in  | 
702  | 
self.command_template]  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
703  | 
|
704  | 
def _execute(self, old_path, new_path):  | 
|
| 
3145.1.1
by Aaron Bentley
 Handle missing tools gracefully in diff --using  | 
705  | 
command = self._get_command(old_path, new_path)  | 
706  | 
try:  | 
|
707  | 
proc = subprocess.Popen(command, stdout=subprocess.PIPE,  | 
|
708  | 
cwd=self._root)  | 
|
709  | 
except OSError, e:  | 
|
710  | 
if e.errno == errno.ENOENT:  | 
|
711  | 
raise errors.ExecutableMissing(command[0])  | 
|
| 
3145.1.2
by Aaron Bentley
 Don't swallow other OSErrors  | 
712  | 
else:  | 
713  | 
                raise
 | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
714  | 
self.to_file.write(proc.stdout.read())  | 
715  | 
return proc.wait()  | 
|
716  | 
||
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
717  | 
def _try_symlink_root(self, tree, prefix):  | 
| 
3287.18.3
by Matt McClure
 Toward a more acceptable patch for bug 209281.  | 
718  | 
if (getattr(tree, 'abspath', None) is None  | 
| 
3287.18.14
by Matt McClure
 Extracted a host_os_dereferences_symlinks method.  | 
719  | 
or not osutils.host_os_dereferences_symlinks()):  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
720  | 
return False  | 
721  | 
try:  | 
|
722  | 
os.symlink(tree.abspath(''), osutils.pathjoin(self._root, prefix))  | 
|
723  | 
except OSError, e:  | 
|
724  | 
if e.errno != errno.EEXIST:  | 
|
725  | 
                raise
 | 
|
726  | 
return True  | 
|
727  | 
||
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
728  | 
def _write_file(self, file_id, tree, prefix, relpath, force_temp=False,  | 
729  | 
allow_write=False):  | 
|
| 
4845.2.3
by Gary van der Merwe
 Still write temp file for working tree is force_temp is true. (Suggestion from jam.)  | 
730  | 
if not force_temp and isinstance(tree, WorkingTree):  | 
| 
4845.2.1
by Gary van der Merwe
 When launching an external diff app, don't write temporary files for a working tree.  | 
731  | 
return tree.abspath(tree.id2path(file_id))  | 
732  | 
||
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
733  | 
full_path = osutils.pathjoin(self._root, prefix, relpath)  | 
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
734  | 
if not force_temp and self._try_symlink_root(tree, prefix):  | 
| 
3123.6.5
by Aaron Bentley
 Symlink to real files if possible  | 
735  | 
return full_path  | 
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
736  | 
parent_dir = osutils.dirname(full_path)  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
737  | 
try:  | 
738  | 
os.makedirs(parent_dir)  | 
|
739  | 
except OSError, e:  | 
|
740  | 
if e.errno != errno.EEXIST:  | 
|
741  | 
                raise
 | 
|
| 
3123.6.6
by Aaron Bentley
 Use relpath for get_file  | 
742  | 
source = tree.get_file(file_id, relpath)  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
743  | 
try:  | 
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
744  | 
target = open(full_path, 'wb')  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
745  | 
try:  | 
746  | 
osutils.pumpfile(source, target)  | 
|
747  | 
finally:  | 
|
748  | 
target.close()  | 
|
749  | 
finally:  | 
|
750  | 
source.close()  | 
|
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
751  | 
if not allow_write:  | 
| 
4603.1.1
by Aaron Bentley
 Initial pass at shelve-via-editor.  | 
752  | 
osutils.make_readonly(full_path)  | 
| 
4976.1.3
by Jelmer Vernooij
 Cope with ghosts in 'bzr diff'  | 
753  | 
try:  | 
754  | 
mtime = tree.get_file_mtime(file_id)  | 
|
755  | 
except errors.FileTimestampUnavailable:  | 
|
756  | 
mtime = 0  | 
|
| 
3123.6.4
by Aaron Bentley
 Set mtime (and atime) on files for --using  | 
757  | 
os.utime(full_path, (mtime, mtime))  | 
758  | 
return full_path  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
759  | 
|
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
760  | 
def _prepare_files(self, file_id, old_path, new_path, force_temp=False,  | 
761  | 
allow_write_new=False):  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
762  | 
old_disk_path = self._write_file(file_id, self.old_tree, 'old',  | 
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
763  | 
old_path, force_temp)  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
764  | 
new_disk_path = self._write_file(file_id, self.new_tree, 'new',  | 
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
765  | 
new_path, force_temp,  | 
766  | 
allow_write=allow_write_new)  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
767  | 
return old_disk_path, new_disk_path  | 
768  | 
||
769  | 
def finish(self):  | 
|
| 
4354.6.1
by Martitza Mendez
 Fix 363837 : catch OSError from osutils.rmtree and mutter to trace file.  | 
770  | 
try:  | 
771  | 
osutils.rmtree(self._root)  | 
|
772  | 
except OSError, e:  | 
|
773  | 
if e.errno != errno.ENOENT:  | 
|
| 
4399.1.1
by Ian Clatworthy
 (igc) address temp file issue with diff --using on Windows (Martitza Mendez)  | 
774  | 
mutter("The temporary directory \"%s\" was not "  | 
775  | 
"cleanly removed: %s." % (self._root, e))  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
776  | 
|
777  | 
def diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
778  | 
if (old_kind, new_kind) != ('file', 'file'):  | 
|
779  | 
return DiffPath.CANNOT_DIFF  | 
|
| 
4845.2.1
by Gary van der Merwe
 When launching an external diff app, don't write temporary files for a working tree.  | 
780  | 
(old_disk_path, new_disk_path) = self._prepare_files(  | 
781  | 
file_id, old_path, new_path)  | 
|
782  | 
self._execute(old_disk_path, new_disk_path)  | 
|
| 
4603.1.1
by Aaron Bentley
 Initial pass at shelve-via-editor.  | 
783  | 
|
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
784  | 
def edit_file(self, file_id):  | 
785  | 
"""Use this tool to edit a file.  | 
|
786  | 
||
787  | 
        A temporary copy will be edited, and the new contents will be
 | 
|
788  | 
        returned.
 | 
|
789  | 
||
790  | 
        :param file_id: The id of the file to edit.
 | 
|
791  | 
        :return: The new contents of the file.
 | 
|
792  | 
        """
 | 
|
793  | 
old_path = self.old_tree.id2path(file_id)  | 
|
794  | 
new_path = self.new_tree.id2path(file_id)  | 
|
795  | 
new_abs_path = self._prepare_files(file_id, old_path, new_path,  | 
|
796  | 
allow_write_new=True,  | 
|
797  | 
force_temp=True)[1]  | 
|
| 
4603.3.1
by Benoît Pierre
 Fix edit_diff when using a console command.  | 
798  | 
command = self._get_command(osutils.pathjoin('old', old_path),  | 
799  | 
osutils.pathjoin('new', new_path))  | 
|
| 
4603.1.24
by Aaron Bentley
 Fix call import/invocation.  | 
800  | 
subprocess.call(command, cwd=self._root)  | 
| 
4603.1.4
by Aaron Bentley
 Implement DiffFromTool.edit_file  | 
801  | 
new_file = open(new_abs_path, 'r')  | 
802  | 
try:  | 
|
803  | 
return new_file.read()  | 
|
804  | 
finally:  | 
|
805  | 
new_file.close()  | 
|
806  | 
||
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
807  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
808  | 
class DiffTree(object):  | 
809  | 
"""Provides textual representations of the difference between two trees.  | 
|
810  | 
||
811  | 
    A DiffTree examines two trees and where a file-id has altered
 | 
|
812  | 
    between them, generates a textual representation of the difference.
 | 
|
813  | 
    DiffTree uses a sequence of DiffPath objects which are each
 | 
|
814  | 
    given the opportunity to handle a given altered fileid. The list
 | 
|
815  | 
    of DiffPath objects can be extended globally by appending to
 | 
|
816  | 
    DiffTree.diff_factories, or for a specific diff operation by
 | 
|
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
817  | 
    supplying the extra_factories option to the appropriate method.
 | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
818  | 
    """
 | 
819  | 
||
820  | 
    # list of factories that can provide instances of DiffPath objects
 | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
821  | 
    # may be extended by plugins.
 | 
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
822  | 
diff_factories = [DiffSymlink.from_diff_tree,  | 
823  | 
DiffDirectory.from_diff_tree]  | 
|
| 
3009.2.13
by Aaron Bentley
 Refactor differ to support registering differ factories  | 
824  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
825  | 
def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8',  | 
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
826  | 
diff_text=None, extra_factories=None):  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
827  | 
"""Constructor  | 
828  | 
||
829  | 
        :param old_tree: Tree to show as old in the comparison
 | 
|
830  | 
        :param new_tree: Tree to show as new in the comparison
 | 
|
831  | 
        :param to_file: File to write comparision to
 | 
|
832  | 
        :param path_encoding: Character encoding to write paths in
 | 
|
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
833  | 
        :param diff_text: DiffPath-type object to use as a last resort for
 | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
834  | 
            diffing text files.
 | 
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
835  | 
        :param extra_factories: Factories of DiffPaths to try before any other
 | 
836  | 
            DiffPaths"""
 | 
|
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
837  | 
if diff_text is None:  | 
838  | 
diff_text = DiffText(old_tree, new_tree, to_file, path_encoding,  | 
|
839  | 
'', '', internal_diff)  | 
|
| 
3009.2.4
by Aaron Bentley
 Make old_tree/new_tree construction parameters of Differ  | 
840  | 
self.old_tree = old_tree  | 
841  | 
self.new_tree = new_tree  | 
|
| 
3009.2.2
by Aaron Bentley
 Implement Differ object for abstracting diffing  | 
842  | 
self.to_file = to_file  | 
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
843  | 
self.path_encoding = path_encoding  | 
| 
3009.2.13
by Aaron Bentley
 Refactor differ to support registering differ factories  | 
844  | 
self.differs = []  | 
| 
3009.2.27
by Aaron Bentley
 Use extra_factories instead of extra_diffs  | 
845  | 
if extra_factories is not None:  | 
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
846  | 
self.differs.extend(f(self) for f in extra_factories)  | 
847  | 
self.differs.extend(f(self) for f in self.diff_factories)  | 
|
848  | 
self.differs.extend([diff_text, DiffKindChange.from_diff_tree(self)])  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
849  | 
|
850  | 
    @classmethod
 | 
|
851  | 
def from_trees_options(klass, old_tree, new_tree, to_file,  | 
|
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
852  | 
path_encoding, external_diff_options, old_label,  | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
853  | 
new_label, using):  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
854  | 
"""Factory for producing a DiffTree.  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
855  | 
|
856  | 
        Designed to accept options used by show_diff_trees.
 | 
|
857  | 
        :param old_tree: The tree to show as old in the comparison
 | 
|
858  | 
        :param new_tree: The tree to show as new in the comparison
 | 
|
859  | 
        :param to_file: File to write comparisons to
 | 
|
860  | 
        :param path_encoding: Character encoding to use for writing paths
 | 
|
861  | 
        :param external_diff_options: If supplied, use the installed diff
 | 
|
862  | 
            binary to perform file comparison, using supplied options.
 | 
|
863  | 
        :param old_label: Prefix to use for old file labels
 | 
|
864  | 
        :param new_label: Prefix to use for new file labels
 | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
865  | 
        :param using: Commandline to use to invoke an external diff tool
 | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
866  | 
        """
 | 
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
867  | 
if using is not None:  | 
868  | 
extra_factories = [DiffFromTool.make_from_diff_tree(using)]  | 
|
869  | 
else:  | 
|
870  | 
extra_factories = []  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
871  | 
if external_diff_options:  | 
872  | 
opts = external_diff_options.split()  | 
|
873  | 
def diff_file(olab, olines, nlab, nlines, to_file):  | 
|
874  | 
external_diff(olab, olines, nlab, nlines, to_file, opts)  | 
|
875  | 
else:  | 
|
876  | 
diff_file = internal_diff  | 
|
| 
3009.2.28
by Aaron Bentley
 Add from_diff_tree factories  | 
877  | 
diff_text = DiffText(old_tree, new_tree, to_file, path_encoding,  | 
878  | 
old_label, new_label, diff_file)  | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
879  | 
return klass(old_tree, new_tree, to_file, path_encoding, diff_text,  | 
880  | 
extra_factories)  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
881  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
882  | 
def show_diff(self, specific_files, extra_trees=None):  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
883  | 
"""Write tree diff to self.to_file  | 
884  | 
||
| 
5131.1.4
by Jelmer Vernooij
 Add test for custom diff format.  | 
885  | 
        :param specific_files: the specific files to compare (recursive)
 | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
886  | 
        :param extra_trees: extra trees to use for mapping paths to file_ids
 | 
887  | 
        """
 | 
|
| 
3123.6.2
by Aaron Bentley
 Implement diff --using natively  | 
888  | 
try:  | 
889  | 
return self._show_diff(specific_files, extra_trees)  | 
|
890  | 
finally:  | 
|
891  | 
for differ in self.differs:  | 
|
892  | 
differ.finish()  | 
|
893  | 
||
894  | 
def _show_diff(self, specific_files, extra_trees):  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
895  | 
        # TODO: Generation of pseudo-diffs for added/deleted files could
 | 
896  | 
        # be usefully made into a much faster special case.
 | 
|
| 
3254.1.1
by Aaron Bentley
 Make Tree.iter_changes a public method  | 
897  | 
iterator = self.new_tree.iter_changes(self.old_tree,  | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
898  | 
specific_files=specific_files,  | 
899  | 
extra_trees=extra_trees,  | 
|
900  | 
require_versioned=True)  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
901  | 
has_changes = 0  | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
902  | 
def changes_key(change):  | 
903  | 
old_path, new_path = change[1]  | 
|
904  | 
path = new_path  | 
|
905  | 
if path is None:  | 
|
906  | 
path = old_path  | 
|
907  | 
return path  | 
|
908  | 
def get_encoded_path(path):  | 
|
909  | 
if path is not None:  | 
|
910  | 
return path.encode(self.path_encoding, "replace")  | 
|
911  | 
for (file_id, paths, changed_content, versioned, parent, name, kind,  | 
|
912  | 
executable) in sorted(iterator, key=changes_key):  | 
|
| 
3619.4.2
by Robert Collins
 Change bzrlib.diff.DiffTree.show_diff to skip entries missing in both trees.  | 
913  | 
            # The root does not get diffed, and items with no known kind (that
 | 
914  | 
            # is, missing) in both trees are skipped as well.
 | 
|
915  | 
if parent == (None, None) or kind == (None, None):  | 
|
| 
3123.4.3
by Aaron Bentley
 Tweak path handling  | 
916  | 
                continue
 | 
917  | 
oldpath, newpath = paths  | 
|
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
918  | 
oldpath_encoded = get_encoded_path(paths[0])  | 
919  | 
newpath_encoded = get_encoded_path(paths[1])  | 
|
920  | 
old_present = (kind[0] is not None and versioned[0])  | 
|
921  | 
new_present = (kind[1] is not None and versioned[1])  | 
|
922  | 
renamed = (parent[0], name[0]) != (parent[1], name[1])  | 
|
| 
3268.1.1
by C Miller
 Describe the property changes in diffs. Currently, this is the executable-bit  | 
923  | 
|
924  | 
properties_changed = []  | 
|
925  | 
properties_changed.extend(get_executable_change(executable[0], executable[1]))  | 
|
926  | 
||
927  | 
if properties_changed:  | 
|
928  | 
prop_str = " (properties changed: %s)" % (", ".join(properties_changed),)  | 
|
929  | 
else:  | 
|
930  | 
prop_str = ""  | 
|
931  | 
||
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
932  | 
if (old_present, new_present) == (True, False):  | 
933  | 
self.to_file.write("=== removed %s '%s'\n" %  | 
|
934  | 
(kind[0], oldpath_encoded))  | 
|
| 
3123.4.3
by Aaron Bentley
 Tweak path handling  | 
935  | 
newpath = oldpath  | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
936  | 
elif (old_present, new_present) == (False, True):  | 
937  | 
self.to_file.write("=== added %s '%s'\n" %  | 
|
938  | 
(kind[1], newpath_encoded))  | 
|
| 
3123.4.3
by Aaron Bentley
 Tweak path handling  | 
939  | 
oldpath = newpath  | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
940  | 
elif renamed:  | 
941  | 
self.to_file.write("=== renamed %s '%s' => '%s'%s\n" %  | 
|
942  | 
(kind[0], oldpath_encoded, newpath_encoded, prop_str))  | 
|
| 
3123.4.2
by Aaron Bentley
 Handle diff with property change correctly  | 
943  | 
else:  | 
| 
3254.1.1
by Aaron Bentley
 Make Tree.iter_changes a public method  | 
944  | 
                # if it was produced by iter_changes, it must be
 | 
| 
3123.4.2
by Aaron Bentley
 Handle diff with property change correctly  | 
945  | 
                # modified *somehow*, either content or execute bit.
 | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
946  | 
self.to_file.write("=== modified %s '%s'%s\n" % (kind[0],  | 
947  | 
newpath_encoded, prop_str))  | 
|
948  | 
if changed_content:  | 
|
| 
4377.3.1
by Ian Clatworthy
 faster diff on large trees  | 
949  | 
self._diff(file_id, oldpath, newpath, kind[0], kind[1])  | 
| 
3123.4.1
by Aaron Bentley
 Diff sorts files in alphabetical order  | 
950  | 
has_changes = 1  | 
951  | 
if renamed:  | 
|
952  | 
has_changes = 1  | 
|
| 
3009.2.6
by Aaron Bentley
 Convert show_diff_trees into a Differ method  | 
953  | 
return has_changes  | 
| 
3009.2.2
by Aaron Bentley
 Implement Differ object for abstracting diffing  | 
954  | 
|
| 
3009.2.12
by Aaron Bentley
 Associate labels with text diffing only  | 
955  | 
def diff(self, file_id, old_path, new_path):  | 
| 
3009.2.17
by Aaron Bentley
 Update docs  | 
956  | 
"""Perform a diff of a single file  | 
957  | 
||
958  | 
        :param file_id: file-id of the file
 | 
|
959  | 
        :param old_path: The path of the file in the old tree
 | 
|
960  | 
        :param new_path: The path of the file in the new tree
 | 
|
961  | 
        """
 | 
|
| 
3009.2.2
by Aaron Bentley
 Implement Differ object for abstracting diffing  | 
962  | 
try:  | 
| 
3009.2.8
by Aaron Bentley
 Support diffing without indirecting through inventory entries  | 
963  | 
old_kind = self.old_tree.kind(file_id)  | 
| 
3087.1.1
by Aaron Bentley
 Diff handles missing files correctly, with no tracebacks  | 
964  | 
except (errors.NoSuchId, errors.NoSuchFile):  | 
| 
3009.2.8
by Aaron Bentley
 Support diffing without indirecting through inventory entries  | 
965  | 
old_kind = None  | 
| 
3009.2.3
by Aaron Bentley
 Detect missing files from inv operation  | 
966  | 
try:  | 
| 
3009.2.8
by Aaron Bentley
 Support diffing without indirecting through inventory entries  | 
967  | 
new_kind = self.new_tree.kind(file_id)  | 
| 
3087.1.1
by Aaron Bentley
 Diff handles missing files correctly, with no tracebacks  | 
968  | 
except (errors.NoSuchId, errors.NoSuchFile):  | 
| 
3009.2.8
by Aaron Bentley
 Support diffing without indirecting through inventory entries  | 
969  | 
new_kind = None  | 
| 
4377.3.1
by Ian Clatworthy
 faster diff on large trees  | 
970  | 
self._diff(file_id, old_path, new_path, old_kind, new_kind)  | 
971  | 
||
972  | 
||
973  | 
def _diff(self, file_id, old_path, new_path, old_kind, new_kind):  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
974  | 
result = DiffPath._diff_many(self.differs, file_id, old_path,  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
975  | 
new_path, old_kind, new_kind)  | 
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
976  | 
if result is DiffPath.CANNOT_DIFF:  | 
| 
3009.2.11
by Aaron Bentley
 Refactor diff to be more pluggable  | 
977  | 
error_path = new_path  | 
978  | 
if error_path is None:  | 
|
979  | 
error_path = old_path  | 
|
| 
3009.2.22
by Aaron Bentley
 Update names & docstring  | 
980  | 
raise errors.NoDiffFound(error_path)  | 
| 
5131.1.1
by Jelmer Vernooij
 Add --format option to 'bzr diff'.  | 
981  | 
|
982  | 
||
983  | 
format_registry = Registry()  | 
|
984  | 
format_registry.register('default', DiffTree)  |