~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/diff.py

  • Committer: Aaron Bentley
  • Date: 2005-09-13 02:42:07 UTC
  • mto: (1185.1.16)
  • mto: This revision was merged to the branch mainline in revision 1390.
  • Revision ID: aaron.bentley@utoronto.ca-20050913024207-489d573af4b76c4d
Fixed issues with pull not having a default location after branch

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 os
18
 
import re
19
 
import sys
20
 
 
21
 
from bzrlib.lazy_import import lazy_import
22
 
lazy_import(globals(), """
23
 
import errno
24
 
import subprocess
25
 
import tempfile
26
 
import time
27
 
 
28
 
from bzrlib import (
29
 
    errors,
30
 
    osutils,
31
 
    patiencediff,
32
 
    textfile,
33
 
    timestamp,
34
 
    )
35
 
""")
36
 
 
37
 
# compatability - plugins import compare_trees from diff!!!
38
 
# deprecated as of 0.10
 
18
from bzrlib.trace import mutter
 
19
from bzrlib.errors import BzrError
39
20
from bzrlib.delta import compare_trees
40
 
from bzrlib.symbol_versioning import (
41
 
        deprecated_function,
42
 
        zero_eight,
43
 
        )
44
 
from bzrlib.trace import mutter, warning
45
 
 
46
21
 
47
22
# TODO: Rather than building a changeset object, we should probably
48
23
# invoke callbacks on an object.  That object can either accumulate a
49
24
# list, write them out directly, etc etc.
50
25
 
51
 
def internal_diff(old_filename, oldlines, new_filename, newlines, to_file,
52
 
                  allow_binary=False, sequence_matcher=None,
53
 
                  path_encoding='utf8'):
 
26
def internal_diff(old_label, oldlines, new_label, newlines, to_file):
 
27
    import difflib
 
28
    
54
29
    # FIXME: difflib is wrong if there is no trailing newline.
55
30
    # The syntax used by patch seems to be "\ No newline at
56
31
    # end of file" following the last diff line from that
66
41
    # both sequences are empty.
67
42
    if not oldlines and not newlines:
68
43
        return
69
 
    
70
 
    if allow_binary is False:
71
 
        textfile.check_text_lines(oldlines)
72
 
        textfile.check_text_lines(newlines)
73
 
 
74
 
    if sequence_matcher is None:
75
 
        sequence_matcher = patiencediff.PatienceSequenceMatcher
76
 
    ud = patiencediff.unified_diff(oldlines, newlines,
77
 
                      fromfile=old_filename.encode(path_encoding),
78
 
                      tofile=new_filename.encode(path_encoding),
79
 
                      sequencematcher=sequence_matcher)
80
 
 
81
 
    ud = list(ud)
 
44
 
 
45
    ud = difflib.unified_diff(oldlines, newlines,
 
46
                              fromfile=old_label, tofile=new_label)
 
47
 
82
48
    # work-around for difflib being too smart for its own good
83
49
    # if /dev/null is "1,0", patch won't recognize it as /dev/null
84
50
    if not oldlines:
 
51
        ud = list(ud)
85
52
        ud[2] = ud[2].replace('-1,0', '-0,0')
86
53
    elif not newlines:
 
54
        ud = list(ud)
87
55
        ud[2] = ud[2].replace('+1,0', '+0,0')
88
 
    # work around for difflib emitting random spaces after the label
89
 
    ud[0] = ud[0][:-2] + '\n'
90
 
    ud[1] = ud[1][:-2] + '\n'
91
56
 
92
57
    for line in ud:
93
58
        to_file.write(line)
96
61
    print >>to_file
97
62
 
98
63
 
99
 
def _spawn_external_diff(diffcmd, capture_errors=True):
100
 
    """Spawn the externall diff process, and return the child handle.
101
 
 
102
 
    :param diffcmd: The command list to spawn
103
 
    :param capture_errors: Capture stderr as well as setting LANG=C
104
 
        and LC_ALL=C. This lets us read and understand the output of diff,
105
 
        and respond to any errors.
106
 
    :return: A Popen object.
107
 
    """
