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