/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/diff.py

  • Committer: John Arbash Meinel
  • Date: 2008-04-30 15:02:49 UTC
  • mto: This revision was merged to the branch mainline in revision 3394.
  • Revision ID: john@arbash-meinel.com-20080430150249-viu4vgzglhfu7k7y
Update some tests to pass with the branch checks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/diff.py
1
1
# Copyright (C) 2004, 2005, 2006 Canonical Ltd.
2
 
 
 
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
 
 
7
#
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
12
 
 
 
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
 
from bzrlib.delta import compare_trees
18
 
from bzrlib.errors import BzrError
19
 
import bzrlib.errors as errors
20
 
from bzrlib.symbol_versioning import *
21
 
from bzrlib.trace import mutter
 
17
import difflib
 
18
import os
 
19
import re
 
20
import shutil
 
21
import sys
 
22
 
 
23
from bzrlib.lazy_import import lazy_import
 
24
lazy_import(globals(), """
 
25
import errno
 
26
import subprocess
 
27
import tempfile
 
28
import time
 
29
 
 
30
from bzrlib import (
 
31
    branch as _mod_branch,
 
32
    bzrdir,
 
33
    commands,
 
34
    errors,
 
35
    osutils,
 
36
    patiencediff,
 
37
    textfile,
 
38
    timestamp,
 
39
    )
 
40
""")
 
41
 
 
42
from bzrlib.symbol_versioning import (
 
43
        deprecated_function,
 
44
        one_zero,
 
45
        one_three
 
46
        )
 
47
from bzrlib.trace import mutter, warning
 
48
 
22
49
 
23
50
# TODO: Rather than building a changeset object, we should probably
24
51
# invoke callbacks on an object.  That object can either accumulate a
25
52
# list, write them out directly, etc etc.
26
53
 
27
 
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file):
28
 
    import difflib
29
 
    
 
54
 
 
55
class _PrematchedMatcher(difflib.SequenceMatcher):
 
56
    """Allow SequenceMatcher operations to use predetermined blocks"""
 
57
 
 
58
    def __init__(self, matching_blocks):
 
59
        difflib.SequenceMatcher(self, None, None)
 
60
        self.matching_blocks = matching_blocks
 
61
        self.opcodes = None
 
62
 
 
63
 
 
64
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file,
 
65
                  allow_binary=False, sequence_matcher=None,
 
66
                  path_encoding='utf8'):
30
67
    # FIXME: difflib is wrong if there is no trailing newline.
31
68
    # The syntax used by patch seems to be "\ No newline at
32
69
    # end of file" following the last diff line from that
42
79
    # both sequences are empty.
43
80
    if not oldlines and not newlines:
44
81
        return
 
82
    
 
83
    if allow_binary is False:
 
84
        textfile.check_text_lines(oldlines)
 
85
        textfile.check_text_lines(newlines)
45
86
 
46
 
    ud = difflib.unified_diff(oldlines, newlines,
47
 
                              fromfile=old_filename+'\t', 
48
 
                              tofile=new_filename+'\t')
 
87
    if sequence_matcher is None:
 
88
        sequence_matcher = patiencediff.PatienceSequenceMatcher
 
89
    ud = patiencediff.unified_diff(oldlines, newlines,
 
90
                      fromfile=old_filename.encode(path_encoding),
 
91
                      tofile=new_filename.encode(path_encoding),
 
92
                      sequencematcher=sequence_matcher)
49
93
 
50
94
    ud = list(ud)
 
95
    if len(ud) == 0: # Identical contents, nothing to do
 
96
        return
51
97
    # work-around for difflib being too smart for its own good
52
98
    # if /dev/null is "1,0", patch won't recognize it as /dev/null
53
99
    if not oldlines:
62
108
        to_file.write(line)
63
109
        if not line.endswith('\n'):
64
110
            to_file.write("\n\\ No newline at end of file\n")
65
 
    print >>to_file
 
111
    to_file.write('\n')
 
112
 
 
113
 
 
114
def _spawn_external_diff(diffcmd, capture_errors=True):
 
115
    """Spawn the externall diff process, and return the child handle.
 
116
 
 
117
    :param diffcmd: The command list to spawn
 
118
    :param capture_errors: Capture stderr as well as setting LANG=C
 
119
        and LC_ALL=C. This lets us read and understand the output of diff,
 
120
        and respond to any errors.
 
121
    :return: A Popen object.
 
122
    """
 
123
    if capture_errors:
 
124
        # construct minimal environment
 
125
        env = {}
 
126
        path = os.environ.get('PATH')
 
127
        if path is not None:
 
128
            env['PATH'] = path
 
129
        env['LANGUAGE'] = 'C'   # on win32 only LANGUAGE has effect
 
130
        env['LANG'] = 'C'
 
131
        env['LC_ALL'] = 'C'
 
132
        stderr = subprocess.PIPE
 
133
    else:
 
134
        env = None
 
135
        stderr = None
 
136
 
 
137
    try:
 
138
        pipe = subprocess.Popen(diffcmd,
 
139
                                stdin=subprocess.PIPE,
 
140
                                stdout=subprocess.PIPE,
 
141
                                stderr=stderr,
 
142
                                env=env)
 
143
    except OSError, e:
 
144
        if e.errno == errno.ENOENT:
 
145
            raise errors.NoDiff(str(e))
 
146
        raise
 
147
 
 
148
    return pipe
66
149
 
67
150
 
68
151
def external_diff(old_filename, oldlines, new_filename, newlines, to_file,
69
152
                  diff_opts):
70
153
    """Display a diff by calling out to the external diff program."""
71
 
    import sys
72
 
    
73
 
    if to_file != sys.stdout:
74
 
        raise NotImplementedError("sorry, can't send external diff other than to stdout yet",
75
 
                                  to_file)
76
 
 
77
154
    # make sure our own output is properly ordered before the diff
78
155
    to_file.flush()
79
156
 
80
 
    from tempfile import NamedTemporaryFile
81
 
    import os
82
 
 
83
 
    oldtmpf = NamedTemporaryFile()
84
 
    newtmpf = NamedTemporaryFile()
 
157
    oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-')
 
158
    newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-')
 