108
 
    if capture_errors:
109
 
        # construct minimal environment
110
 
        env = {}
111
 
        path = os.environ.get('PATH')
112
 
        if path is not None:
113
 
            env['PATH'] = path
114
 
        env['LANGUAGE'] = 'C'   # on win32 only LANGUAGE has effect
115
 
        env['LANG'] = 'C'
116
 
        env['LC_ALL'] = 'C'
117
 
        stderr = subprocess.PIPE
118
 
    else:
119
 
        env = None
120
 
        stderr = None
121
 
 
122
 
    try:
123
 
        pipe = subprocess.Popen(diffcmd,
124
 
                                stdin=subprocess.PIPE,
125
 
                                stdout=subprocess.PIPE,
126
 
                                stderr=stderr,
127
 
                                env=env)
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,
 
64
 
 
65
 
 
66
def external_diff(old_label, oldlines, new_label, newlines, to_file,
137
67
                  diff_opts):
138
68
    """Display a diff by calling out to the external diff program."""
 
69
    import sys
 
70
    
 
71
    if to_file != sys.stdout:
 
72
        raise NotImplementedError("sorry, can't send external diff other than to stdout yet",
 
73
                                  to_file)
 
74
 
139
75
    # make sure our own output is properly ordered before the diff
140
76
    to_file.flush()
141
77
 
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')
 
78
    from tempfile import NamedTemporaryFile
 
79
    import os
 
80
 
 
81
    oldtmpf = NamedTemporaryFile()
 
82
    newtmpf = NamedTemporaryFile()
146
83
 
147
84
    try:
148
85
        # TODO: perhaps a special case for comparing to or from the empty
155
92
        oldtmpf.writelines(oldlines)
156
93
        newtmpf.writelines(newlines)
157
94
 
158
 
        oldtmpf.close()
159
 
        newtmpf.close()
 
95
        oldtmpf.flush()
 
96
        newtmpf.flush()
160
97
 
161
98
        if not diff_opts:
162
99
            diff_opts = []
163
100
        diffcmd = ['diff',
164
 
                   '--label', old_filename,
165
 
                   old_abspath,
166
 
                   '--label', new_filename,
167
 
                   new_abspath,
168
 
                   '--binary',
169
 
                  ]
 
101
                   '--label', old_label,
 
102
                   oldtmpf.name,
 
103
                   '--label', new_label,
 
104
                   newtmpf.name]
170
105
 
171
106
        # diff only allows one style to be specified; they don't override.
172
107
        # note that some of these take optargs, and the optargs can be
192
127
        if diff_opts:
193
128
            diffcmd.extend(diff_opts)
194
129
 
195
 
        pipe = _spawn_external_diff(diffcmd, capture_errors=True)
196
 
        out,err = pipe.communicate()
197
 
        rc = pipe.returncode
 
130
        rc = os.spawnvp(os.P_WAIT, 'diff', diffcmd)
198
131
        
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 errors.BzrError(
216
 
                               'external diff failed with exit code 2'
217
 
                               ' when run with LANG=C and LC_ALL=C,'
218
 
                               ' but not when run natively: %r' % (diffcmd,))
219
 
 
220
 
            first_line = lang_c_out.split('\n', 1)[0]
221
 
            # Starting with diffutils 2.8.4 the word "binary" was dropped.
222
 
            m = re.match('^(binary )?files.*differ$', first_line, re.I)
223
 
            if m is None:
224
 
                raise errors.BzrError('external diff failed with exit code 2;'
225
 
                                      ' command: %r' % (diffcmd,))
226
 
            else:
227
 
                # Binary files differ, just return
228
 
                return
229
 
 
230
 
        # If we got to here, we haven't written out the output of diff
231
 
        # do so now
232
 
        to_file.write(out)
233
 
        if rc not in (0, 1):
 
