~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/diff.py

  • Committer: Martin Pool
  • Date: 2005-05-12 02:19:09 UTC
  • Revision ID: mbp@sourcefrog.net-20050512021909-acb201f8ffc52b41
doc

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
 
# compatability - plugins import compare_trees from diff!!!
26
 
# deprecated as of 0.10
27
 
from bzrlib.delta import compare_trees
28
 
from bzrlib.errors import BzrError
29
 
import bzrlib.errors as errors
30
 
import bzrlib.osutils
31
 
from bzrlib.patiencediff import unified_diff
32
 
import bzrlib.patiencediff
33
 
from bzrlib.symbol_versioning import (deprecated_function,
34
 
        zero_eight)
35
 
from bzrlib.textfile import check_text_lines
36
 
from bzrlib.trace import mutter, warning
37
 
 
38
 
 
39
 
# TODO: Rather than building a changeset object, we should probably
40
 
# invoke callbacks on an object.  That object can either accumulate a
41
 
# list, write them out directly, etc etc.
42
 
 
43
 
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file,
44
 
                  allow_binary=False, sequence_matcher=None,
45
 
                  path_encoding='utf8'):
 
18
from sets import Set, ImmutableSet
 
19
 
 
20
from trace import mutter
 
21
from errors import BzrError
 
22
 
 
23
 
 
24
 
 
25
def _diff_one(oldlines, newlines, to_file, **kw):
 
26
    import difflib
 
27
    
46
28
    # FIXME: difflib is wrong if there is no trailing newline.
47
29
    # The syntax used by patch seems to be "\ No newline at
48
30
    # end of file" following the last diff line from that
58
40
    # both sequences are empty.
59
41
    if not oldlines and not newlines:
60
42
        return
61
 
    
62
 
    if allow_binary is False:
63
 
        check_text_lines(oldlines)
64
 
        check_text_lines(newlines)
65
 
 
66
 
    if sequence_matcher is None:
67
 
        sequence_matcher = bzrlib.patiencediff.PatienceSequenceMatcher
68
 
    ud = unified_diff(oldlines, newlines,
69
 
                      fromfile=old_filename.encode(path_encoding),
70
 
                      tofile=new_filename.encode(path_encoding),
71
 
                      sequencematcher=sequence_matcher)
72
 
 
73
 
    ud = list(ud)
 
43
 
 
44
    nonl = False
 
45
 
 
46
    if oldlines and (oldlines[-1][-1] != '\n'):
 
47
        oldlines[-1] += '\n'
 
48
        nonl = True
 
49
    if newlines and (newlines[-1][-1] != '\n'):
 
50
        newlines[-1] += '\n'
 
51
        nonl = True
 
52
 
 
53
    ud = difflib.unified_diff(oldlines, newlines, **kw)
 
54
 
74
55
    # work-around for difflib being too smart for its own good
75
56
    # if /dev/null is "1,0", patch won't recognize it as /dev/null
76
57
    if not oldlines:
 
58
        ud = list(ud)
77
59
        ud[2] = ud[2].replace('-1,0', '-0,0')
78
60
    elif not newlines:
 
61
        ud = list(ud)
79
62
        ud[2] = ud[2].replace('+1,0', '+0,0')
80
 
    # work around for difflib emitting random spaces after the label
81
 
    ud[0] = ud[0][:-2] + '\n'
82
 
    ud[1] = ud[1][:-2] + '\n'
83
63
 
84
 
    for line in ud:
85
 
        to_file.write(line)
86
 
        if not line.endswith('\n'):
87
 
            to_file.write("\n\\ No newline at end of file\n")
 
64
    to_file.writelines(ud)
 
65
    if nonl:
 
66
        print >>to_file, "\\ No newline at end of file"
88
67
    print >>to_file
89
68
 
90
69
 
91
 
def external_diff(old_filename, oldlines, new_filename, newlines, to_file,
92
 
                  diff_opts):