159
    oldtmpf = os.fdopen(oldtmp_fd, 'wb')
 
160
    newtmpf = os.fdopen(newtmp_fd, 'wb')
85
161
 
86
162
    try:
87
163
        # TODO: perhaps a special case for comparing to or from the empty
94
170
        oldtmpf.writelines(oldlines)
95
171
        newtmpf.writelines(newlines)
96
172
 
97
 
        oldtmpf.flush()
98
 
        newtmpf.flush()
 
173
        oldtmpf.close()
 
174
        newtmpf.close()
99
175
 
100
176
        if not diff_opts:
101
177
            diff_opts = []
102
178
        diffcmd = ['diff',
103
 
                   '--label', old_filename+'\t',
104
 
                   oldtmpf.name,
105
 
                   '--label', new_filename+'\t',
106
 
                   newtmpf.name]
 
179
                   '--label', old_filename,
 
180
                   old_abspath,
 
181
                   '--label', new_filename,
 
182
                   new_abspath,
 
183
                   '--binary',
 
184
                  ]
107
185
 
108
186
        # diff only allows one style to be specified; they don't override.
109
187
        # note that some of these take optargs, and the optargs can be
129
207
        if diff_opts:
130
208
            diffcmd.extend(diff_opts)
131
209
 
132
 
        rc = os.spawnvp(os.P_WAIT, 'diff', diffcmd)
 
210
        pipe = _spawn_external_diff(diffcmd, capture_errors=True)
 
211
        out,err = pipe.communicate()
 
212
        rc = pipe.returncode
133
213
        
134
 
        if rc != 0 and rc != 1:
 
214
        # internal_diff() adds a trailing newline, add one here for consistency
 
215
        out += '\n'
 
216
        if rc == 2:
 
217
            # 'diff' gives retcode == 2 for all sorts of errors
 
218
            # one of those is 'Binary files differ'.
 
219
            # Bad options could also be the problem.
 
220
            # 'Binary files' is not a real error, so we suppress that error.
 
221
            lang_c_out = out
 
222
 
 
223
            # Since we got here, we want to make sure to give an i18n error
 
224
            pipe = _spawn_external_diff(diffcmd, capture_errors=False)
 
225
            out, err = pipe.communicate()
 
226
 
 
227
            # Write out the new i18n diff response
 
228
            to_file.write(out+'\n')
 
229
            if pipe.returncode != 2:
 
230
                raise errors.BzrError(
 
231
                               'external diff failed with exit code 2'
 
232
                               ' when run with LANG=C and LC_ALL=C,'
 
233
                               ' but not when run natively: %r' % (diffcmd,))
 
234
 
 
235
            first_line = lang_c_out.split('\n', 1)[0]
 
236
            # Starting with diffutils 2.8.4 the word "binary" was dropped.
 
237
            m = re.match('^(binary )?files.*differ$', first_line, re.I)
 
238
            if m is None:
 
239
                raise errors.BzrError('external diff failed with exit code 2;'
 
240
                                      ' command: %r' % (diffcmd,))
 
241
            else:
 
242
                # Binary files differ, just return
 
243
                return
 
244
 
 
245
        # If we got to here, we haven't written out the output of diff
 
246
        # do so now
 
247
        to_file.write(out)
 
248
        if rc not in (0, 1):
135
249
            # returns 1 if files differ; that's OK
136
250
            if rc < 0:
137
251
                msg = 'signal %d' % (-rc)
138
252
            else:
139
253
                msg = 'exit code %d' % rc
140
254
                
141
 
            raise BzrError('external diff failed with %s; command: %r' % (rc, diffcmd))
 
255
            raise errors.BzrError('external diff failed with %s; command: %r' 
 
256
                                  % (rc, diffcmd))
 
257
 
 
258
 
142
259
    finally:
143
260
        oldtmpf.close()                 # and delete
144
261
        newtmpf.close()
145
 
 
146
 
 
147
 
@deprecated_function(zero_eight)
148
 
def show_diff(b, from_spec, specific_files, external_diff_options=None,
149
 
              revision2=None, output=None, b2=None):
150
 
    """Shortcut for showing the diff to the working tree.
151
 
 
152
 
    Please use show_diff_trees instead.
153
 
 
154
 
    b
155
 
        Branch.
156
 
 
157
 
    revision
158
 
        None for 'basis tree', or otherwise the old revision to compare against.
159
 
    
160
 
    The more general form is show_diff_trees(), where the caller
161
 
    supplies any two trees.
162
 
    """
163
 
    if output is None:
164
 
        import sys
165
 
        output = sys.stdout
166
 
 
167
 
    if from_spec is None:
168
 
        old_tree = b.bzrdir.open_workingtree()
169
 
        if b2 is None:
170
 
            old_tree = old_tree = old_tree.basis_tree()
171
 
    else:
172
 
        old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id)
173
 
 
174
 
    if revision2 is None:
175
 
        if b2 is None:
176
 
            new_tree = b.bzrdir.open_workingtree()
177
 
        else:
178
 
            new_tree = b2.bzrdir.open_workingtree()
179
 
    else:
180
 
        new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id)
181
 
 
182
 
    return show_diff_trees(old_tree, new_tree, output, specific_files,
183
 
                           external_diff_options)
184
 
 
185
 
 
 
262
        # Clean up. Warn in case the files couldn't be deleted
 
263
        # (in case windows still holds the file open, but not
 
264
        # if the files have already been deleted)
 
265
        try:
 
266
            os.remove(old_abspath)
 
267
        except OSError, e:
 
268
            if e.errno not in (errno.ENOENT,):
 
269
                warning('Failed to delete temporary file: %s %s',
 
270
                        old_abspath, e)
 
271
        try:
 
272
            os.remove(new_abspath)
 
273
        except OSError:
 
274
            if e.errno not in (errno.ENOENT,):
 
275
                warning('Failed to delete temporary file: %s %s',
 
276
                        new_abspath, e)
 
277
 
 
278
 
 
279
@deprecated_function(one_zero)
186
280
def diff_cmd_helper(tree, specific_files, external_diff_options, 
187
 
                    old_revision_spec=None, new_revision_spec=None):
 
281
                    old_revision_spec=None, new_revision_spec=None,
 