132
        if rc != 0 and rc != 1:
234
133
            # returns 1 if files differ; that's OK
235
134
            if rc < 0:
236
135
                msg = 'signal %d' % (-rc)
237
136
            else:
238
137
                msg = 'exit code %d' % rc
239
138
                
240
 
            raise errors.BzrError('external diff failed with %s; command: %r' 
241
 
                                  % (rc, diffcmd))
242
 
 
243
 
 
 
139
            raise BzrError('external diff failed with %s; command: %r' % (rc, diffcmd))
244
140
    finally:
245
141
        oldtmpf.close()                 # and delete
246
142
        newtmpf.close()
247
 
        # Clean up. Warn in case the files couldn't be deleted
248
 
        # (in case windows still holds the file open, but not
249
 
        # if the files have already been deleted)
250
 
        try:
251
 
            os.remove(old_abspath)
252
 
        except OSError, e:
253
 
            if e.errno not in (errno.ENOENT,):
254
 
                warning('Failed to delete temporary file: %s %s',
255
 
                        old_abspath, e)
256
 
        try:
257
 
            os.remove(new_abspath)
258
 
        except OSError:
259
 
            if e.errno not in (errno.ENOENT,):
260
 
                warning('Failed to delete temporary file: %s %s',
261
 
                        new_abspath, e)
262
 
 
263
 
 
264
 
@deprecated_function(zero_eight)
265
 
def show_diff(b, from_spec, specific_files, external_diff_options=None,
266
 
              revision2=None, output=None, b2=None):
 
143
    
 
144
 
 
145
 
 
146
def show_diff(b, revision, specific_files, external_diff_options=None,
 
147
              revision2=None):
267
148
    """Shortcut for showing the diff to the working tree.
268
149
 
269
 
    Please use show_diff_trees instead.
270
 
 
271
150
    b
272
151
        Branch.
273
152
 
274
153
    revision
275
 
        None for 'basis tree', or otherwise the old revision to compare against.
276
 
    
277
 
    The more general form is show_diff_trees(), where the caller
278
 
    supplies any two trees.
279
 
    """
280
 
    if output is None:
281
 
        output = sys.stdout
282
 
 
283
 
    if from_spec is None:
284
 
        old_tree = b.bzrdir.open_workingtree()
285
 
        if b2 is None:
286
 
            old_tree = old_tree = old_tree.basis_tree()
287
 
    else:
288
 
        old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id)
289
 
 
290
 
    if revision2 is None:
291
 
        if b2 is None:
292
 
            new_tree = b.bzrdir.open_workingtree()
293
 
        else:
294
 
            new_tree = b2.bzrdir.open_workingtree()
295
 
    else:
296
 
        new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id)
297
 
 
298
 
    return show_diff_trees(old_tree, new_tree, output, specific_files,
299
 
                           external_diff_options)
300
 
 
301
 
 
302
 
def diff_cmd_helper(tree, specific_files, external_diff_options, 
303
 
                    old_revision_spec=None, new_revision_spec=None,
304
 
                    revision_specs=None,
305
 
                    old_label='a/', new_label='b/'):
306
 
    """Helper for cmd_diff.
307
 
 
308
 
    :param tree:
309
 
        A WorkingTree
310
 
 
311
 
    :param specific_files:
312
 
        The specific files to compare, or None
313
 
 
314
 
    :param external_diff_options:
315
 
        If non-None, run an external diff, and pass it these options
316
 
 
317
 
    :param old_revision_spec:
318
 
        If None, use basis tree as old revision, otherwise use the tree for
319
 
        the specified revision. 
320
 
 
321
 
    :param new_revision_spec:
322
 
        If None, use working tree as new revision, otherwise use the tree for
323
 
        the specified revision.
324
 
    
325
 
    :param revision_specs: 
326
 
        Zero, one or two RevisionSpecs from the command line, saying what revisions 
327
 
        to compare.  This can be passed as an alternative to the old_revision_spec 
328
 
        and new_revision_spec parameters.
329
 
 
330
 
    The more general form is show_diff_trees(), where the caller
331
 
    supplies any two trees.
332
 
    """
