~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/diff.py

  • Committer: mbp at sourcefrog
  • Date: 2005-03-21 22:29:49 UTC
  • Revision ID: mbp@sourcefrog.net-20050321222949-232c2093a6eadd80
fixup doctest for new module structure

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2004, 2005, 2006 Canonical Ltd.
2
 
#
 
1
#! /usr/bin/env python
 
2
# -*- coding: UTF-8 -*-
 
3
 
3
4
# This program is free software; you can redistribute it and/or modify
4
5
# it under the terms of the GNU General Public License as published by
5
6
# the Free Software Foundation; either version 2 of the License, or
6
7
# (at your option) any later version.
7
 
#
 
8
 
8
9
# This program is distributed in the hope that it will be useful,
9
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
12
# GNU General Public License for more details.
12
 
#
 
13
 
13
14
# You should have received a copy of the GNU General Public License
14
15
# along with this program; if not, write to the Free Software
15
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
17
 
17
 
import errno
18
 
import os
19
 
import re
20
 
import subprocess
21
 
import sys
22
 
import tempfile
23
 
import time
24
 
 
25
 
from bzrlib import (
26
 
    errors,
27
 
    osutils,
28
 
    )
29
 
# compatability - plugins import compare_trees from diff!!!
30
 
# deprecated as of 0.10
31
 
from bzrlib.delta import compare_trees
32
 
from bzrlib.errors import BzrError
33
 
from bzrlib.patiencediff import unified_diff
34
 
import bzrlib.patiencediff
35
 
from bzrlib.symbol_versioning import (deprecated_function,
36
 
        zero_eight)
37
 
from bzrlib.textfile import check_text_lines
38
 
from bzrlib.trace import mutter, warning
39
 
 
40
 
 
41
 
# TODO: Rather than building a changeset object, we should probably
42
 
# invoke callbacks on an object.  That object can either accumulate a
43
 
# list, write them out directly, etc etc.
44
 
 
45
 
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file,
46
 
                  allow_binary=False, sequence_matcher=None,
47
 
                  path_encoding='utf8'):
48
 
    # FIXME: difflib is wrong if there is no trailing newline.
49
 
    # The syntax used by patch seems to be "\ No newline at
50
 
    # end of file" following the last diff line from that
51
 
    # file.  This is not trivial to insert into the
52
 
    # unified_diff output and it might be better to just fix
53
 
    # or replace that function.
54
 
 
55
 
    # In the meantime we at least make sure the patch isn't
56
 
    # mangled.
57
 
 
58
 
 
59
 
    # Special workaround for Python2.3, where difflib fails if
60
 
    # both sequences are empty.
61
 
    if not oldlines and not newlines:
62
 
        return
63
 
    
64
 
    if allow_binary is False:
65
 
        check_text_lines(oldlines)
66
 
        check_text_lines(newlines)
67
 
 
68
 
    if sequence_matcher is None:
69
 
        sequence_matcher = bzrlib.patiencediff.PatienceSequenceMatcher
70
 
    ud = unified_diff(oldlines, newlines,
71
 
                      fromfile=old_filename.encode(path_encoding),
72
 
                      tofile=new_filename.encode(path_encoding),
73
 
                      sequencematcher=sequence_matcher)
74
 
 
75
 
    ud = list(ud)
76
 
    # work-around for difflib being too smart for its own good
77
 
    # if /dev/null is "1,0", patch won't recognize it as /dev/null
78
 
    if not oldlines:
79
 
        ud[2] = ud[2].replace('-1,0', '-0,0')
80
 
    elif not newlines:
81
 
        ud[2] = ud[2].replace('+1,0', '+0,0')
82
 
    # work around for difflib emitting random spaces after the label
83
 
    ud[0] = ud[0][:-2] + '\n'
84
 
    ud[1] = ud[1][:-2] + '\n'
85
 
 
86
 
    for line in ud:
87
 
        to_file.write(line)
88
 
        if not line.endswith('\n'):
89
 
            to_file.write("\n\\ No newline at end of file\n")
90
 
    print >>to_file
91
 
 
92
 
 
93
 
def _set_lang_C():
94
 
    """Set the env var LANG=C"""
95
 
    osutils.set_or_unset_env('LANG', 'C')
96
 
    osutils.set_or_unset_env('LC_ALL', None)
97
 
    osutils.set_or_unset_env('LC_CTYPE', None)
98
 
    osutils.set_or_unset_env('LANGUAGE', None)