93
 
    """Display a diff by calling out to the external diff program."""
94
 
    # make sure our own output is properly ordered before the diff
95
 
    to_file.flush()
96
 
 
97
 
    oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-')
98
 
    newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-')
99
 
    oldtmpf = os.fdopen(oldtmp_fd, 'wb')
100
 
    newtmpf = os.fdopen(newtmp_fd, 'wb')
101
 
 
102
 
    try:
103
 
        # TODO: perhaps a special case for comparing to or from the empty
104
 
        # sequence; can just use /dev/null on Unix
105
 
 
106
 
        # TODO: if either of the files being compared already exists as a
107
 
        # regular named file (e.g. in the working directory) then we can
108
 
        # compare directly to that, rather than copying it.
109
 
 
110
 
        oldtmpf.writelines(oldlines)
111
 
        newtmpf.writelines(newlines)
112
 
 
113
 
        oldtmpf.close()
114
 
        newtmpf.close()
115
 
 
116
 
        if not diff_opts:
117
 
            diff_opts = []
118
 
        diffcmd = ['diff',
119
 
                   '--label', old_filename,
120
 
                   old_abspath,
121
 
                   '--label', new_filename,
122
 
                   new_abspath,
123
 
                   '--binary',
124
 
                  ]
125
 
 
126
 
        # diff only allows one style to be specified; they don't override.
127
 
        # note that some of these take optargs, and the optargs can be
128
 
        # directly appended to the options.
129
 
        # this is only an approximate parser; it doesn't properly understand
130
 
        # the grammar.
131
 
        for s in ['-c', '-u', '-C', '-U',
132
 
                  '-e', '--ed',
133
 
                  '-q', '--brief',
134
 
                  '--normal',
135
 
                  '-n', '--rcs',
136
 
                  '-y', '--side-by-side',
137
 
                  '-D', '--ifdef']:
138
 
            for j in diff_opts:
139
 
                if j.startswith(s):
140
 
                    break
141
 
            else:
142
 
                continue
143
 
            break
144
 
        else:
145
 
            diffcmd.append('-u')
146
 
                  
147
 
        if diff_opts:
148
 
            diffcmd.extend(diff_opts)
149
 
 
150
 
        try:
151
 
            pipe = subprocess.Popen(diffcmd,
152
 
                                    stdin=subprocess.PIPE,
153
 
                                    stdout=subprocess.PIPE)
154
 
        except OSError, e:
155
 
            if e.errno == errno.ENOENT:
156
 
                raise errors.NoDiff(str(e))
157
 
            raise
158
 
        pipe.stdin.close()
159
 
 
160
 
        first_line = pipe.stdout.readline()
161
 
        to_file.write(first_line)
162
 
        bzrlib.osutils.pumpfile(pipe.stdout, to_file)
163
 
        rc = pipe.wait()
 
70
def show_diff(b, revision, specific_files):
 
71
    import sys
 
72
 
 
73
    if revision == None:
 
74
        old_tree = b.basis_tree()
 
75
    else:
 
76
        old_tree = b.revision_tree(b.lookup_revision(revision))
164
77
        
165
 
        if rc == 2:
166
 
            # 'diff' gives retcode == 2 for all sorts of errors
167
 
            # one of those is 'Binary files differ'.
168
 
            # Bad options could also be the problem.
169
 
            # 'Binary files' is not a real error, so we suppress that error
170
 
            m = re.match('^binary files.*differ$', first_line, re.I)
171
 
            if not m:
172
 
                raise BzrError('external diff failed with exit code 2;'
173
 
                               ' command: %r' % (diffcmd,))
174
 
        elif rc not in (0, 1):
175
 
            # returns 1 if files differ; that's OK
176
 
            if rc < 0:
177
 
                msg = 'signal %d' % (-rc)
178
 
            else:
179
 
                msg = 'exit code %d' % rc