282
                    revision_specs=None,
 
283
                    old_label='a/', new_label='b/'):
188
284
    """Helper for cmd_diff.
189
285
 
190
 
   tree 
 
286
    :param tree:
191
287
        A WorkingTree
192
288
 
193
 
    specific_files
 
289
    :param specific_files:
194
290
        The specific files to compare, or None
195
291
 
196
 
    external_diff_options
 
292
    :param external_diff_options:
197
293
        If non-None, run an external diff, and pass it these options
198
294
 
199
 
    old_revision_spec
 
295
    :param old_revision_spec:
200
296
        If None, use basis tree as old revision, otherwise use the tree for
201
297
        the specified revision. 
202
298
 
203
 
    new_revision_spec
 
299
    :param new_revision_spec:
204
300
        If None, use working tree as new revision, otherwise use the tree for
205
301
        the specified revision.
206
302
    
 
303
    :param revision_specs: 
 
304
        Zero, one or two RevisionSpecs from the command line, saying what revisions 
 
305
        to compare.  This can be passed as an alternative to the old_revision_spec 
 
306
        and new_revision_spec parameters.
 
307
 
207
308
    The more general form is show_diff_trees(), where the caller
208
309
    supplies any two trees.
209
310
    """
210
 
    import sys
211
 
    output = sys.stdout
 
311
 
 
312
    # TODO: perhaps remove the old parameters old_revision_spec and
 
313
    # new_revision_spec, since this is only really for use from cmd_diff and
 
314
    # it now always passes through a sequence of revision_specs -- mbp
 
315
    # 20061221
 
316
 
212
317
    def spec_tree(spec):
213
 
        revision_id = spec.in_store(tree.branch).rev_id
214
 
        return tree.branch.repository.revision_tree(revision_id)
 
318
        if tree:
 
319
            revision = spec.in_store(tree.branch)
 
320
        else:
 
321
            revision = spec.in_store(None)
 
322
        revision_id = revision.rev_id
 
323
        branch = revision.branch
 
324
        return branch.repository.revision_tree(revision_id)
 
325
 
 
326
    if revision_specs is not None:
 
327
        assert (old_revision_spec is None
 
328
                and new_revision_spec is None)
 
329
        if len(revision_specs) > 0:
 
330
            old_revision_spec = revision_specs[0]
 
331
        if len(revision_specs) > 1:
 
332
            new_revision_spec = revision_specs[1]
 
333
 
215
334
    if old_revision_spec is None:
216
335
        old_tree = tree.basis_tree()
217
336
    else:
218
337
        old_tree = spec_tree(old_revision_spec)
219
338
 
220
 
    if new_revision_spec is None:
 
339
    if (new_revision_spec is None
 
340
        or new_revision_spec.spec is None):
221
341
        new_tree = tree
222
342
    else:
223
343
        new_tree = spec_tree(new_revision_spec)
224
344
 
 
345
    if new_tree is not tree:
 
346
        extra_trees = (tree,)
 
347
    else:
 
348
        extra_trees = None
 
349
 
225
350
    return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files,
226
 
                           external_diff_options)
 
351
                           external_diff_options,
 
352
                           old_label=old_label, new_label=new_label,
 
353
                           extra_trees=extra_trees)
 
354
 
 
355
 
 
356
def _get_trees_to_diff(path_list, revision_specs, old_url, new_url):
 
357
    """Get the trees and specific files to diff given a list of paths.
 
358
 
 
359
    This method works out the trees to be diff'ed and the files of
 
360
    interest within those trees.
 
361
 
 
362
    :param path_list:
 
363
        the list of arguments passed to the diff command
 
364
    :param revision_specs:
 
365
        Zero, one or two RevisionSpecs from the diff command line,
 
366
        saying what revisions to compare.
 
367
    :param old_url:
 
368
        The url of the old branch or tree. If None, the tree to use is
 
369
        taken from the first path, if any, or the current working tree.
 
370
    :param new_url:
 
371
        The url of the new branch or tree. If None, the tree to use is
 
372
        taken from the first path, if any, or the current working tree.
 
373
    :returns:
 
374
        a tuple of (old_tree, new_tree, specific_files, extra_trees) where
 
375
        extra_trees is a sequence of additional trees to search in for
 
376
        file-ids.
 
377
    """
 
378
    # Get the old and new revision specs
 
379
    old_revision_spec = None
 
380
    new_revision_spec = None
 
381
    if revision_specs is not None:
 
382
        if len(revision_specs) > 0:
 
383
            old_revision_spec = revision_specs[0]
 
384
            if old_url is None:
 
385
                old_url = old_revision_spec.get_branch()
 
386
        if len(revision_specs) > 1:
 
387
            new_revision_spec = revision_specs[1]
 
388
            if new_url is None:
 
389
                new_url = new_revision_spec.get_branch()
 
390
 
 
391
    other_paths = []
 
392
    make_paths_wt_relative = True
 
393
    consider_relpath = True
 
394
    if path_list is None or len(path_list) == 0:
 
395
        # If no path is given, the current working tree is used
 
396
        default_location = u'.'
 
397
        consider_relpath = False
 
398
    elif old_url is not None and new_url is not None:
 
399
        other_paths = path_list
 
400
        make_paths_wt_relative = False
 
401
    else:
 
402
        default_location = path_list[0]
 
403
        other_paths = path_list[1:]
 
404
 
 
405
    # Get the old location
 
406
    specific_files = []
 
407
    if old_url is None:
 
408
        old_url = default_location
 
409
    working_tree, branch, relpath = \
 
410
        bzrdir.BzrDir.open_containing_tree_or_branch(old_url)
 
411
    if consider_relpath and relpath != '':
 
412
        specific_files.append(relpath)
 
413
    old_tree = _get_tree_to_diff(old_revision_spec, working_tree, branch)
 
414
 
 
415
    # Get the new location
 
416
    if new_url is None:
 
417
        new_url = default_location
 
418
    if new_url != old_url:
 
419
        working_tree, branch, relpath = \
 
420
            bzrdir.BzrDir.open_containing_tree_or_branch(new_url)
 
421
        if consider_relpath and relpath != '':
 