99
 
 
100
 
 
101
 
def _spawn_external_diff(diffcmd, capture_errors=True):
102
 
    """Spawn the externall diff process, and return the child handle.
103
 
 
104
 
    :param diffcmd: The command list to spawn
105
 
    :param capture_errors: Capture stderr as well as setting LANG=C.
106
 
        This lets us read and understand the output of diff, and respond 
107
 
        to any errors.
108
 
    :return: A Popen object.
109
 
    """
110
 
    if capture_errors:
111
 
        if sys.platform == 'win32':
112
 
            # Win32 doesn't support preexec_fn, but that is
113
 
            # okay, because it doesn't support LANG either.
114
 
            preexec_fn = None
115
 
        else:
116
 
            preexec_fn = _set_lang_C
117
 
        stderr = subprocess.PIPE
118
 
    else:
119
 
        preexec_fn = None
120
 
        stderr = None
121
 
 
122
 
    try:
123
 
        pipe = subprocess.Popen(diffcmd,
124
 
                                stdin=subprocess.PIPE,
125
 
                                stdout=subprocess.PIPE,
126
 
                                stderr=stderr,
127
 
                                preexec_fn=preexec_fn)
128
 
    except OSError, e:
129
 
        if e.errno == errno.ENOENT:
130
 
            raise errors.NoDiff(str(e))
131
 
        raise
132
 
 
133
 
    return pipe
134
 
 
135
 
 
136
 
def external_diff(old_filename, oldlines, new_filename, newlines, to_file,
137
 
                  diff_opts):
138
 
    """Display a diff by calling out to the external diff program."""
139
 
    # make sure our own output is properly ordered before the diff
140
 
    to_file.flush()
141
 
 
142
 
    oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-')
143
 
    newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-')
144
 
    oldtmpf = os.fdopen(oldtmp_fd, 'wb')
145
 
    newtmpf = os.fdopen(newtmp_fd, 'wb')
146
 
 
147
 
    try:
148
 
        # TODO: perhaps a special case for comparing to or from the empty
149
 
        # sequence; can just use /dev/null on Unix
150
 
 
151
 
        # TODO: if either of the files being compared already exists as a
152
 
        # regular named file (e.g. in the working directory) then we can
153
 
        # compare directly to that, rather than copying it.
154
 
 
155
 
        oldtmpf.writelines(oldlines)
156
 
        newtmpf.writelines(newlines)
157
 
 
158
 
        oldtmpf.close()
159
 
        newtmpf.close()
160
 
 
161
 
        if not diff_opts:
162
 
            diff_opts = []
163
 
        diffcmd = ['diff',
164
 
                   '--label', old_filename,
165
 
                   old_abspath,
166
 
                   '--label', new_filename,
167
 
                   new_abspath,
168
 
                   '--binary',
169
 
                  ]
170
 
 
171
 
        # diff only allows one style to be specified; they don't override.
172
 
        # note that some of these take optargs, and the optargs can be
173
 
        # directly appended to the options.
174
 
        # this is only an approximate parser; it doesn't properly understand
175
 
        # the grammar.
176
 
        for s in ['-c', '-u', '-C', '-U',
177
 
                  '-e', '--ed',
178
 
                  '-q', '--brief',
179
 
                  '--normal',
180
 
                  '-n', '--rcs',
181
 
                  '-y', '--side-by-side',
182
 
                  '-D', '--ifdef']:
183
 
            for j in diff_opts:
184
 
                if j.startswith(s):
185
 
                    break
186
 
            else:
187
 
                continue
188
 
            break
189
 
        else:
190
 
            diffcmd.append('-u')
191
 
                  
192
 
        if diff_opts:
193
 
            diffcmd.extend(diff_opts)
194
 
 
195
 
        pipe = _spawn_external_diff(diffcmd, capture_errors=True)
196
 
        out,err = pipe.communicate()
197
 
        rc = pipe.returncode
198
 
        
199
 
        # internal_diff() adds a trailing newline, add one here for consistency
200
 
        out += '\n'
201
 
        if rc == 2:
202
 
            # 'diff' gives retcode == 2 for all sorts of errors
203
 
            # one of those is 'Binary files differ'.
204
 
            # Bad options could also be the problem.
205
 
            # 'Binary files' is not a real error, so we suppress that error.
206
 
            lang_c_out = out
207
 
 
208
 
            # Since we got here, we want to make sure to give an i18n error
