~bzr-pqm/bzr/bzr.dev

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