422
            specific_files.append(relpath)
 
423
    new_tree = _get_tree_to_diff(new_revision_spec, working_tree, branch,
 
424
        basis_is_default=working_tree is None)
 
425
 
 
426
    # Get the specific files (all files is None, no files is [])
 
427
    if make_paths_wt_relative and working_tree is not None:
 
428
        other_paths = _relative_paths_in_tree(working_tree, other_paths)
 
429
    specific_files.extend(other_paths)
 
430
    if len(specific_files) == 0:
 
431
        specific_files = None
 
432
 
 
433
    # Get extra trees that ought to be searched for file-ids
 
434
    extra_trees = None
 
435
    if working_tree is not None and working_tree not in (old_tree, new_tree):
 
436
        extra_trees = (working_tree,)
 
437
    return old_tree, new_tree, specific_files, extra_trees
 
438
 
 
439
 
 
440
def _get_tree_to_diff(spec, tree=None, branch=None, basis_is_default=True):
 
441
    if branch is None and tree is not None:
 
442
        branch = tree.branch
 
443
    if spec is None or spec.spec is None:
 
444
        if basis_is_default:
 
445
            if tree is not None:
 
446
                return tree.basis_tree()
 
447
            else:
 
448
                return branch.basis_tree()
 
449
        else:
 
450
            return tree
 
451
    if not spec.needs_branch():
 
452
        branch = _mod_branch.Branch.open(spec.get_branch())
 
453
    revision_id = spec.as_revision_id(branch)
 
454
    return branch.repository.revision_tree(revision_id)
 
455
 
 
456
 
 
457
def _relative_paths_in_tree(tree, paths):
 
458
    """Get the relative paths within a working tree.
 
459
 
 
460
    Each path may be either an absolute path or a path relative to the
 
461
    current working directory.
 
462
    """
 
463
    result = []
 
464
    for filename in paths:
 
465
        try:
 
466
            result.append(tree.relpath(osutils.dereference_path(filename)))
 
467
        except errors.PathNotChild:
 
468
            raise errors.BzrCommandError("Files are in different branches")
 
469
    return result
227
470
 
228
471
 
229
472
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None,
230
 
                    external_diff_options=None):
 
473
                    external_diff_options=None,
 
474
                    old_label='a/', new_label='b/',
 
475
                    extra_trees=None,
 
476
                    path_encoding='utf8',
 
477
                    using=None):
231
478
    """Show in text form the changes from one tree to another.
232
479
 
233
 
    to_files
234
 
        If set, include only changes to these files.
 
480
    to_file
 
481
        The output stream.
 
482
 
 
483
    specific_files
 
484
        Include only changes to these files - None for all changes.
235
485
 
236
486
    external_diff_options
237
487
        If set, use an external GNU diff and pass these options.
 
488
 
 
489
    extra_trees
 
490
        If set, more Trees to use for looking up file ids
 
491
 
 
492
    path_encoding
 
493
        If set, the path will be encoded as specified, otherwise is supposed
 
494
        to be utf8
238
495
    """
239
496
    old_tree.lock_read()
240
497
    try:
 
498
        if extra_trees is not None:
 
499
            for tree in extra_trees:
 
500
                tree.lock_read()
241
501
        new_tree.lock_read()
242
502
        try:
243
 
            return _show_diff_trees(old_tree, new_tree, to_file,
244
 
                                    specific_files, external_diff_options)
 
503
            differ = DiffTree.from_trees_options(old_tree, new_tree, to_file,
 
504
                                                 path_encoding,
 
505
                                                 external_diff_options,
 
506
                                                 old_label, new_label, using)
 
507
            return differ.show_diff(specific_files, extra_trees)
245
508
        finally:
246
509
            new_tree.unlock()
 
510
            if extra_trees is not None:
 
511
                for tree in extra_trees:
 
512
                    tree.unlock()
247
513
    finally:
248
514
        old_tree.unlock()
249
515
 
250
516
 
251
 
def _show_diff_trees(old_tree, new_tree, to_file,
252
 
                     specific_files, external_diff_options):
253
 
 
254
 
    # TODO: Options to control putting on a prefix or suffix, perhaps
255
 
    # as a format string?
256
 
    old_label = 'a/'
257
 
    new_label = 'b/'
258
 
 
259
 
    DEVNULL = '/dev/null'
260
 
    # Windows users, don't panic about this filename -- it is a
261
 
    # special signal to GNU patch that the file should be created or
262
 
    # deleted respectively.
263
 
 
264
 
    # TODO: Generation of pseudo-diffs for added/deleted files could
265
 
    # be usefully made into a much faster special case.
266
 
 
267
 
    _raise_if_doubly_unversioned(specific_files, old_tree, new_tree)
268
 
 
269
 
    if external_diff_options:
270
 
        assert isinstance(external_diff_options, basestring)
271
 
        opts = external_diff_options.split()
272
 
        def diff_file(olab, olines, nlab, nlines, to_file):
273
 
            external_diff(olab, olines, nlab, nlines, to_file, opts)
274
 
    else:
275
 
        diff_file = internal_diff
 
517
def _patch_header_date(tree, file_id, path):
 
518
    """Returns a timestamp suitable for use in a patch header."""
 
519
    mtime = tree.get_file_mtime(file_id, path)
 
520
    assert mtime is not None, \
 
521
        "got an mtime of None for file-id %s, path %s in tree %s" % (
 
522
                file_id, path, tree)
 
523
    return timestamp.format_patch_date(mtime)
 
524
 
 
525
 
 
526
def _raise_if_nonexistent(paths, old_tree, new_tree):
 