209
 
            pipe = _spawn_external_diff(diffcmd, capture_errors=False)
210
 
            out, err = pipe.communicate()
211
 
 
212
 
            # Write out the new i18n diff response
213
 
            to_file.write(out+'\n')
214
 
            if pipe.returncode != 2:
215
 
                raise BzrError('external diff failed with exit code 2'
216
 
                               ' when run with LANG=C, but not when run'
217
 
                               ' natively: %r' % (diffcmd,))
218
 
 
219
 
            first_line = lang_c_out.split('\n', 1)[0]
220
 
            # Starting with diffutils 2.8.4 the word "binary" was dropped.
221
 
            m = re.match('^(binary )?files.*differ$', first_line, re.I)
222
 
            if m is None:
223
 
                raise BzrError('external diff failed with exit code 2;'
224
 
                               ' command: %r' % (diffcmd,))
225
 
            else:
226
 
                # Binary files differ, just return
227
 
                return
228
 
 
229
 
        # If we got to here, we haven't written out the output of diff
230
 
        # do so now
231
 
        to_file.write(out)
232
 
        if rc not in (0, 1):
233
 
            # returns 1 if files differ; that's OK
234
 
            if rc < 0:
235
 
                msg = 'signal %d' % (-rc)
236
 
            else:
237
 
                msg = 'exit code %d' % rc
238
 
                
239
 
            raise BzrError('external diff failed with %s; command: %r' 
240
 
                           % (rc, diffcmd))
241
 
 
242
 
 
243
 
    finally:
244
 
        oldtmpf.close()                 # and delete
245
 
        newtmpf.close()
246
 
        # Clean up. Warn in case the files couldn't be deleted
247
 
        # (in case windows still holds the file open, but not
248
 
        # if the files have already been deleted)
249
 
        try:
250
 
            os.remove(old_abspath)
251
 
        except OSError, e:
252
 
            if e.errno not in (errno.ENOENT,):
253
 
                warning('Failed to delete temporary file: %s %s',
254
 
                        old_abspath, e)
255
 
        try:
256
 
            os.remove(new_abspath)
257
 
        except OSError:
258
 
            if e.errno not in (errno.ENOENT,):
259
 
                warning('Failed to delete temporary file: %s %s',
260
 
                        new_abspath, e)
261
 
 
262
 
 
263
 
@deprecated_function(zero_eight)
264
 
def show_diff(b, from_spec, specific_files, external_diff_options=None,
265
 
              revision2=None, output=None, b2=None):
266
 
    """Shortcut for showing the diff to the working tree.
267
 
 
268
 
    Please use show_diff_trees instead.
269
 
 
270
 
    b
271
 
        Branch.
272
 
 
273
 
    revision
274
 
        None for 'basis tree', or otherwise the old revision to compare against.
275
 
    
276
 
    The more general form is show_diff_trees(), where the caller
277
 
    supplies any two trees.
278
 
    """
279
 
    if output is None:
280
 
        output = sys.stdout
281
 
 
282
 
    if from_spec is None:
283
 
        old_tree = b.bzrdir.open_workingtree()
284
 
        if b2 is None:
285
 
            old_tree = old_tree = old_tree.basis_tree()
286
 
    else:
287
 
        old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id)
288
 
 
289
 
    if revision2 is None:
290
 
        if b2 is None:
291
 
            new_tree = b.bzrdir.open_workingtree()
292
 
        else:
293
 
            new_tree = b2.bzrdir.open_workingtree()
294
 
    else:
295
 
        new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id)
296
 
 
297
 
    return show_diff_trees(old_tree, new_tree, output, specific_files,
298
 
                           external_diff_options)
299
 
 
300
 
 
301
 
def diff_cmd_helper(tree, specific_files, external_diff_options, 
302
 
                    old_revision_spec=None, new_revision_spec=None,
303
 
                    old_label='a/', new_label='b/'):
304
 
    """Helper for cmd_diff.
305
 
 
306
 
   tree 
307
 
        A WorkingTree
308
 
 
309
 
    specific_files
310
 
        The specific files to compare, or None
311
 
 
312
 
    external_diff_options
313
 
        If non-None, run an external diff, and pass it these options
314
 
 
315
 
    old_revision_spec
316
 
        If None, use basis tree as old revision, otherwise use the tree for
317
 
        the specified revision. 
318
 
 
319
 
    new_revision_spec
320
 
        If None, use working tree as new revision, otherwise use the tree for
321
 
        the specified revision.
322
 
    
323
 
    The more general form is show_diff_trees(), where the caller
324
 
    supplies any two trees.
325
 
    """