180
 
                
181
 
            raise BzrError('external diff failed with %s; command: %r' 
182
 
                           % (rc, diffcmd))
183
 
 
184
 
        # internal_diff() adds a trailing newline, add one here for consistency
185
 
        to_file.write('\n')
186
 
 
187
 
    finally:
188
 
        oldtmpf.close()                 # and delete
189
 
        newtmpf.close()
190
 
        # Clean up. Warn in case the files couldn't be deleted
191
 
        # (in case windows still holds the file open, but not
192
 
        # if the files have already been deleted)
193
 
        try:
194
 
            os.remove(old_abspath)
195
 
        except OSError, e:
196
 
            if e.errno not in (errno.ENOENT,):
197
 
                warning('Failed to delete temporary file: %s %s',
198
 
                        old_abspath, e)
199
 
        try:
200
 
            os.remove(new_abspath)
201
 
        except OSError:
202
 
            if e.errno not in (errno.ENOENT,):
203
 
                warning('Failed to delete temporary file: %s %s',
204
 
                        new_abspath, e)
205
 
 
206
 
 
207
 
@deprecated_function(zero_eight)
208
 
def show_diff(b, from_spec, specific_files, external_diff_options=None,
209
 
              revision2=None, output=None, b2=None):
210
 
    """Shortcut for showing the diff to the working tree.
211
 
 
212
 
    Please use show_diff_trees instead.
213
 
 
214
 
    b
215
 
        Branch.
216
 
 
217
 
    revision
218
 
        None for 'basis tree', or otherwise the old revision to compare against.
219
 
    
220
 
    The more general form is show_diff_trees(), where the caller
221
 
    supplies any two trees.
222
 
    """
223
 
    if output is None:
224
 
        output = sys.stdout
225
 
 
226
 
    if from_spec is None:
227
 
        old_tree = b.bzrdir.open_workingtree()
228
 
        if b2 is None:
229
 
            old_tree = old_tree = old_tree.basis_tree()
230
 
    else:
231
 
        old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id)
232
 
 
233
 
    if revision2 is None:
234
 
        if b2 is None:
235
 
            new_tree = b.bzrdir.open_workingtree()
236
 
        else:
237
 
            new_tree = b2.bzrdir.open_workingtree()
238
 
    else:
239
 
        new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id)
240
 
 
241
 
    return show_diff_trees(old_tree, new_tree, output, specific_files,
242
 
                           external_diff_options)
243
 
 
244
 
 
245
 
def diff_cmd_helper(tree, specific_files, external_diff_options, 
246
 
                    old_revision_spec=None, new_revision_spec=None,
247
 
                    old_label='a/', new_label='b/'):
248
 
    """Helper for cmd_diff.
249
 
 
250
 
   tree 
251
 
        A WorkingTree
252
 
 
253
 
    specific_files
254
 
        The specific files to compare, or None
255
 
 
256
 
    external_diff_options
257
 
        If non-None, run an external diff, and pass it these options
258
 
 
259
 
    old_revision_spec
260
 
        If None, use basis tree as old revision, otherwise use the tree for
261
 
        the specified revision. 
262
 
 
263
 
    new_revision_spec
264
 
        If None, use working tree as new revision, otherwise use the tree for
265
 
        the specified revision.
266
 
    
267
 
    The more general form is show_diff_trees(), where the caller
268
 
    supplies any two trees.
269
 
    """
270
 
    def spec_tree(spec):
271
 
        if tree:
272
 
            revision = spec.in_store(tree.branch)
273
 
        else:
274
 
            revision = spec.in_store(None)
275
 
        revision_id = revision.rev_id
276
 
        branch = revision.branch
277
 
        return branch.repository.revision_tree(revision_id)
278
 
    if old_revision_spec is None:
279
 
        old_tree = tree.basis_tree()
280
 
    else:
281
 
        old_tree = spec_tree(old_revision_spec)
282
 
 
283
 
    if new_revision_spec is None:
284
 
        new_tree = tree
285
 
    else:
286
 
        new_tree = spec_tree(new_revision_spec)
287
 
    if new_tree is not tree:
288
 
        extra_trees = (tree,)
289
 
    else:
290
 
        extra_trees = None
291
 
 
292
 
    return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files,
293
 
                           external_diff_options,
294
 
                           old_label=old_label, new_label=new_label,
295
 
                           extra_trees=extra_trees)
296
 
 
297
 
 
298
 
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None,
299
 
                    external_diff_options=None,
300
 
                    old_label='a/', new_label='b/',
301
 
                    extra_trees=None):
302
 
    """Show in text form the changes from one tree to another.
303
 
 
304
 
    to_files
305
 
        If set, include only changes to these files.
306
 
 
307
 
    external_diff_options
308
 
        If set, use an external GNU diff and pass these options.
309
 
 
310
 
    extra_trees
311
 
        If set, more Trees to use for looking up file ids
312
 
    """
313
 
    old_tree.lock_read()
314
 
    try:
315
 
        new_tree.lock_read()
316
 
        try:
317
 
            return _show_diff_trees(old_tree, new_tree, to_file,
318
 
                                    specific_files, external_diff_options,
319
 
                                    old_label=old_label, new_label=new_label,
320
 
                                    extra_trees=extra_trees)
321
 
        finally:
322
 
            new_tree.unlock()
323
 
    finally:
324
 
        old_tree.unlock()
325
 
 
326
 
 
327
 
def _show_diff_trees(old_tree, new_tree, to_file,
328
 
                     specific_files, external_diff_options, 
329
 
                     old_label='a/', new_label='b/', extra_trees=None):
330
 
 
331
 
    # GNU Patch uses the epoch date to detect files that are being added
332
 
    # or removed in a diff.
333
 
    EPOCH_DATE = '1970-01-01 00:00:00 +0000'
 
78
    new_tree = b.working_tree()
 
79
 
 
80
    # TODO: Options to control putting on a prefix or suffix, perhaps as a format string
 
81
    old_label = ''
 
82
    new_label = ''
 
83
 
 
84
    DEVNULL = '/dev/null'
 
85
    # Windows users, don't panic about this filename -- it is a
 
86
    # special signal to GNU patch that the file should be created or
 
87
    # deleted respectively.
334
88
 
335
89
    # TODO: Generation of pseudo-diffs for added/deleted files could
336
90
    # be usefully made into a much faster special case.
337
91
 
338
 
    if external_diff_options:
339
 
        assert isinstance(external_diff_options, basestring)
340
 
        opts = external_diff_options.split()
341
 
        def diff_file(olab, olines, nlab, nlines, to_file):
342
 
            external_diff(olab, olines, nlab, nlines, to_file, opts)
343
 
    else:
344
 
        diff_file = internal_diff
345
 
    
346
 
    delta = new_tree.changes_from(old_tree,
347
 
        specific_files=specific_files,
348
 
        extra_trees=extra_trees, require_versioned=True)
 
92
    delta = compare_trees(old_tree, new_tree, want_unchanged=False,
 
93
                          specific_files=specific_files)
349
94
 
350
 
    has_changes = 0
351
95
    for path, file_id, kind in delta.removed:
352
 
        has_changes = 1
353
 
        print >>to_file, '=== removed %s %r' % (kind, path.encode('utf8'))
354
 
        old_name = '%s%s\t%s' % (old_label, path,
355
 
                                 _patch_header_date(old_tree, file_id, path))
356
 
        new_name = '%s%s\t%s' % (new_label, path, EPOCH_DATE)
357
 
        old_tree.inventory[file_id].diff(diff_file, old_name, old_tree,
358
 
                                         new_name, None, None, to_file)
 
96
        print '*** removed %s %r' % (kind, path)
 
97
        if kind == 'file':
 