527
    """Complain if paths are not in either inventory or tree.
 
528
 
 
529
    It's OK with the files exist in either tree's inventory, or 
 
530
    if they exist in the tree but are not versioned.
276
531
    
277
 
    delta = compare_trees(old_tree, new_tree, want_unchanged=False,
278
 
                          specific_files=specific_files)
279
 
 
280
 
    has_changes = 0
281
 
    for path, file_id, kind in delta.removed:
282
 
        has_changes = 1
283
 
        print >>to_file, '=== removed %s %r' % (kind, old_label + path)
284
 
        old_tree.inventory[file_id].diff(diff_file, old_label + path, old_tree,
285
 
                                         DEVNULL, None, None, to_file)
286
 
    for path, file_id, kind in delta.added:
287
 
        has_changes = 1
288
 
        print >>to_file, '=== added %s %r' % (kind, new_label + path)
289
 
        new_tree.inventory[file_id].diff(diff_file, new_label + path, new_tree,
290
 
                                         DEVNULL, None, None, to_file, 
291
 
                                         reverse=True)
292
 
    for (old_path, new_path, file_id, kind,
293
 
         text_modified, meta_modified) in delta.renamed:
294
 
        has_changes = 1
295
 
        prop_str = get_prop_change(meta_modified)
296
 
        print >>to_file, '=== renamed %s %r => %r%s' % (
297
 
                    kind, old_label + old_path, new_label + new_path, prop_str)
298
 
        _maybe_diff_file_or_symlink(old_label, old_path, old_tree, file_id,
299
 
                                    new_label, new_path, new_tree,
300
 
                                    text_modified, kind, to_file, diff_file)
301
 
    for path, file_id, kind, text_modified, meta_modified in delta.modified:
302
 
        has_changes = 1
303
 
        prop_str = get_prop_change(meta_modified)
304
 
        print >>to_file, '=== modified %s %r%s' % (kind, old_label + path,
305
 
                    prop_str)
306
 
        if text_modified:
307
 
            _maybe_diff_file_or_symlink(old_label, path, old_tree, file_id,
308
 
                                        new_label, path, new_tree,
309
 
                                        True, kind, to_file, diff_file)
310
 
 
311
 
    return has_changes
312
 
 
313
 
 
314
 
def _raise_if_doubly_unversioned(specific_files, old_tree, new_tree):
315
 
    """Complain if paths are not versioned in either tree."""
316
 
    if not specific_files:
 
532
    This can be used by operations such as bzr status that can accept
 
533
    unknown or ignored files.
 
534
    """
 
535
    mutter("check paths: %r", paths)
 
536
    if not paths:
317
537
        return
318
 
    old_unversioned = old_tree.filter_unversioned_files(specific_files)
319
 
    new_unversioned = new_tree.filter_unversioned_files(specific_files)
320
 
    unversioned = old_unversioned.intersection(new_unversioned)
321
 
    if unversioned:
322
 
        raise errors.PathsNotVersionedError(sorted(unversioned))
323
 
    
324
 
 
 
538
    s = old_tree.filter_unversioned_files(paths)
 
539
    s = new_tree.filter_unversioned_files(s)
 
540
    s = [path for path in s if not new_tree.has_filename(path)]
 
541
    if s:
 
542
        raise errors.PathsDoNotExist(sorted(s))
 
543
 
 
544
 
 
545
@deprecated_function(one_three)
325
546
def get_prop_change(meta_modified):
326
547
    if meta_modified:
327
548
        return " (properties changed)"
328
549
    else:
329
550
        return  ""
330
551
 
331
 
 
332
 
def _maybe_diff_file_or_symlink(old_label, old_path, old_tree, file_id,
333
 
                                new_label, new_path, new_tree, text_modified,
334
 
                                kind, to_file, diff_file):
335
 
    if text_modified:
336
 
        new_entry = new_tree.inventory[file_id]
337
 
        old_tree.inventory[file_id].diff(diff_file,
338
 
                                         old_label + old_path, old_tree,
339
 
                                         new_label + new_path, new_entry, 
340
 
                                         new_tree, to_file)
 
552
def get_executable_change(old_is_x, new_is_x):
 
553
    descr = { True:"+x", False:"-x", None:"??" }
 
554
    if old_is_x != new_is_x:
 
555
        return ["%s to %s" % (descr[old_is_x], descr[new_is_x],)]
 
556
    else:
 
557
        return []
 
558
 
 
559
 
 
560
class DiffPath(object):
 
561
    """Base type for command object that compare files"""
 
562
 
 
563
    # The type or contents of the file were unsuitable for diffing
 
564
    CANNOT_DIFF = 'CANNOT_DIFF'
 
565
    # The file has changed in a semantic way
 
566
    CHANGED = 'CHANGED'
 
567
    # The file content may have changed, but there is no semantic change
 
568
    UNCHANGED = 'UNCHANGED'
 
569
 
 
570
    def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8'):
 
571
        """Constructor.
 
572
 
 
573
        :param old_tree: The tree to show as the old tree in the comparison
 
574
        :param new_tree: The tree to show as new in the comparison
 
575
        :param to_file: The file to write comparison data to
 
576
        :param path_encoding: The character encoding to write paths in
 
577
        """
 
578
        self.old_tree = old_tree
 
579
        self.new_tree = new_tree
 
580
        self.to_file = to_file
 
581
        self.path_encoding = path_encoding
 
582
 
 
583
    def finish(self):
 
584
        pass
 
585
 
 
586
    @classmethod
 
587
    def from_diff_tree(klass, diff_tree):
 
588
        return klass(diff_tree.old_tree, diff_tree.new_tree,
 
589
                     diff_tree.to_file, diff_tree.path_encoding)
 
590
 
 
591
    @staticmethod
 
592
    def _diff_many(differs, file_id, old_path, new_path, old_kind, new_kind):
 
593
        for file_differ in differs:
 
594
            result = file_differ.diff(file_id, old_path, new_path, old_kind,
 
595
                                      new_kind)
 
596
            if result is not DiffPath.CANNOT_DIFF:
 
597
                return result
 
598
        else:
 
599
            return DiffPath.CANNOT_DIFF
 
600
 
 
601
 
 
602
class DiffKindChange(object):
 
603
    """Special differ for file kind changes.
 
604
 
 
605
    Represents kind change as deletion + creation.  Uses the other differs
 
606
    to do this.
 
607
    """
 
608
    def __init__(self, differs):
 
609
        self.differs = differs
 
610
 
 
611
    def finish(self):
 
612
        pass
 
613
 
 
614
    @classmethod
 
615
    def from_diff_tree(klass, diff_tree):
 
616
        return klass(diff_tree.differs)
 
617
 
 
618
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
 