326
 
    def spec_tree(spec):
327
 
        if tree:
328
 
            revision = spec.in_store(tree.branch)
329
 
        else:
330
 
            revision = spec.in_store(None)
331
 
        revision_id = revision.rev_id
332
 
        branch = revision.branch
333
 
        return branch.repository.revision_tree(revision_id)
334
 
    if old_revision_spec is None:
335
 
        old_tree = tree.basis_tree()
336
 
    else:
337
 
        old_tree = spec_tree(old_revision_spec)
338
 
 
339
 
    if new_revision_spec is None:
340
 
        new_tree = tree
341
 
    else:
342
 
        new_tree = spec_tree(new_revision_spec)
343
 
    if new_tree is not tree:
344
 
        extra_trees = (tree,)
345
 
    else:
346
 
        extra_trees = None
347
 
 
348
 
    return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files,
349
 
                           external_diff_options,
350
 
                           old_label=old_label, new_label=new_label,
351
 
                           extra_trees=extra_trees)
352
 
 
353
 
 
354
 
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None,
355
 
                    external_diff_options=None,
356
 
                    old_label='a/', new_label='b/',
357
 
                    extra_trees=None):
358
 
    """Show in text form the changes from one tree to another.
359
 
 
360
 
    to_files
361
 
        If set, include only changes to these files.
362
 
 
363
 
    external_diff_options
364
 
        If set, use an external GNU diff and pass these options.
365
 
 
366
 
    extra_trees
367
 
        If set, more Trees to use for looking up file ids
368
 
    """
369
 
    old_tree.lock_read()
370
 
    try:
371
 
        new_tree.lock_read()
372
 
        try:
373
 
            return _show_diff_trees(old_tree, new_tree, to_file,
374
 
                                    specific_files, external_diff_options,
375
 
                                    old_label=old_label, new_label=new_label,
376
 
                                    extra_trees=extra_trees)
377
 
        finally:
378
 
            new_tree.unlock()
379
 
    finally:
380
 
        old_tree.unlock()
381
 
 
382
 
 
383
 
def _show_diff_trees(old_tree, new_tree, to_file,
384
 
                     specific_files, external_diff_options, 
385
 
                     old_label='a/', new_label='b/', extra_trees=None):
386
 
 
387
 
    # GNU Patch uses the epoch date to detect files that are being added
388
 
    # or removed in a diff.
389
 
    EPOCH_DATE = '1970-01-01 00:00:00 +0000'
390
 
 
391
 
    # TODO: Generation of pseudo-diffs for added/deleted files could
392
 
    # be usefully made into a much faster special case.
393
 
 
394
 
    if external_diff_options:
395
 
        assert isinstance(external_diff_options, basestring)
396
 
        opts = external_diff_options.split()
397
 
        def diff_file(olab, olines, nlab, nlines, to_file):
398
 
            external_diff(olab, olines, nlab, nlines, to_file, opts)
399
 
    else:
400
 
        diff_file = internal_diff
401
 
    
402
 
    delta = new_tree.changes_from(old_tree,
403
 
        specific_files=specific_files,
404
 
        extra_trees=extra_trees, require_versioned=True)
405
 
 
406
 
    has_changes = 0
407
 
    for path, file_id, kind in delta.removed:
408
 
        has_changes = 1
409
 
        print >>to_file, '=== removed %s %r' % (kind, path.encode('utf8'))
410
 
        old_name = '%s%s\t%s' % (old_label, path,
411
 
                                 _patch_header_date(old_tree, file_id, path))
412
 
        new_name = '%s%s\t%s' % (new_label, path, EPOCH_DATE)
413
 
        old_tree.inventory[file_id].diff(diff_file, old_name, old_tree,
414
 
                                         new_name, None, None, to_file)
415
 
    for path, file_id, kind in delta.added:
416
 
        has_changes = 1
417
 
        print >>to_file, '=== added %s %r' % (kind, path.encode('utf8'))
418
 
        old_name = '%s%s\t%s' % (old_label, path, EPOCH_DATE)
419
 
        new_name = '%s%s\t%s' % (new_label, path,
420
 
                                 _patch_header_date(new_tree, file_id, path))