333
 
 
334
 
    # TODO: perhaps remove the old parameters old_revision_spec and
335
 
    # new_revision_spec, since this is only really for use from cmd_diff and
336
 
    # it now always passes through a sequence of revision_specs -- mbp
337
 
    # 20061221
338
 
 
339
 
    def spec_tree(spec):
340
 
        if tree:
341
 
            revision = spec.in_store(tree.branch)
342
 
        else:
343
 
            revision = spec.in_store(None)
344
 
        revision_id = revision.rev_id
345
 
        branch = revision.branch
346
 
        return branch.repository.revision_tree(revision_id)
347
 
 
348
 
    if revision_specs is not None:
349
 
        assert (old_revision_spec is None
350
 
                and new_revision_spec is None)
351
 
        if len(revision_specs) > 0:
352
 
            old_revision_spec = revision_specs[0]
353
 
        if len(revision_specs) > 1:
354
 
            new_revision_spec = revision_specs[1]
355
 
 
356
 
    if old_revision_spec is None:
357
 
        old_tree = tree.basis_tree()
358
 
    else:
359
 
        old_tree = spec_tree(old_revision_spec)
360
 
 
361
 
    if (new_revision_spec is None
362
 
        or new_revision_spec.spec is None):
363
 
        new_tree = tree
364
 
    else:
365
 
        new_tree = spec_tree(new_revision_spec)
366
 
 
367
 
    if new_tree is not tree:
368
 
        extra_trees = (tree,)
369
 
    else:
370
 
        extra_trees = None
371
 
 
372
 
    return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files,
373
 
                           external_diff_options,
374
 
                           old_label=old_label, new_label=new_label,
375
 
                           extra_trees=extra_trees)
 
154
        None for each, or otherwise the old revision to compare against.
 
155
    
 
156
    The more general form is show_diff_trees(), where the caller
 
157
    supplies any two trees.
 
158
    """
 
159
    import sys
 
160
 
 
161
    if revision == None:
 
162
        old_tree = b.basis_tree()
 
163
    else:
 
164
        old_tree = b.revision_tree(b.lookup_revision(revision))
 
165
 
 
166
    if revision2 == None:
 
167
        new_tree = b.working_tree()
 
168
    else:
 
169
        new_tree = b.revision_tree(b.lookup_revision(revision2))
 
170
 
 
171
    show_diff_trees(old_tree, new_tree, sys.stdout, specific_files,
 
172
                    external_diff_options)
 
173
 
376
174
 
377
175
 
378
176
def show_diff_trees(old_tree, new_tree, to_file, specific_files=None,
379
 
                    external_diff_options=None,
380
 
                    old_label='a/', new_label='b/',
381
 
                    extra_trees=None):
 
177
                    external_diff_options=None):
382
178
    """Show in text form the changes from one tree to another.
383
179
 
384
180
    to_files
386
182
 
387
183
    external_diff_options
388
184
        If set, use an external GNU diff and pass these options.
389
 
 
390
 
    extra_trees
391
 
        If set, more Trees to use for looking up file ids