619
        """Perform comparison
 
620
 
 
621
        :param file_id: The file_id of the file to compare
 
622
        :param old_path: Path of the file in the old tree
 
623
        :param new_path: Path of the file in the new tree
 
624
        :param old_kind: Old file-kind of the file
 
625
        :param new_kind: New file-kind of the file
 
626
        """
 
627
        if None in (old_kind, new_kind):
 
628
            return DiffPath.CANNOT_DIFF
 
629
        result = DiffPath._diff_many(self.differs, file_id, old_path,
 
630
                                       new_path, old_kind, None)
 
631
        if result is DiffPath.CANNOT_DIFF:
 
632
            return result
 
633
        return DiffPath._diff_many(self.differs, file_id, old_path, new_path,
 
634
                                     None, new_kind)
 
635
 
 
636
 
 
637
class DiffDirectory(DiffPath):
 
638
 
 
639
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
 
640
        """Perform comparison between two directories.  (dummy)
 
641
 
 
642
        """
 
643
        if 'directory' not in (old_kind, new_kind):
 
644
            return self.CANNOT_DIFF
 
645
        if old_kind not in ('directory', None):
 
646
            return self.CANNOT_DIFF
 
647
        if new_kind not in ('directory', None):
 
648
            return self.CANNOT_DIFF
 
649
        return self.CHANGED
 
650
 
 
651
 
 
652
class DiffSymlink(DiffPath):
 
653
 
 
654
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
 
655
        """Perform comparison between two symlinks
 
656
 
 
657
        :param file_id: The file_id of the file to compare
 
658
        :param old_path: Path of the file in the old tree
 
659
        :param new_path: Path of the file in the new tree
 
660
        :param old_kind: Old file-kind of the file
 
661
        :param new_kind: New file-kind of the file
 
662
        """
 
663
        if 'symlink' not in (old_kind, new_kind):
 
664
            return self.CANNOT_DIFF
 
665
        if old_kind == 'symlink':
 
666
            old_target = self.old_tree.get_symlink_target(file_id)
 
667
        elif old_kind is None:
 
668
            old_target = None
 
669
        else:
 
670
            return self.CANNOT_DIFF
 
671
        if new_kind == 'symlink':
 
672
            new_target = self.new_tree.get_symlink_target(file_id)
 
673
        elif new_kind is None:
 
674
            new_target = None
 
675
        else:
 
676
            return self.CANNOT_DIFF
 
677
        return self.diff_symlink(old_target, new_target)
 
678
 
 
679
    def diff_symlink(self, old_target, new_target):
 
680
        if old_target is None:
 
681
            self.to_file.write('=== target is %r\n' % new_target)
 
682
        elif new_target is None:
 
683
            self.to_file.write('=== target was %r\n' % old_target)
 
684
        else:
 
685
            self.to_file.write('=== target changed %r => %r\n' %
 
686
                              (old_target, new_target))
 
687
        return self.CHANGED
 
688
 
 
689
 
 
690
class DiffText(DiffPath):
 
691
 
 
692
    # GNU Patch uses the epoch date to detect files that are being added
 
693
    # or removed in a diff.
 
694
    EPOCH_DATE = '1970-01-01 00:00:00 +0000'
 
695
 
 
696
    def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8',
 
697
                 old_label='', new_label='', text_differ=internal_diff):
 
698
        DiffPath.__init__(self, old_tree, new_tree, to_file, path_encoding)
 
699
        self.text_differ = text_differ
 
700
        self.old_label = old_label
 
701
        self.new_label = new_label
 
702
        self.path_encoding = path_encoding
 
703
 
 
704
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
 
705
        """Compare two files in unified diff format
 
706
 
 
707
        :param file_id: The file_id of the file to compare
 
708
        :param old_path: Path of the file in the old tree
 
709
        :param new_path: Path of the file in the new tree
 
710
        :param old_kind: Old file-kind of the file
 
711
        :param new_kind: New file-kind of the file
 
712
        """
 
713
        if 'file' not in (old_kind, new_kind):
 
714
            return self.CANNOT_DIFF
 
715
        from_file_id = to_file_id = file_id
 
716
        if old_kind == 'file':
 
717
            old_date = _patch_header_date(self.old_tree, file_id, old_path)
 
718
        elif old_kind is None:
 
719
            old_date = self.EPOCH_DATE
 
720
            from_file_id = None
 
721
        else:
 
722
            return self.CANNOT_DIFF
 
723
        if new_kind == 'file':
 
724
            new_date = _patch_header_date(self.new_tree, file_id, new_path)
 
725
        elif new_kind is None:
 
726
            new_date = self.EPOCH_DATE
 
727
            to_file_id = None
 
728
        else:
 
729
            return self.CANNOT_DIFF
 
730
        from_label = '%s%s\t%s' % (self.old_label, old_path, old_date)
 
731
        to_label = '%s%s\t%s' % (self.new_label, new_path, new_date)
 
732
        return self.diff_text(from_file_id, to_file_id, from_label, to_label)
 
733
 
 
734
    def diff_text(self, from_file_id, to_file_id, from_label, to_label):
 
735
        """Diff the content of given files in two trees
 
736
 
 
737
        :param from_file_id: The id of the file in the from tree.  If None,
 
738
            the file is not present in the from tree.
 
739
        :param to_file_id: The id of the file in the to tree.  This may refer
 
740
            to a different file from from_file_id.  If None,
 
741
            the file is not present in the to tree.
 
742
        """
 
743
        def _get_text(tree, file_id):
 
744
            if file_id is not None:
 
745
                return tree.get_file(file_id).readlines()
 
746
            else:
 
747
                return []
 
748
        try:
 
749
            from_text = _get_text(self.old_tree, from_file_id)
 
750
            to_text = _get_text(self.new_tree, to_file_id)
 
751
            self.text_differ(from_label, from_text, to_label, to_text,
 
752
                             self.to_file)
 
753
        except errors.BinaryFile:
 
754
            self.to_file.write(
 
755
                  ("Binary files %s and %s differ\n" %
 
756
                  (from_label, to_label)).encode(self.path_encoding))
 
757
        return self.CHANGED
 
758
 
 
759
 
 
760
class DiffFromTool(DiffPath):
 