98
            _diff_one(old_tree.get_file(file_id).readlines(),
 
99
                   [],
 
100
                   sys.stdout,
 
101
                   fromfile=old_label + path,
 
102
                   tofile=DEVNULL)
 
103
 
359
104
    for path, file_id, kind in delta.added:
360
 
        has_changes = 1
361
 
        print >>to_file, '=== added %s %r' % (kind, path.encode('utf8'))
362
 
        old_name = '%s%s\t%s' % (old_label, path, EPOCH_DATE)
363
 
        new_name = '%s%s\t%s' % (new_label, path,
364
 
                                 _patch_header_date(new_tree, file_id, path))
365
 
        new_tree.inventory[file_id].diff(diff_file, new_name, new_tree,
366
 
                                         old_name, None, None, to_file, 
367
 
                                         reverse=True)
368
 
    for (old_path, new_path, file_id, kind,
369
 
         text_modified, meta_modified) in delta.renamed:
370
 
        has_changes = 1
371
 
        prop_str = get_prop_change(meta_modified)
372
 
        print >>to_file, '=== renamed %s %r => %r%s' % (
373
 
                    kind, old_path.encode('utf8'),
374
 
                    new_path.encode('utf8'), prop_str)
375
 
        old_name = '%s%s\t%s' % (old_label, old_path,
376
 
                                 _patch_header_date(old_tree, file_id,
377
 
                                                    old_path))
378
 
        new_name = '%s%s\t%s' % (new_label, new_path,
379
 
                                 _patch_header_date(new_tree, file_id,
380
 
                                                    new_path))
381
 
        _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
382
 
                                    new_name, new_tree,
383
 
                                    text_modified, kind, to_file, diff_file)
384
 
    for path, file_id, kind, text_modified, meta_modified in delta.modified:
385
 
        has_changes = 1
386
 
        prop_str = get_prop_change(meta_modified)
387
 
        print >>to_file, '=== modified %s %r%s' % (kind, path.encode('utf8'), prop_str)
388
 
        old_name = '%s%s\t%s' % (old_label, path,
389
 
                                 _patch_header_date(old_tree, file_id, path))
390
 
        new_name = '%s%s\t%s' % (new_label, path,
391
 
                                 _patch_header_date(new_tree, file_id, path))
 
105
        print '*** added %s %r' % (kind, path)
 
106
        if kind == 'file':
 
107
            _diff_one([],
 
108
                   new_tree.get_file(file_id).readlines(),
 
109
                   sys.stdout,
 
110
                   fromfile=DEVNULL,
 
111
                   tofile=new_label + path)
 
112
 
 
113
    for old_path, new_path, file_id, kind, text_modified in delta.renamed:
 
114
        print '*** renamed %s %r => %r' % (kind, old_path, new_path)
392
115
        if text_modified:
393
 
            _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
394
 
                                        new_name, new_tree,
395
 
                                        True, kind, to_file, diff_file)
396
 
 
397
 
    return has_changes
398
 
 
399
 
 
400
 
def _patch_header_date(tree, file_id, path):
401
 
    """Returns a timestamp suitable for use in a patch header."""
402
 
    tm = time.gmtime(tree.get_file_mtime(file_id, path))
403
 
    return time.strftime('%Y-%m-%d %H:%M:%S +0000', tm)
404
 
 
405
 
 
406
 
def _raise_if_nonexistent(paths, old_tree, new_tree):
407
 
    """Complain if paths are not in either inventory or tree.
408
 
 
409
 
    It's OK with the files exist in either tree's inventory, or 
410
 
    if they exist in the tree but are not versioned.
 
116
            _diff_one(old_tree.get_file(file_id).readlines(),
 
117
                   new_tree.get_file(file_id).readlines(),
 
118
                   sys.stdout,
 
119
                   fromfile=old_label + old_path,
 
120
                   tofile=new_label + new_path)
 
121
 
 
122
    for path, file_id, kind in delta.modified:
 
123
        print '*** modified %s %r' % (kind, path)
 
124
        if kind == 'file':
 
125
            _diff_one(old_tree.get_file(file_id).readlines(),
 
126
                   new_tree.get_file(file_id).readlines(),
 
127
                   sys.stdout,
 
128
                   fromfile=old_label + path,
 
129
                   tofile=new_label + path)
 
130
 
 
131
 
 
132
 
 
133
class TreeDelta:
 
134
    """Describes changes from one tree to another.
 