392
185
    """
393
 
    old_tree.lock_read()
394
 
    try:
395
 
        if extra_trees is not None:
396
 
            for tree in extra_trees:
397
 
                tree.lock_read()
398
 
        new_tree.lock_read()
399
 
        try:
400
 
            return _show_diff_trees(old_tree, new_tree, to_file,
401
 
                                    specific_files, external_diff_options,
402
 
                                    old_label=old_label, new_label=new_label,
403
 
                                    extra_trees=extra_trees)
404
 
        finally:
405
 
            new_tree.unlock()
406
 
            if extra_trees is not None:
407
 
                for tree in extra_trees:
408
 
                    tree.unlock()
409
 
    finally:
410
 
        old_tree.unlock()
411
 
 
412
 
 
413
 
def _show_diff_trees(old_tree, new_tree, to_file,
414
 
                     specific_files, external_diff_options, 
415
 
                     old_label='a/', new_label='b/', extra_trees=None):
416
 
 
417
 
    # GNU Patch uses the epoch date to detect files that are being added
418
 
    # or removed in a diff.
419
 
    EPOCH_DATE = '1970-01-01 00:00:00 +0000'
 
186
 
 
187
    # TODO: Options to control putting on a prefix or suffix, perhaps as a format string
 
188
    old_label = ''
 
189
    new_label = ''
 
190
 
 
191
    DEVNULL = '/dev/null'
 
192
    # Windows users, don't panic about this filename -- it is a
 
193
    # special signal to GNU patch that the file should be created or
 
194
    # deleted respectively.
420
195
 
421
196
    # TODO: Generation of pseudo-diffs for added/deleted files could
422
197
    # be usefully made into a much faster special case.
429
204
    else:
430
205
        diff_file = internal_diff
431
206
    
432
 
    delta = new_tree.changes_from(old_tree,
433
 
        specific_files=specific_files,
434
 
        extra_trees=extra_trees, require_versioned=True)
435
 
 
436
 
    has_changes = 0
 
207
 
 
208
    delta = compare_trees(old_tree, new_tree, want_unchanged=False,
 
209
                          specific_files=specific_files)
 
210
 
437
211
    for path, file_id, kind in delta.removed:
438
 
        has_changes = 1
439
 
        print >>to_file, '=== removed %s %r' % (kind, path.encode('utf8'))
440
 
        old_name = '%s%s\t%s' % (old_label, path,
441
 
                                 _patch_header_date(old_tree, file_id, path))
442
 
        new_name = '%s%s\t%s' % (new_label, path, EPOCH_DATE)
443
 
        old_tree.inventory[file_id].diff(diff_file, old_name, old_tree,
444
 
                                         new_name, None, None, to_file)
 
212
        print >>to_file, '*** removed %s %r' % (kind, path)
 
213
        if kind == 'file':
 
214
            diff_file(old_label + path,
 
215
                      old_tree.get_file(file_id).readlines(),
 
216
                      DEVNULL, 
 
217
                      [],
 
218
                      to_file)
 
219
 
445
220
    for path, file_id, kind in delta.added:
446
 
        has_changes = 1
447
 
        print >>to_file, '=== added %s %r' % (kind, path.encode('utf8'))
448
 
        old_name = '%s%s\t%s' % (old_label, path, EPOCH_DATE)
449
 
        new_name = '%s%s\t%s' % (new_label, path,
450
 
                                 _patch_header_date(new_tree, file_id, path))
451
 
        new_tree.inventory[file_id].diff(diff_file, new_name, new_tree,
452
 
                                         old_name, None, None, to_file, 
453
 
                                         reverse=True)
454
 
    for (old_path, new_path, file_id, kind,
455
 
         text_modified, meta_modified) in delta.renamed:
456
 
        has_changes = 1
457
 
        prop_str = get_prop_change(meta_modified)
458
 
        print >>to_file, '=== renamed %s %r => %r%s' % (
459
 
                    kind, old_path.encode('utf8'),
460
 
                    new_path.encode('utf8'), prop_str)
461
 
        old_name = '%s%s\t%s' % (old_label, old_path,
462
 
                                 _patch_header_date(old_tree, file_id,
463
 
                                                    old_path))
464
 
        new_name = '%s%s\t%s' % (new_label, new_path,
465
 
                                 _patch_header_date(new_tree, file_id,
466
 
                                                    new_path))
467
 
        _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
468
 
                                    new_name, new_tree,
469
 
                                    text_modified, kind, to_file, diff_file)
470
 
    for path, file_id, kind, text_modified, meta_modified in delta.modified:
471
 
        has_changes = 1
472
 
        prop_str = get_prop_change(meta_modified)
473
 
        print >>to_file, '=== modified %s %r%s' % (kind, path.encode('utf8'), prop_str)
474
 
        # The file may be in a different location in the old tree (because
475
 
        # the containing dir was renamed, but the file itself was not)
476
 
        old_path = old_tree.id2path(file_id)
477
 
        old_name = '%s%s\t%s' % (old_label, old_path,
478
 
                                 _patch_header_date(old_tree, file_id, old_path))
479
 
        new_name = '%s%s\t%s' % (new_label, path,
480
 
                                 _patch_header_date(new_tree, file_id, path))
 
221
        print >>to_file, '*** added %s %r' % (kind, path)
 
222
        if kind == 'file':
 
223
            diff_file(DEVNULL,
 
224
                      [],
 
225
                      new_label + path,
 
226
                      new_tree.get_file(file_id).readlines(),
 
227
                      to_file)
 
228
 
 
229
    for old_path, new_path, file_id, kind, text_modified in delta.renamed:
 
230
        print >>to_file, '*** renamed %s %r => %r' % (kind, old_path, new_path)
481
231
        if text_modified:
482
 
            _maybe_diff_file_or_symlink(old_name, old_tree, file_id,
483
 
                                        new_name, new_tree,
484
 
                                        True, kind, to_file, diff_file)
485
 
 
486
 
    return has_changes
487
 
 
488
 
 
489
 
def _patch_header_date(tree, file_id, path):
490
 
    """Returns a timestamp suitable for use in a patch header."""
491
 
    mtime = tree.get_file_mtime(file_id, path)
492
 
    assert mtime is not None, \
493
 
        "got an mtime of None for file-id %s, path %s in tree %s" % (
494
 
                file_id, path, tree)
495
 
    return timestamp.format_patch_date(mtime)
496
 
 
497
 
 
498
 
def _raise_if_nonexistent(paths, old_tree, new_tree):
499
 
    """Complain if paths are not in either inventory or tree.