761
 
 
762
    def __init__(self, command_template, old_tree, new_tree, to_file,
 
763
                 path_encoding='utf-8'):
 
764
        DiffPath.__init__(self, old_tree, new_tree, to_file, path_encoding)
 
765
        self.command_template = command_template
 
766
        self._root = tempfile.mkdtemp(prefix='bzr-diff-')
 
767
 
 
768
    @classmethod
 
769
    def from_string(klass, command_string, old_tree, new_tree, to_file,
 
770
                    path_encoding='utf-8'):
 
771
        command_template = commands.shlex_split_unicode(command_string)
 
772
        command_template.extend(['%(old_path)s', '%(new_path)s'])
 
773
        return klass(command_template, old_tree, new_tree, to_file,
 
774
                     path_encoding)
 
775
 
 
776
    @classmethod
 
777
    def make_from_diff_tree(klass, command_string):
 
778
        def from_diff_tree(diff_tree):
 
779
            return klass.from_string(command_string, diff_tree.old_tree,
 
780
                                     diff_tree.new_tree, diff_tree.to_file)
 
781
        return from_diff_tree
 
782
 
 
783
    def _get_command(self, old_path, new_path):
 
784
        my_map = {'old_path': old_path, 'new_path': new_path}
 
785
        return [t % my_map for t in self.command_template]
 
786
 
 
787
    def _execute(self, old_path, new_path):
 
788
        command = self._get_command(old_path, new_path)
 
789
        try:
 
790
            proc = subprocess.Popen(command, stdout=subprocess.PIPE,
 
791
                                    cwd=self._root)
 
792
        except OSError, e:
 
793
            if e.errno == errno.ENOENT:
 
794
                raise errors.ExecutableMissing(command[0])
 
795
            else:
 
796
                raise
 
797
        self.to_file.write(proc.stdout.read())
 
798
        return proc.wait()
 
799
 
 
800
    def _try_symlink_root(self, tree, prefix):
 
801
        if not (getattr(tree, 'abspath', None) is not None
 
802
                and osutils.has_symlinks()):
 
803
            return False
 
804
        try:
 
805
            os.symlink(tree.abspath(''), osutils.pathjoin(self._root, prefix))
 
806
        except OSError, e:
 
807
            if e.errno != errno.EEXIST:
 
808
                raise
 
809
        return True
 
810
 
 
811
    def _write_file(self, file_id, tree, prefix, relpath):
 
812
        full_path = osutils.pathjoin(self._root, prefix, relpath)
 
813
        if self._try_symlink_root(tree, prefix):
 
814
            return full_path
 
815
        parent_dir = osutils.dirname(full_path)
 
816
        try:
 
817
            os.makedirs(parent_dir)
 
818
        except OSError, e:
 
819
            if e.errno != errno.EEXIST:
 
820
                raise
 
821
        source = tree.get_file(file_id, relpath)
 
822
        try:
 
823
            target = open(full_path, 'wb')
 
824
            try:
 
825
                osutils.pumpfile(source, target)
 
826
            finally:
 
827
                target.close()
 
828
        finally:
 
829
            source.close()
 
830
        osutils.make_readonly(full_path)
 
831
        mtime = tree.get_file_mtime(file_id)
 
832
        os.utime(full_path, (mtime, mtime))
 
833
        return full_path
 
834
 
 
835
    def _prepare_files(self, file_id, old_path, new_path):
 
836
        old_disk_path = self._write_file(file_id, self.old_tree, 'old',
 
837
                                         old_path)
 
838
        new_disk_path = self._write_file(file_id, self.new_tree, 'new',
 
839
                                         new_path)
 
840
        return old_disk_path, new_disk_path
 
841
 
 
842
    def finish(self):
 
843
        osutils.rmtree(self._root)
 
844
 
 
845
    def diff(self, file_id, old_path, new_path, old_kind, new_kind):
 
846
        if (old_kind, new_kind) != ('file', 'file'):
 
847
            return DiffPath.CANNOT_DIFF
 
848
        self._prepare_files(file_id, old_path, new_path)
 
849
        self._execute(osutils.pathjoin('old', old_path),
 
850
                      osutils.pathjoin('new', new_path))
 
851
 
 
852
 
 
853
class DiffTree(object):
 
854
    """Provides textual representations of the difference between two trees.
 
855
 
 
856
    A DiffTree examines two trees and where a file-id has altered
 
857
    between them, generates a textual representation of the difference.
 
858
    DiffTree uses a sequence of DiffPath objects which are each
 
859
    given the opportunity to handle a given altered fileid. The list
 
860
    of DiffPath objects can be extended globally by appending to
 
861
    DiffTree.diff_factories, or for a specific diff operation by
 
862
    supplying the extra_factories option to the appropriate method.
 
863
    """
 
864
 
 
865
    # list of factories that can provide instances of DiffPath objects
 
866
    # may be extended by plugins.
 
867
    diff_factories = [DiffSymlink.from_diff_tree,
 
868
                      DiffDirectory.from_diff_tree]
 
869
 
 
870
    def __init__(self, old_tree, new_tree, to_file, path_encoding='utf-8',
 
871
                 diff_text=None, extra_factories=None):
 
872
        """Constructor
 
873
 
 
874
        :param old_tree: Tree to show as old in the comparison
 
875
        :param new_tree: Tree to show as new in the comparison
 
876
        :param to_file: File to write comparision to
 
877
        :param path_encoding: Character encoding to write paths in
 
878
        :param diff_text: DiffPath-type object to use as a last resort for
 
879
            diffing text files.
 
880
        :param extra_factories: Factories of DiffPaths to try before any other
 
881
            DiffPaths"""
 
882
        if diff_text is None:
 
883
            diff_text = DiffText(old_tree, new_tree, to_file, path_encoding,
 
884
                                 '', '',  internal_diff)
 
885
        self.old_tree = old_tree
 
886
        self.new_tree = new_tree
 
887
        self.to_file = to_file
 
888
        self.path_encoding = path_encoding
 
889
        self.differs = []
 
890
        if extra_factories is not None:
 
891
            self.differs.extend(f(self) for f in extra_factories)
 
892
        self.differs.extend(f(self) for f in self.diff_factories)
 