421
 
        new_tree.inventory[file_id].diff(diff_file, new_name, new_tree,
422
 
                                         old_name, None, None, to_file, 
423
 
                                         reverse=True)
424
 
    for (old_path, new_path, file_id, kind,
425
 
         text_modified, meta_modified) in delta.renamed:
426
 
        has_changes = 1
427
 
        prop_str = get_prop_change(meta_modified)
428
 
        print >>to_file, '=== renamed %s %r => %r%s' % (
429
 
                    kind, old_path.encode('utf8'),
430
 
                    new_path.encode('utf8'), prop_str)
431
 
        old_name = '%s%s\t%s' % (old_label, old_path,
432
 
                                 _patch_header_date(old_tree, file_id,
433
 
                                                    old_path))
434
 
        new_name = '%s%s\t%s' % (new_label, new_path,
435
 
                                 _patch_header_date(new_tree, file_id,
436
 
                                                    new_path))
437
 
        _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
438
 
                                    new_name, new_tree,
439
 
                                    text_modified, kind, to_file, diff_file)
440
 
    for path, file_id, kind, text_modified, meta_modified in delta.modified:
441
 
        has_changes = 1
442
 
        prop_str = get_prop_change(meta_modified)
443
 
        print >>to_file, '=== modified %s %r%s' % (kind, path.encode('utf8'), prop_str)
444
 
        old_name = '%s%s\t%s' % (old_label, path,
445
 
                                 _patch_header_date(old_tree, file_id, path))
446
 
        new_name = '%s%s\t%s' % (new_label, path,
447
 
                                 _patch_header_date(new_tree, file_id, path))
448
 
        if text_modified:
449
 
            _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
450
 
                                        new_name, new_tree,
451
 
                                        True, kind, to_file, diff_file)
452
 
 
453
 
    return has_changes
454
 
 
455
 
 
456
 
def _patch_header_date(tree, file_id, path):
457
 
    """Returns a timestamp suitable for use in a patch header."""
458
 
    tm = time.gmtime(tree.get_file_mtime(file_id, path))
459
 
    return time.strftime('%Y-%m-%d %H:%M:%S +0000', tm)
460
 
 
461
 
 
462
 
def _raise_if_nonexistent(paths, old_tree, new_tree):
463
 
    """Complain if paths are not in either inventory or tree.
464
 
 
465
 
    It's OK with the files exist in either tree's inventory, or 
466
 
    if they exist in the tree but are not versioned.
467
 
    
468
 
    This can be used by operations such as bzr status that can accept
469
 
    unknown or ignored files.
470
 
    """
471
 
    mutter("check paths: %r", paths)
472
 
    if not paths:
473
 
        return
474
 
    s = old_tree.filter_unversioned_files(paths)
475
 
    s = new_tree.filter_unversioned_files(s)
476
 
    s = [path for path in s if not new_tree.has_filename(path)]
477
 
    if s:
478
 
        raise errors.PathsDoNotExist(sorted(s))
479
 
 
480
 
 
481
 
def get_prop_change(meta_modified):
482
 
    if meta_modified:
483
 
        return " (properties changed)"
484
 
    else:
485
 
        return  ""
486
 
 
487
 
 
488
 
def _maybe_diff_file_or_symlink(old_path, old_tree, file_id,
489
 
                                new_path, new_tree, text_modified,
490
 
                                kind, to_file, diff_file):
491
 
    if text_modified:
492
 
        new_entry = new_tree.inventory[file_id]
493
 
        old_tree.inventory[file_id].diff(diff_file,
494
 
                                         old_path, old_tree,
495
 
                                         new_path, new_entry, 
496
 
                                         new_tree, to_file)
 
18
from sets import Set
 
19
 
 
20
from trace import mutter
 
21
 
 
22
 
 
23
 
 
24
 
 
25
 
 
26
 
 
27
def diff_trees(old_tree, new_tree):
 
28
    """Compute diff between two trees.
 
29
 
 
30
    They may be in different branches and may be working or historical
 
31
    trees.
 
32
 
 
33
    Yields a sequence of (state, id, old_name, new_name, kind).
 
34
    Each filename and each id is listed only once.
 
35
    """
 
36
 
 
37
    ## TODO: Compare files before diffing; only mention those that have changed
 
38
 
 
39
    ## TODO: Set nice names in the headers, maybe include diffstat
 
40
 
 
41
    ## TODO: Perhaps make this a generator rather than using
 
42
    ## a callback object?
 