500
 
 
501
 
    It's OK with the files exist in either tree's inventory, or 
502
 
    if they exist in the tree but are not versioned.
503
 
    
504
 
    This can be used by operations such as bzr status that can accept
505
 
    unknown or ignored files.
506
 
    """
507
 
    mutter("check paths: %r", paths)
508
 
    if not paths:
509
 
        return
510
 
    s = old_tree.filter_unversioned_files(paths)
511
 
    s = new_tree.filter_unversioned_files(s)
512
 
    s = [path for path in s if not new_tree.has_filename(path)]
513
 
    if s:
514
 
        raise errors.PathsDoNotExist(sorted(s))
515
 
 
516
 
 
517
 
def get_prop_change(meta_modified):
518
 
    if meta_modified:
519
 
        return " (properties changed)"
520
 
    else:
521
 
        return  ""
522
 
 
523
 
 
524
 
def _maybe_diff_file_or_symlink(old_path, old_tree, file_id,
525
 
                                new_path, new_tree, text_modified,
526
 
                                kind, to_file, diff_file):
527
 
    if text_modified:
528
 
        new_entry = new_tree.inventory[file_id]
529
 
        old_tree.inventory[file_id].diff(diff_file,
530
 
                                         old_path, old_tree,
531
 
                                         new_path, new_entry, 
532
 
                                         new_tree, to_file)
 
232
            diff_file(old_label + old_path,
 
233
                      old_tree.get_file(file_id).readlines(),
 
234
                      new_label + new_path,
 
235
                      new_tree.get_file(file_id).readlines(),
 
236
                      to_file)
 
237
 
 
238
    for path, file_id, kind in delta.modified:
 
239
        print >>to_file, '*** modified %s %r' % (kind, path)
 
240
        if kind == 'file':
 
241
            diff_file(old_label + path,
 
242
                      old_tree.get_file(file_id).readlines(),
 
243
                      new_label + path,
 
244
                      new_tree.get_file(file_id).readlines(),
 
245
                      to_file)
 
246
 
 
247
 
 
248
 
 
249
 
 
250