893
        self.differs.extend([diff_text, DiffKindChange.from_diff_tree(self)])
 
894
 
 
895
    @classmethod
 
896
    def from_trees_options(klass, old_tree, new_tree, to_file,
 
897
                           path_encoding, external_diff_options, old_label,
 
898
                           new_label, using):
 
899
        """Factory for producing a DiffTree.
 
900
 
 
901
        Designed to accept options used by show_diff_trees.
 
902
        :param old_tree: The tree to show as old in the comparison
 
903
        :param new_tree: The tree to show as new in the comparison
 
904
        :param to_file: File to write comparisons to
 
905
        :param path_encoding: Character encoding to use for writing paths
 
906
        :param external_diff_options: If supplied, use the installed diff
 
907
            binary to perform file comparison, using supplied options.
 
908
        :param old_label: Prefix to use for old file labels
 
909
        :param new_label: Prefix to use for new file labels
 
910
        :param using: Commandline to use to invoke an external diff tool
 
911
        """
 
912
        if using is not None:
 
913
            extra_factories = [DiffFromTool.make_from_diff_tree(using)]
 
914
        else:
 
915
            extra_factories = []
 
916
        if external_diff_options:
 
917
            assert isinstance(external_diff_options, basestring)
 
918
            opts = external_diff_options.split()
 
919
            def diff_file(olab, olines, nlab, nlines, to_file):
 
920
                external_diff(olab, olines, nlab, nlines, to_file, opts)
 
921
        else:
 
922
            diff_file = internal_diff
 
923
        diff_text = DiffText(old_tree, new_tree, to_file, path_encoding,
 
924
                             old_label, new_label, diff_file)
 
925
        return klass(old_tree, new_tree, to_file, path_encoding, diff_text,
 
926
                     extra_factories)
 
927
 
 
928
    def show_diff(self, specific_files, extra_trees=None):
 
929
        """Write tree diff to self.to_file
 
930
 
 
931
        :param sepecific_files: the specific files to compare (recursive)
 
932
        :param extra_trees: extra trees to use for mapping paths to file_ids
 
933
        """
 
934
        try:
 
935
            return self._show_diff(specific_files, extra_trees)
 
936
        finally:
 
937
            for differ in self.differs:
 
938
                differ.finish()
 
939
 
 
940
    def _show_diff(self, specific_files, extra_trees):
 
941
        # TODO: Generation of pseudo-diffs for added/deleted files could
 
942
        # be usefully made into a much faster special case.
 
943
        iterator = self.new_tree.iter_changes(self.old_tree,
 
944
                                               specific_files=specific_files,
 
945
                                               extra_trees=extra_trees,
 
946
                                               require_versioned=True)
 
947
        has_changes = 0
 
948
        def changes_key(change):
 
949
            old_path, new_path = change[1]
 
950
            path = new_path
 
951
            if path is None:
 
952
                path = old_path
 
953
            return path
 
954
        def get_encoded_path(path):
 
955
            if path is not None:
 
956
                return path.encode(self.path_encoding, "replace")
 
957
        for (file_id, paths, changed_content, versioned, parent, name, kind,
 
958
             executable) in sorted(iterator, key=changes_key):
 
959
            if parent == (None, None):
 
960
                continue
 
961
            oldpath, newpath = paths
 
962
            oldpath_encoded = get_encoded_path(paths[0])
 
963
            newpath_encoded = get_encoded_path(paths[1])
 
964
            old_present = (kind[0] is not None and versioned[0])
 
965
            new_present = (kind[1] is not None and versioned[1])
 
966
            renamed = (parent[0], name[0]) != (parent[1], name[1])
 
967
 
 
968
            properties_changed = []
 
969
            properties_changed.extend(get_executable_change(executable[0], executable[1]))
 
970
 
 
971
            if properties_changed:
 
972
                prop_str = " (properties changed: %s)" % (", ".join(properties_changed),)
 
973
            else:
 
974
                prop_str = ""
 
975
 
 
976
            if (old_present, new_present) == (True, False):
 
977
                self.to_file.write("=== removed %s '%s'\n" %
 
978
                                   (kind[0], oldpath_encoded))
 
979
                newpath = oldpath
 
980
            elif (old_present, new_present) == (False, True):
 
981
                self.to_file.write("=== added %s '%s'\n" %
 
982
                                   (kind[1], newpath_encoded))
 
983
                oldpath = newpath
 
984
            elif renamed:
 
985
                self.to_file.write("=== renamed %s '%s' => '%s'%s\n" %
 
986
                    (kind[0], oldpath_encoded, newpath_encoded, prop_str))
 
987
            else:
 
988
                # if it was produced by iter_changes, it must be
 
989
                # modified *somehow*, either content or execute bit.
 
990
                self.to_file.write("=== modified %s '%s'%s\n" % (kind[0],
 
991
                                   newpath_encoded, prop_str))
 
992
            if changed_content:
 
993
                self.diff(file_id, oldpath, newpath)
 
994
                has_changes = 1
 
995
            if renamed:
 
996
                has_changes = 1
 
997
        return has_changes
 
998
 
 
999
    def diff(self, file_id, old_path, new_path):
 
1000
        """Perform a diff of a single file
 
1001
 
 
1002
        :param file_id: file-id of the file
 
1003
        :param old_path: The path of the file in the old tree
 
1004
        :param new_path: The path of the file in the new tree
 
1005
        """
 
1006
        try:
 
1007
            old_kind = self.old_tree.kind(file_id)
 
1008
        except (errors.NoSuchId, errors.NoSuchFile):
 
1009
            old_kind = None
 
1010
        try:
 
1011
            new_kind = self.new_tree.kind(file_id)
 
1012
        except (errors.NoSuchId, errors.NoSuchFile):
 
1013
            new_kind = None
 
1014
 
 
1015
        result = DiffPath._diff_many(self.differs, file_id, old_path,
 
1016
                                       new_path, old_kind, new_kind)
 
1017
        if result is DiffPath.CANNOT_DIFF:
 
1018
            error_path = new_path
 
1019
            if error_path is None:
 
1020
                error_path = old_path
 
1021
            raise errors.NoDiffFound(error_path)