43
 
 
44
    ## TODO: Allow specifying a list of files to compare, rather than
 
45
    ## doing the whole tree?  (Not urgent.)
 
46
 
 
47
    ## TODO: Allow diffing any two inventories, not just the
 
48
    ## current one against one.  We mgiht need to specify two
 
49
    ## stores to look for the files if diffing two branches.  That
 
50
    ## might imply this shouldn't be primarily a Branch method.
 
51
 
 
52
    ## XXX: This doesn't report on unknown files; that can be done
 
53
    ## from a separate method.
 
54
 
 
55
    old_it = old_tree.list_files()
 
56
    new_it = new_tree.list_files()
 
57
 
 
58
    def next(it):
 
59
        try:
 
60
            return it.next()
 
61
        except StopIteration:
 
62
            return None
 
63
 
 
64
    old_item = next(old_it)
 
65
    new_item = next(new_it)
 
66
 
 
67
    # We step through the two sorted iterators in parallel, trying to
 
68
    # keep them lined up.
 
69
 
 
70
    while (old_item != None) or (new_item != None):
 
71
        # OK, we still have some remaining on both, but they may be
 
72
        # out of step.        
 
73
        if old_item != None:
 
74
            old_name, old_class, old_kind, old_id = old_item
 
75
        else:
 
76
            old_name = None
 
77
            
 
78
        if new_item != None:
 
79
            new_name, new_class, new_kind, new_id = new_item
 
80
        else:
 
81
            new_name = None
 
82
 
 
83
        mutter("   diff pairwise %r" % (old_item,))
 
84
        mutter("                 %r" % (new_item,))
 
85
 
 
86
        if old_item:
 
87
            # can't handle the old tree being a WorkingTree
 
88
            assert old_class == 'V'
 
89
 
 
90
        if new_item and (new_class != 'V'):
 
91
            yield new_class, None, None, new_name, new_kind
 
92
            new_item = next(new_it)
 
93
        elif (not new_item) or (old_item and (old_name < new_name)):
 
94
            mutter("     extra entry in old-tree sequence")
 
95
            if new_tree.has_id(old_id):
 
96
                # will be mentioned as renamed under new name
 
97
                pass
 
98
            else:
 
99
                yield 'D', old_id, old_name, None, old_kind
 
100
            old_item = next(old_it)
 
101
        elif (not old_item) or (new_item and (new_name < old_name)):
 
102
            mutter("     extra entry in new-tree sequence")
 
103
            if old_tree.has_id(new_id):
 
104
                yield 'R', new_id, old_tree.id2path(new_id), new_name, new_kind
 
105
            else:
 
106
                yield 'A', new_id, None, new_name, new_kind
 
107
            new_item = next(new_it)
 
108
        elif old_id != new_id:
 
109
            assert old_name == new_name
 
110
            # both trees have a file of this name, but it is not the
 
111
            # same file.  in other words, the old filename has been
 
112
            # overwritten by either a newly-added or a renamed file.
 
113
            # (should we return something about the overwritten file?)
 
114
            if old_tree.has_id(new_id):
 
115
                # renaming, overlying a deleted file
 
116
                yield 'R', new_id, old_tree.id2path(new_id), new_name, new_kind
 
117
            else:
 
118
                yield 'A', new_id, None, new_name, new_kind
 
119
 
 
120
            new_item = next(new_it)
 
121
            old_item = next(old_it)
 
122
        else:
 
123
            assert old_id == new_id
 
124
            assert old_name == new_name
 
125
            assert old_kind == new_kind
 
126
 
 
127
            if old_kind == 'directory':
 
128
                yield '.', new_id, old_name, new_name, new_kind
 
129
            elif old_tree.get_file_size(old_id) != new_tree.get_file_size(old_id):
 
130
                mutter("    file size has changed, must be different")
 
131
                yield 'M', new_id, old_name, new_name, new_kind
 
132
            elif old_tree.get_file_sha1(old_id) == new_tree.get_file_sha1(old_id):
 
133
                mutter("      SHA1 indicates they're identical")
 
134
                ## assert compare_files(old_tree.get_file(i), new_tree.get_file(i))
 
135
                yield '.', new_id, old_name, new_name, new_kind
 
136
            else:
 
137
                mutter("      quick compare shows different")
 
138
                yield 'M', new_id, old_name, new_name, new_kind
 
139
 
 
140
            new_item = next(new_it)
 
141
            old_item = next(old_it)
 
142
 
 
143