~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/bundle/bundle_data.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-07-16 23:23:13 UTC
  • mfrom: (1551.19.47 Aaron's mergeable stuff)
  • Revision ID: pqm@pqm.ubuntu.com-20080716232313-9pcoze89hualg5qn
Fetch copies all required data even with stacking (abentley, #248506)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
#!/usr/bin/env python
2
 
"""\
3
 
Read in a bundle stream, and process it into a BundleReader object.
4
 
"""
 
1
# Copyright (C) 2006 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Read in a bundle stream, and process it into a BundleReader object."""
5
18
 
6
19
import base64
7
20
from cStringIO import StringIO
8
21
import os
9
22
import pprint
10
23
 
 
24
from bzrlib import (
 
25
    osutils,
 
26
    timestamp,
 
27
    )
 
28
import bzrlib.errors
 
29
from bzrlib.bundle import apply_bundle
11
30
from bzrlib.errors import (TestamentMismatch, BzrError, 
12
31
                           MalformedHeader, MalformedPatches, NotABundle)
13
 
from bzrlib.bundle.common import get_header, header_str
14
32
from bzrlib.inventory import (Inventory, InventoryEntry,
15
33
                              InventoryDirectory, InventoryFile,
16
34
                              InventoryLink)
17
 
from bzrlib.osutils import sha_file, sha_string
 
35
from bzrlib.osutils import sha_file, sha_string, pathjoin
18
36
from bzrlib.revision import Revision, NULL_REVISION
19
37
from bzrlib.testament import StrictTestament
20
38
from bzrlib.trace import mutter, warning
 
39
import bzrlib.transport
21
40
from bzrlib.tree import Tree
 
41
import bzrlib.urlutils
22
42
from bzrlib.xml5 import serializer_v5
23
43
 
24
44
 
57
77
        if self.properties:
58
78
            for property in self.properties:
59
79
                key_end = property.find(': ')
60
 
                assert key_end is not None
61
 
                key = property[:key_end].encode('utf-8')
62
 
                value = property[key_end+2:].encode('utf-8')
 
80
                if key_end == -1:
 
81
                    if not property.endswith(':'):
 
82
                        raise ValueError(property)
 
83
                    key = str(property[:-1])
 
84
                    value = ''
 
85
                else:
 
86
                    key = str(property[:key_end])
 
87
                    value = property[key_end+2:]
63
88
                rev.properties[key] = value
64
89
 
65
90
        return rev
66
91
 
 
92
    @staticmethod
 
93
    def from_revision(revision):
 
94
        revision_info = RevisionInfo(revision.revision_id)
 
95
        date = timestamp.format_highres_date(revision.timestamp,
 
96
                                             revision.timezone)
 
97
        revision_info.date = date
 
98
        revision_info.timezone = revision.timezone
 
99
        revision_info.timestamp = revision.timestamp
 
100
        revision_info.message = revision.message.split('\n')
 
101
        revision_info.properties = [': '.join(p) for p in
 
102
                                    revision.properties.iteritems()]
 
103
        return revision_info
 
104
 
67
105
 
68
106
class BundleInfo(object):
69
107
    """This contains the meta information. Stuff that allows you to
70
108
    recreate the revision or inventory XML.
71
109
    """
72
 
    def __init__(self):
 
110
    def __init__(self, bundle_format=None):
 
111
        self.bundle_format = None
73
112
        self.committer = None
74
113
        self.date = None
75
114
        self.message = None
86
125
        self.timestamp = None
87
126
        self.timezone = None
88
127
 
 
128
        # Have we checked the repository yet?
 
129
        self._validated_revisions_against_repo = False
 
130
 
89
131
    def __str__(self):
90
132
        return pprint.pformat(self.__dict__)
91
133
 
94
136
        split up, based on the assumptions that can be made
95
137
        when information is missing.
96
138
        """
97
 
        from bzrlib.bundle.common import unpack_highres_date
 
139
        from bzrlib.timestamp import unpack_highres_date
98
140
        # Put in all of the guessable information.
99
141
        if not self.timestamp and self.date:
100
142
            self.timestamp, self.timezone = unpack_highres_date(self.date)
152
194
                return r
153
195
        raise KeyError(revision_id)
154
196
 
155
 
 
156
 
class BundleReader(object):
157
 
    """This class reads in a bundle from a file, and returns
158
 
    a Bundle object, which can then be applied against a tree.
159
 
    """
160
 
    def __init__(self, from_file):
161
 
        """Read in the bundle from the file.
162
 
 
163
 
        :param from_file: A file-like object (must have iterator support).
164
 
        """
165
 
        object.__init__(self)
166
 
        self.from_file = iter(from_file)
167
 
        self._next_line = None
168
 
        
169
 
        self.info = BundleInfo()
170
 
        # We put the actual inventory ids in the footer, so that the patch
171
 
        # is easier to read for humans.
172
 
        # Unfortunately, that means we need to read everything before we
173
 
        # can create a proper bundle.
174
 
        self._read()
175
 
        self._validate()
176
 
 
177
 
    def _read(self):
178
 
        self._read_header()
179
 
        while self._next_line is not None:
180
 
            self._read_revision_header()
181
 
            if self._next_line is None:
182
 
                break
183
 
            self._read_patches()
184
 
            self._read_footer()
185
 
 
186
 
    def _validate(self):
187
 
        """Make sure that the information read in makes sense
188
 
        and passes appropriate checksums.
189
 
        """
190
 
        # Fill in all the missing blanks for the revisions
191
 
        # and generate the real_revisions list.
192
 
        self.info.complete_info()
193
 
 
194
 
    def _validate_revision(self, inventory, revision_id):
195
 
        """Make sure all revision entries match their checksum."""
196
 
 
197
 
        # This is a mapping from each revision id to it's sha hash
198
 
        rev_to_sha1 = {}
199
 
        
200
 
        rev = self.info.get_revision(revision_id)
201
 
        rev_info = self.info.get_revision_info(revision_id)
202
 
        assert rev.revision_id == rev_info.revision_id
203
 
        assert rev.revision_id == revision_id
204
 
        sha1 = StrictTestament(rev, inventory).as_sha1()
205
 
        if sha1 != rev_info.sha1:
206
 
            raise TestamentMismatch(rev.revision_id, rev_info.sha1, sha1)
207
 
        if rev_to_sha1.has_key(rev.revision_id):
208
 
            raise BzrError('Revision {%s} given twice in the list'
209
 
                    % (rev.revision_id))
210
 
        rev_to_sha1[rev.revision_id] = sha1
 
197
    def revision_tree(self, repository, revision_id, base=None):
 
198
        revision = self.get_revision(revision_id)
 
199
        base = self.get_base(revision)
 
200
        if base == revision_id:
 
201
            raise AssertionError()
 
202
        if not self._validated_revisions_against_repo:
 
203
            self._validate_references_from_repository(repository)
 
204
        revision_info = self.get_revision_info(revision_id)
 
205
        inventory_revision_id = revision_id
 
206
        bundle_tree = BundleTree(repository.revision_tree(base), 
 
207
                                  inventory_revision_id)
 
208
        self._update_tree(bundle_tree, revision_id)
 
209
 
 
210
        inv = bundle_tree.inventory
 
211
        self._validate_inventory(inv, revision_id)
 
212
        self._validate_revision(inv, revision_id)
 
213
 
 
214
        return bundle_tree
211
215
 
212
216
    def _validate_references_from_repository(self, repository):
213
217
        """Now that we have a repository which should have some of the
235
239
        # All of the contained revisions were checked
236
240
        # in _validate_revisions
237
241
        checked = {}
238
 
        for rev_info in self.info.revisions:
 
242
        for rev_info in self.revisions:
239
243
            checked[rev_info.revision_id] = True
240
244
            add_sha(rev_to_sha, rev_info.revision_id, rev_info.sha1)
241
245
                
242
 
        for (rev, rev_info) in zip(self.info.real_revisions, self.info.revisions):
 
246
        for (rev, rev_info) in zip(self.real_revisions, self.revisions):
243
247
            add_sha(inv_to_sha, rev_info.revision_id, rev_info.inventory_sha1)
244
248
 
245
249
        count = 0
248
252
            if repository.has_revision(revision_id):
249
253
                testament = StrictTestament.from_revision(repository, 
250
254
                                                          revision_id)
251
 
                local_sha1 = testament.as_sha1()
 
255
                local_sha1 = self._testament_sha1_from_revision(repository,
 
256
                                                                revision_id)
252
257
                if sha1 != local_sha1:
253
258
                    raise BzrError('sha1 mismatch. For revision id {%s}' 
254
259
                            'local: %s, bundle: %s' % (revision_id, local_sha1, sha1))
257
262
            elif revision_id not in checked:
258
263
                missing[revision_id] = sha1
259
264
 
260
 
        for inv_id, sha1 in inv_to_sha.iteritems():
261
 
            if repository.has_revision(inv_id):
262
 
                # Note: branch.get_inventory_sha1() just returns the value that
263
 
                # is stored in the revision text, and that value may be out
264
 
                # of date. This is bogus, because that means we aren't
265
 
                # validating the actual text, just that we wrote and read the
266
 
                # string. But for now, what the hell.
267
 
                local_sha1 = repository.get_inventory_sha1(inv_id)
268
 
                if sha1 != local_sha1:
269
 
                    raise BzrError('sha1 mismatch. For inventory id {%s}' 
270
 
                                   'local: %s, bundle: %s' % 
271
 
                                   (inv_id, local_sha1, sha1))
272
 
                else:
273
 
                    count += 1
274
 
 
275
265
        if len(missing) > 0:
276
266
            # I don't know if this is an error yet
277
267
            warning('Not all revision hashes could be validated.'
278
268
                    ' Unable validate %d hashes' % len(missing))
279
269
        mutter('Verified %d sha hashes for the bundle.' % count)
 
270
        self._validated_revisions_against_repo = True
280
271
 
281
272
    def _validate_inventory(self, inv, revision_id):
282
273
        """At this point we should have generated the BundleTree,
283
274
        so build up an inventory, and make sure the hashes match.
284
275
        """
285
 
 
286
 
        assert inv is not None
287
 
 
288
276
        # Now we should have a complete inventory entry.
289
277
        s = serializer_v5.write_inventory_to_string(inv)
290
278
        sha1 = sha_string(s)
291
279
        # Target revision is the last entry in the real_revisions list
292
 
        rev = self.info.get_revision(revision_id)
293
 
        assert rev.revision_id == revision_id
 
280
        rev = self.get_revision(revision_id)
 
281
        if rev.revision_id != revision_id:
 
282
            raise AssertionError()
294
283
        if sha1 != rev.inventory_sha1:
295
284
            open(',,bogus-inv', 'wb').write(s)
296
285
            warning('Inventory sha hash mismatch for revision %s. %s'
297
286
                    ' != %s' % (revision_id, sha1, rev.inventory_sha1))
298
287
 
299
 
    def get_bundle(self, repository):
300
 
        """Return the meta information, and a Bundle tree which can
301
 
        be used to populate the local stores and working tree, respectively.
302
 
        """
303
 
        return self.info, self.revision_tree(repository, self.info.target)
304
 
 
305
 
    def revision_tree(self, repository, revision_id, base=None):
306
 
        revision = self.info.get_revision(revision_id)
307
 
        base = self.info.get_base(revision)
308
 
        assert base != revision_id
309
 
        self._validate_references_from_repository(repository)
310
 
        revision_info = self.info.get_revision_info(revision_id)
311
 
        inventory_revision_id = revision_id
312
 
        bundle_tree = BundleTree(repository.revision_tree(base), 
313
 
                                  inventory_revision_id)
314
 
        self._update_tree(bundle_tree, revision_id)
315
 
 
316
 
        inv = bundle_tree.inventory
317
 
        self._validate_inventory(inv, revision_id)
318
 
        self._validate_revision(inv, revision_id)
319
 
 
320
 
        return bundle_tree
321
 
 
322
 
    def _next(self):
323
 
        """yield the next line, but secretly
324
 
        keep 1 extra line for peeking.
325
 
        """
326
 
        for line in self.from_file:
327
 
            last = self._next_line
328
 
            self._next_line = line
329
 
            if last is not None:
330
 
                #mutter('yielding line: %r' % last)
331
 
                yield last
332
 
        last = self._next_line
333
 
        self._next_line = None
334
 
        #mutter('yielding line: %r' % last)
335
 
        yield last
336
 
 
337
 
    def _read_header(self):
338
 
        """Read the bzr header"""
339
 
        header = get_header()
340
 
        found = False
341
 
        for line in self._next():
342
 
            if found:
343
 
                # not all mailers will keep trailing whitespace
344
 
                if line == '#\n':
345
 
                    line = '# \n'
346
 
                if (not line.startswith('# ') or not line.endswith('\n')
347
 
                        or line[2:-1].decode('utf-8') != header[0]):
348
 
                    raise MalformedHeader('Found a header, but it'
349
 
                        ' was improperly formatted')
350
 
                header.pop(0) # We read this line.
351
 
                if not header:
352
 
                    break # We found everything.
353
 
            elif (line.startswith('#') and line.endswith('\n')):
354
 
                line = line[1:-1].strip().decode('utf-8')
355
 
                if line[:len(header_str)] == header_str:
356
 
                    if line == header[0]:
357
 
                        found = True
358
 
                    else:
359
 
                        raise MalformedHeader('Found what looks like'
360
 
                                ' a header, but did not match')
361
 
                    header.pop(0)
362
 
        else:
363
 
            raise NotABundle('Did not find an opening header')
364
 
 
365
 
    def _read_revision_header(self):
366
 
        self.info.revisions.append(RevisionInfo(None))
367
 
        for line in self._next():
368
 
            # The bzr header is terminated with a blank line
369
 
            # which does not start with '#'
370
 
            if line is None or line == '\n':
371
 
                break
372
 
            self._handle_next(line)
373
 
 
374
 
    def _read_next_entry(self, line, indent=1):
375
 
        """Read in a key-value pair
376
 
        """
377
 
        if not line.startswith('#'):
378
 
            raise MalformedHeader('Bzr header did not start with #')
379
 
        line = line[1:-1].decode('utf-8') # Remove the '#' and '\n'
380
 
        if line[:indent] == ' '*indent:
381
 
            line = line[indent:]
382
 
        if not line:
383
 
            return None, None# Ignore blank lines
384
 
 
385
 
        loc = line.find(': ')
386
 
        if loc != -1:
387
 
            key = line[:loc]
388
 
            value = line[loc+2:]
389
 
            if not value:
390
 
                value = self._read_many(indent=indent+2)
391
 
        elif line[-1:] == ':':
392
 
            key = line[:-1]
393
 
            value = self._read_many(indent=indent+2)
394
 
        else:
395
 
            raise MalformedHeader('While looking for key: value pairs,'
396
 
                    ' did not find the colon %r' % (line))
397
 
 
398
 
        key = key.replace(' ', '_')
399
 
        #mutter('found %s: %s' % (key, value))
400
 
        return key, value
401
 
 
402
 
    def _handle_next(self, line):
403
 
        if line is None:
404
 
            return
405
 
        key, value = self._read_next_entry(line, indent=1)
406
 
        mutter('_handle_next %r => %r' % (key, value))
407
 
        if key is None:
408
 
            return
409
 
 
410
 
        revision_info = self.info.revisions[-1]
411
 
        if hasattr(revision_info, key):
412
 
            if getattr(revision_info, key) is None:
413
 
                setattr(revision_info, key, value)
414
 
            else:
415
 
                raise MalformedHeader('Duplicated Key: %s' % key)
416
 
        else:
417
 
            # What do we do with a key we don't recognize
418
 
            raise MalformedHeader('Unknown Key: "%s"' % key)
419
 
    
420
 
    def _read_many(self, indent):
421
 
        """If a line ends with no entry, that means that it should be
422
 
        followed with multiple lines of values.
423
 
 
424
 
        This detects the end of the list, because it will be a line that
425
 
        does not start properly indented.
426
 
        """
427
 
        values = []
428
 
        start = '#' + (' '*indent)
429
 
 
430
 
        if self._next_line is None or self._next_line[:len(start)] != start:
431
 
            return values
432
 
 
433
 
        for line in self._next():
434
 
            values.append(line[len(start):-1].decode('utf-8'))
435
 
            if self._next_line is None or self._next_line[:len(start)] != start:
436
 
                break
437
 
        return values
438
 
 
439
 
    def _read_one_patch(self):
440
 
        """Read in one patch, return the complete patch, along with
441
 
        the next line.
442
 
 
443
 
        :return: action, lines, do_continue
444
 
        """
445
 
        #mutter('_read_one_patch: %r' % self._next_line)
446
 
        # Peek and see if there are no patches
447
 
        if self._next_line is None or self._next_line.startswith('#'):
448
 
            return None, [], False
449
 
 
450
 
        first = True
451
 
        lines = []
452
 
        for line in self._next():
453
 
            if first:
454
 
                if not line.startswith('==='):
455
 
                    raise MalformedPatches('The first line of all patches'
456
 
                        ' should be a bzr meta line "==="'
457
 
                        ': %r' % line)
458
 
                action = line[4:-1].decode('utf-8')
459
 
            elif line.startswith('... '):
460
 
                action += line[len('... '):-1].decode('utf-8')
461
 
 
462
 
            if (self._next_line is not None and 
463
 
                self._next_line.startswith('===')):
464
 
                return action, lines, True
465
 
            elif self._next_line is None or self._next_line.startswith('#'):
466
 
                return action, lines, False
467
 
 
468
 
            if first:
469
 
                first = False
470
 
            elif not line.startswith('... '):
471
 
                lines.append(line)
472
 
 
473
 
        return action, lines, False
474
 
            
475
 
    def _read_patches(self):
476
 
        do_continue = True
477
 
        revision_actions = []
478
 
        while do_continue:
479
 
            action, lines, do_continue = self._read_one_patch()
480
 
            if action is not None:
481
 
                revision_actions.append((action, lines))
482
 
        assert self.info.revisions[-1].tree_actions is None
483
 
        self.info.revisions[-1].tree_actions = revision_actions
484
 
 
485
 
    def _read_footer(self):
486
 
        """Read the rest of the meta information.
487
 
 
488
 
        :param first_line:  The previous step iterates past what it
489
 
                            can handle. That extra line is given here.
490
 
        """
491
 
        for line in self._next():
492
 
            self._handle_next(line)
493
 
            if not self._next_line.startswith('#'):
494
 
                self._next().next()
495
 
                break
496
 
            if self._next_line is None:
497
 
                break
 
288
    def _validate_revision(self, inventory, revision_id):
 
289
        """Make sure all revision entries match their checksum."""
 
290
 
 
291
        # This is a mapping from each revision id to it's sha hash
 
292
        rev_to_sha1 = {}
 
293
        
 
294
        rev = self.get_revision(revision_id)
 
295
        rev_info = self.get_revision_info(revision_id)
 
296
        if not (rev.revision_id == rev_info.revision_id):
 
297
            raise AssertionError()
 
298
        if not (rev.revision_id == revision_id):
 
299
            raise AssertionError()
 
300
        sha1 = self._testament_sha1(rev, inventory)
 
301
        if sha1 != rev_info.sha1:
 
302
            raise TestamentMismatch(rev.revision_id, rev_info.sha1, sha1)
 
303
        if rev.revision_id in rev_to_sha1:
 
304
            raise BzrError('Revision {%s} given twice in the list'
 
305
                    % (rev.revision_id))
 
306
        rev_to_sha1[rev.revision_id] = sha1
498
307
 
499
308
    def _update_tree(self, bundle_tree, revision_id):
500
309
        """This fills out a BundleTree based on the information
505
314
 
506
315
        def get_rev_id(last_changed, path, kind):
507
316
            if last_changed is not None:
508
 
                changed_revision_id = last_changed.decode('utf-8')
 
317
                # last_changed will be a Unicode string because of how it was
 
318
                # read. Convert it back to utf8.
 
319
                changed_revision_id = osutils.safe_revision_id(last_changed,
 
320
                                                               warn=False)
509
321
            else:
510
322
                changed_revision_id = revision_id
511
323
            bundle_tree.note_last_changed(path, changed_revision_id)
522
334
                if name == 'last-changed':
523
335
                    last_changed = value
524
336
                elif name == 'executable':
525
 
                    assert value in ('yes', 'no'), value
526
337
                    val = (value == 'yes')
527
338
                    bundle_tree.note_executable(new_path, val)
528
339
                elif name == 'target':
532
343
            return last_changed, encoding
533
344
 
534
345
        def do_patch(path, lines, encoding):
535
 
            if encoding is not None:
536
 
                assert encoding == 'base64'
 
346
            if encoding == 'base64':
537
347
                patch = base64.decodestring(''.join(lines))
538
 
            else:
 
348
            elif encoding is None:
539
349
                patch =  ''.join(lines)
 
350
            else:
 
351
                raise ValueError(encoding)
540
352
            bundle_tree.note_patch(path, patch)
541
353
 
542
354
        def renamed(kind, extra, lines):
578
390
            if not info[1].startswith('file-id:'):
579
391
                raise BzrError('The file-id should follow the path for an add'
580
392
                        ': %r' % extra)
581
 
            file_id = info[1][8:]
 
393
            # This will be Unicode because of how the stream is read. Turn it
 
394
            # back into a utf8 file_id
 
395
            file_id = osutils.safe_file_id(info[1][8:], warn=False)
582
396
 
583
397
            bundle_tree.note_id(file_id, path, kind)
584
398
            # this will be overridden in extra_info if executable is specified.
608
422
            'modified':modified
609
423
        }
610
424
        for action_line, lines in \
611
 
            self.info.get_revision_info(revision_id).tree_actions:
 
425
            self.get_revision_info(revision_id).tree_actions:
612
426
            first = action_line.find(' ')
613
427
            if first == -1:
614
428
                raise BzrError('Bogus action line'
629
443
                        ' (unrecognized action): %r' % action_line)
630
444
            valid_actions[action](kind, extra, lines)
631
445
 
 
446
    def install_revisions(self, target_repo, stream_input=True):
 
447
        """Install revisions and return the target revision
 
448
 
 
449
        :param target_repo: The repository to install into
 
450
        :param stream_input: Ignored by this implementation.
 
451
        """
 
452
        apply_bundle.install_bundle(target_repo, self)
 
453
        return self.target
 
454
 
 
455
    def get_merge_request(self, target_repo):
 
456
        """Provide data for performing a merge
 
457
 
 
458
        Returns suggested base, suggested target, and patch verification status
 
459
        """
 
460
        return None, self.target, 'inapplicable'
 
461
 
632
462
 
633
463
class BundleTree(Tree):
634
464
    def __init__(self, base_tree, revision_id):
652
482
 
653
483
    def note_rename(self, old_path, new_path):
654
484
        """A file/directory has been renamed from old_path => new_path"""
655
 
        assert not self._renamed.has_key(new_path)
656
 
        assert not self._renamed_r.has_key(old_path)
 
485
        if new_path in self._renamed:
 
486
            raise AssertionError(new_path)
 
487
        if old_path in self._renamed_r:
 
488
            raise AssertionError(old_path)
657
489
        self._renamed[new_path] = old_path
658
490
        self._renamed_r[old_path] = new_path
659
491
 
664
496
        self._kinds[new_id] = kind
665
497
 
666
498
    def note_last_changed(self, file_id, revision_id):
667
 
        if (self._last_changed.has_key(file_id)
 
499
        if (file_id in self._last_changed
668
500
                and self._last_changed[file_id] != revision_id):
669
501
            raise BzrError('Mismatched last-changed revision for file_id {%s}'
670
502
                    ': %s != %s' % (file_id,
689
521
 
690
522
    def old_path(self, new_path):
691
523
        """Get the old_path (path in the base_tree) for the file at new_path"""
692
 
        assert new_path[:1] not in ('\\', '/')
 
524
        if new_path[:1] in ('\\', '/'):
 
525
            raise ValueError(new_path)
693
526
        old_path = self._renamed.get(new_path)
694
527
        if old_path is not None:
695
528
            return old_path
702
535
            if old_dir is None:
703
536
                old_path = None
704
537
            else:
705
 
                old_path = os.path.join(old_dir, basename)
 
538
                old_path = pathjoin(old_dir, basename)
706
539
        else:
707
540
            old_path = new_path
708
541
        #If the new path wasn't in renamed, the old one shouldn't be in
709
542
        #renamed_r
710
 
        if self._renamed_r.has_key(old_path):
 
543
        if old_path in self._renamed_r:
711
544
            return None
712
545
        return old_path 
713
546
 
715
548
        """Get the new_path (path in the target_tree) for the file at old_path
716
549
        in the base tree.
717
550
        """
718
 
        assert old_path[:1] not in ('\\', '/')
 
551
        if old_path[:1] in ('\\', '/'):
 
552
            raise ValueError(old_path)
719
553
        new_path = self._renamed_r.get(old_path)
720
554
        if new_path is not None:
721
555
            return new_path
722
 
        if self._renamed.has_key(new_path):
 
556
        if new_path in self._renamed:
723
557
            return None
724
558
        dirname,basename = os.path.split(old_path)
725
559
        if dirname != '':
727
561
            if new_dir is None:
728
562
                new_path = None
729
563
            else:
730
 
                new_path = os.path.join(new_dir, basename)
 
564
                new_path = pathjoin(new_dir, basename)
731
565
        else:
732
566
            new_path = old_path
733
567
        #If the old path wasn't in renamed, the new one shouldn't be in
734
568
        #renamed_r
735
 
        if self._renamed.has_key(new_path):
 
569
        if new_path in self._renamed:
736
570
            return None
737
571
        return new_path 
738
572
 
746
580
            return None
747
581
        if old_path in self.deleted:
748
582
            return None
749
 
        if hasattr(self.base_tree, 'path2id'):
 
583
        if getattr(self.base_tree, 'path2id', None) is not None:
750
584
            return self.base_tree.path2id(old_path)
751
585
        else:
752
586
            return self.base_tree.inventory.path2id(old_path)
784
618
                then be cached.
785
619
        """
786
620
        base_id = self.old_contents_id(file_id)
787
 
        if base_id is not None:
 
621
        if (base_id is not None and
 
622
            base_id != self.base_tree.inventory.root.file_id):
788
623
            patch_original = self.base_tree.get_file(base_id)
789
624
        else:
790
625
            patch_original = None
793
628
            if (patch_original is None and 
794
629
                self.get_kind(file_id) == 'directory'):
795
630
                return StringIO()
796
 
            assert patch_original is not None, "None: %s" % file_id
 
631
            if patch_original is None:
 
632
                raise AssertionError("None: %s" % file_id)
797
633
            return patch_original
798
634
 
799
 
        assert not file_patch.startswith('\\'), \
800
 
            'Malformed patch for %s, %r' % (file_id, file_patch)
 
635
        if file_patch.startswith('\\'):
 
636
            raise ValueError(
 
637
                'Malformed patch for %s, %r' % (file_id, file_patch))
801
638
        return patched_file(file_patch, patch_original)
802
639
 
803
640
    def get_symlink_target(self, file_id):
850
687
        This need to be called before ever accessing self.inventory
851
688
        """
852
689
        from os.path import dirname, basename
853
 
 
854
 
        assert self.base_tree is not None
855
690
        base_inv = self.base_tree.inventory
856
 
        root_id = base_inv.root.file_id
857
 
        try:
858
 
            # New inventories have a unique root_id
859
 
            inv = Inventory(root_id, self.revision_id)
860
 
        except TypeError:
861
 
            inv = Inventory(revision_id=self.revision_id)
 
691
        inv = Inventory(None, self.revision_id)
862
692
 
863
693
        def add_entry(file_id):
864
694
            path = self.id2path(file_id)
865
695
            if path is None:
866
696
                return
867
 
            parent_path = dirname(path)
868
 
            if parent_path == u'':
869
 
                parent_id = root_id
 
697
            if path == '':
 
698
                parent_id = None
870
699
            else:
 
700
                parent_path = dirname(path)
871
701
                parent_id = self.path2id(parent_path)
872
702
 
873
703
            kind = self.get_kind(file_id)
894
724
 
895
725
        sorted_entries = self.sorted_path_id()
896
726
        for path, file_id in sorted_entries:
897
 
            if file_id == inv.root.file_id:
898
 
                continue
899
727
            add_entry(file_id)
900
728
 
901
729
        return inv
930
758
    from bzrlib.iterablefile import IterableFile
931
759
    if file_patch == "":
932
760
        return IterableFile(())
933
 
    return IterableFile(iter_patched(original, file_patch.splitlines(True)))
 
761
    # string.splitlines(True) also splits on '\r', but the iter_patched code
 
762
    # only expects to iterate over '\n' style lines
 
763
    return IterableFile(iter_patched(original,
 
764
                StringIO(file_patch).readlines()))