135
 
 
136
    Contains four lists:
 
137
 
 
138
    added
 
139
        (path, id, kind)
 
140
    removed
 
141
        (path, id, kind)
 
142
    renamed
 
143
        (oldpath, newpath, id, kind, text_modified)
 
144
    modified
 
145
        (path, id, kind)
 
146
    unchanged
 
147
        (path, id, kind)
 
148
 
 
149
    Each id is listed only once.
 
150
 
 
151
    Files that are both modified and renamed are listed only in
 
152
    renamed, with the text_modified flag true.
 
153
 
 
154
    The lists are normally sorted when the delta is created.
 
155
    """
 
156
    def __init__(self):
 
157
        self.added = []
 
158
        self.removed = []
 
159
        self.renamed = []
 
160
        self.modified = []
 
161
        self.unchanged = []
 
162
 
 
163
    def show(self, to_file, show_ids=False, show_unchanged=False):
 
164
        def show_list(files):
 
165
            for path, fid, kind in files:
 
166
                if kind == 'directory':
 
167
                    path += '/'
 
168
                elif kind == 'symlink':
 
169
                    path += '@'
 
170
                    
 
171
                if show_ids:
 
172
                    print >>to_file, '  %-30s %s' % (path, fid)
 
173
                else:
 
174
                    print >>to_file, ' ', path
 
175
            
 
176
        if self.removed:
 
177
            print >>to_file, 'removed:'
 
178
            show_list(self.removed)
 
179
                
 
180
        if self.added:
 
181
            print >>to_file, 'added:'
 
182
            show_list(self.added)
 
183
 
 
184
        if self.renamed:
 
185
            print >>to_file, 'renamed:'
 
186
            for oldpath, newpath, fid, kind, text_modified in self.renamed:
 
187
                if show_ids:
 
188
                    print >>to_file, '  %s => %s %s' % (oldpath, newpath, fid)
 
189
                else:
 
190
                    print >>to_file, '  %s => %s' % (oldpath, newpath)
 
191
                    
 
192
        if self.modified:
 
193
            print >>to_file, 'modified:'
 
194
            show_list(self.modified)
 
195
            
 
196
        if show_unchanged and self.unchanged:
 
197
            print >>to_file, 'unchanged:'
 
198
            show_list(self.unchanged)
 
199
 
 
200
 
 
201
 
 
202
def compare_trees(old_tree, new_tree, want_unchanged, specific_files=None):
 
203
    """Describe changes from one tree to another.
 
204
 
 
205
    Returns a TreeDelta with details of added, modified, renamed, and
 
206
    deleted entries.
 
207
 
 
208
    The root entry is specifically exempt.
 
209
 
 
210
    This only considers versioned files.
 
211
 
 
212
    want_unchanged
 
213
        If true, also list files unchanged from one version to
 
214
        the next.
 
215
 
 
216
    specific_files
 
217
        If true, only check for changes to specified names or
 
218
        files within them.
 
219
    """
 
220
 
 
221
    from osutils import is_inside_any
411
222
    
412
 
    This can be used by operations such as bzr status that can accept
413
 
    unknown or ignored files.
414
 
    """
415
 
    mutter("check paths: %r", paths)
416
 
    if not paths:
417
 
        return
418
 
    s = old_tree.filter_unversioned_files(paths)
419
 
    s = new_tree.filter_unversioned_files(s)
420
 
    s = [path for path in s if not new_tree.has_filename(path)]
421
 
    if s:
422
 
        raise errors.PathsDoNotExist(sorted(s))
423
 
 
424
 
 
425
 
def get_prop_change(meta_modified):
426
 
    if meta_modified:
427
 
        return " (properties changed)"
428
 
    else:
429
 
        return  ""
430
 
 
431
 
 
432
 
def _maybe_diff_file_or_symlink(old_path, old_tree, file_id,
433
 
                                new_path, new_tree, text_modified,
434
 
                                kind, to_file, diff_file):
435
 
    if text_modified:
436
 
        new_entry = new_tree.inventory[file_id]
437
 
        old_tree.inventory[file_id].diff(diff_file,
438
 
                                         old_path, old_tree,
439
 
                                         new_path, new_entry, 
440
 
                                         new_tree, to_file)
 
223
    old_inv = old_tree.inventory
 
224
    new_inv = new_tree.inventory
 
225
    delta = TreeDelta()
 
226
    mutter('start compare_trees')
 
227
 
 
228
    # TODO: match for specific files can be rather smarter by finding
 
229
    # the IDs of those files up front and then considering only that.
 
230
 
 
231
    for file_id in old_tree:
 
232
        if file_id in new_tree:
 
233
            kind = old_inv.get_file_kind(file_id)
 
234
            assert kind == new_inv.get_file_kind(file_id)
 
235
            
 
236
            assert kind in ('file', 'directory', 'symlink', 'root_directory'), \
 
237
                   'invalid file kind %r' % kind
 
238
 
 
239
            if kind == 'root_directory':
 
240
                continue
 
241
            
 
242
            old_path = old_inv.id2path(file_id)
 
243
            new_path = new_inv.id2path(file_id)
 
244
 
 
245
            if specific_files:
 
246
                if (not is_inside_any(specific_files, old_path) 
 
247
                    and not is_inside_any(specific_files, new_path)):
 
248
                    continue
 
249
 
 
250
            if kind == 'file':
 
251
                old_sha1 = old_tree.get_file_sha1(file_id)
 
252
                new_sha1 = new_tree.get_file_sha1(file_id)
 
253
                text_modified = (old_sha1 != new_sha1)
 
254
            else:
 
255
                ## mutter("no text to check for %r %r" % (file_id, kind))
 
256
                text_modified = False
 
257
 
 
258
            # TODO: Can possibly avoid calculating path strings if the
 
259
            # two files are unchanged and their names and parents are
 
260
            # the same and the parents are unchanged all the way up.
 
261
            # May not be worthwhile.
 
262
            
 
263
            if old_path != new_path:
 
264
                delta.renamed.append((old_path, new_path, file_id, kind,
 
265
                                      text_modified))
 
266
            elif text_modified:
 
267
                delta.modified.append((new_path, file_id, kind))
 
268
            elif want_unchanged:
 
269
                delta.unchanged.append((new_path, file_id, kind))
 
270
        else:
 
271
            old_path = old_inv.id2path(file_id)
 
272
            if specific_files:
 
273
                if not is_inside_any(specific_files, old_path):
 
274
                    continue
 
275
            delta.removed.append((old_path, file_id, kind))
 
276
 
 
277
    mutter('start looking for new files')
 
278
    for file_id in new_inv:
 
279
        if file_id in old_inv:
 
280
            continue
 
281
        new_path = new_inv.id2path(file_id)
 
282
        if specific_files:
 
283
            if not is_inside_any(specific_files, new_path):
 
284
                continue
 
285
        kind = new_inv.get_file_kind(file_id)
 
286
        delta.added.append((new_path, file_id, kind))
 
287
            
 
288
    delta.removed.sort()
 
289
    delta.added.sort()
 
290
    delta.renamed.sort()
 
291
    delta.modified.sort()
 
292
    delta.unchanged.sort()
 
293
 
 
294
    return delta