~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/bundle/bundle_data.py

  • Committer: Ian Clatworthy
  • Date: 2009-01-19 02:24:15 UTC
  • mto: This revision was merged to the branch mainline in revision 3944.
  • Revision ID: ian.clatworthy@canonical.com-20090119022415-mo0mcfeiexfktgwt
apply jam's log --short fix (Ian Clatworthy)

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)
117
159
    def get_base(self, revision):
118
160
        revision_info = self.get_revision_info(revision.revision_id)
119
161
        if revision_info.base_id is not None:
120
 
            if revision_info.base_id == NULL_REVISION:
121
 
                return None
122
 
            else:
123
 
                return revision_info.base_id
 
162
            return revision_info.base_id
124
163
        if len(revision.parent_ids) == 0:
125
164
            # There is no base listed, and
126
165
            # the lowest revision doesn't have a parent
127
166
            # so this is probably against the empty tree
128
 
            # and thus base truly is None
129
 
            return None
 
167
            # and thus base truly is NULL_REVISION
 
168
            return NULL_REVISION
130
169
        else:
131
170
            return revision.parent_ids[-1]
132
171
 
152
191
                return r
153
192
        raise KeyError(revision_id)
154
193
 
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
 
194
    def revision_tree(self, repository, revision_id, base=None):
 
195
        revision = self.get_revision(revision_id)
 
196
        base = self.get_base(revision)
 
197
        if base == revision_id:
 
198
            raise AssertionError()
 
199
        if not self._validated_revisions_against_repo:
 
200
            self._validate_references_from_repository(repository)
 
201
        revision_info = self.get_revision_info(revision_id)
 
202
        inventory_revision_id = revision_id
 
203
        bundle_tree = BundleTree(repository.revision_tree(base), 
 
204
                                  inventory_revision_id)
 
205
        self._update_tree(bundle_tree, revision_id)
 
206
 
 
207
        inv = bundle_tree.inventory
 
208
        self._validate_inventory(inv, revision_id)
 
209
        self._validate_revision(inv, revision_id)
 
210
 
 
211
        return bundle_tree
211
212
 
212
213
    def _validate_references_from_repository(self, repository):
213
214
        """Now that we have a repository which should have some of the
235
236
        # All of the contained revisions were checked
236
237
        # in _validate_revisions
237
238
        checked = {}
238
 
        for rev_info in self.info.revisions:
 
239
        for rev_info in self.revisions:
239
240
            checked[rev_info.revision_id] = True
240
241
            add_sha(rev_to_sha, rev_info.revision_id, rev_info.sha1)
241
242
                
242
 
        for (rev, rev_info) in zip(self.info.real_revisions, self.info.revisions):
 
243
        for (rev, rev_info) in zip(self.real_revisions, self.revisions):
243
244
            add_sha(inv_to_sha, rev_info.revision_id, rev_info.inventory_sha1)
244
245
 
245
246
        count = 0
248
249
            if repository.has_revision(revision_id):
249
250
                testament = StrictTestament.from_revision(repository, 
250
251
                                                          revision_id)
251
 
                local_sha1 = testament.as_sha1()
 
252
                local_sha1 = self._testament_sha1_from_revision(repository,
 
253
                                                                revision_id)
252
254
                if sha1 != local_sha1:
253
255
                    raise BzrError('sha1 mismatch. For revision id {%s}' 
254
256
                            'local: %s, bundle: %s' % (revision_id, local_sha1, sha1))
257
259
            elif revision_id not in checked:
258
260
                missing[revision_id] = sha1
259
261
 
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
262
        if len(missing) > 0:
276
263
            # I don't know if this is an error yet
277
264
            warning('Not all revision hashes could be validated.'
278
265
                    ' Unable validate %d hashes' % len(missing))
279
266
        mutter('Verified %d sha hashes for the bundle.' % count)
 
267
        self._validated_revisions_against_repo = True
280
268
 
281
269
    def _validate_inventory(self, inv, revision_id):
282
270
        """At this point we should have generated the BundleTree,
283
271
        so build up an inventory, and make sure the hashes match.
284
272
        """
285
 
 
286
 
        assert inv is not None
287
 
 
288
273
        # Now we should have a complete inventory entry.
289
274
        s = serializer_v5.write_inventory_to_string(inv)
290
275
        sha1 = sha_string(s)
291
276
        # 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
 
277
        rev = self.get_revision(revision_id)
 
278
        if rev.revision_id != revision_id:
 
279
            raise AssertionError()
294
280
        if sha1 != rev.inventory_sha1:
295
281
            open(',,bogus-inv', 'wb').write(s)
296
282
            warning('Inventory sha hash mismatch for revision %s. %s'
297
283
                    ' != %s' % (revision_id, sha1, rev.inventory_sha1))
298
284
 
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
 
285
    def _validate_revision(self, inventory, revision_id):
 
286
        """Make sure all revision entries match their checksum."""
 
287
 
 
288
        # This is a mapping from each revision id to it's sha hash
 
289
        rev_to_sha1 = {}
 
290
        
 
291
        rev = self.get_revision(revision_id)
 
292
        rev_info = self.get_revision_info(revision_id)
 
293
        if not (rev.revision_id == rev_info.revision_id):
 
294
            raise AssertionError()
 
295
        if not (rev.revision_id == revision_id):
 
296
            raise AssertionError()
 
297
        sha1 = self._testament_sha1(rev, inventory)
 
298
        if sha1 != rev_info.sha1:
 
299
            raise TestamentMismatch(rev.revision_id, rev_info.sha1, sha1)
 
300
        if rev.revision_id in rev_to_sha1:
 
301
            raise BzrError('Revision {%s} given twice in the list'
 
302
                    % (rev.revision_id))
 
303
        rev_to_sha1[rev.revision_id] = sha1
498
304
 
499
305
    def _update_tree(self, bundle_tree, revision_id):
500
306
        """This fills out a BundleTree based on the information
505
311
 
506
312
        def get_rev_id(last_changed, path, kind):
507
313
            if last_changed is not None:
508
 
                changed_revision_id = last_changed.decode('utf-8')
 
314
                # last_changed will be a Unicode string because of how it was
 
315
                # read. Convert it back to utf8.
 
316
                changed_revision_id = osutils.safe_revision_id(last_changed,
 
317
                                                               warn=False)
509
318
            else:
510
319
                changed_revision_id = revision_id
511
320
            bundle_tree.note_last_changed(path, changed_revision_id)
522
331
                if name == 'last-changed':
523
332
                    last_changed = value
524
333
                elif name == 'executable':
525
 
                    assert value in ('yes', 'no'), value
526
334
                    val = (value == 'yes')
527
335
                    bundle_tree.note_executable(new_path, val)
528
336
                elif name == 'target':
532
340
            return last_changed, encoding
533
341
 
534
342
        def do_patch(path, lines, encoding):
535
 
            if encoding is not None:
536
 
                assert encoding == 'base64'
 
343
            if encoding == 'base64':
537
344
                patch = base64.decodestring(''.join(lines))
538
 
            else:
 
345
            elif encoding is None:
539
346
                patch =  ''.join(lines)
 
347
            else:
 
348
                raise ValueError(encoding)
540
349
            bundle_tree.note_patch(path, patch)
541
350
 
542
351
        def renamed(kind, extra, lines):
578
387
            if not info[1].startswith('file-id:'):
579
388
                raise BzrError('The file-id should follow the path for an add'
580
389
                        ': %r' % extra)
581
 
            file_id = info[1][8:]
 
390
            # This will be Unicode because of how the stream is read. Turn it
 
391
            # back into a utf8 file_id
 
392
            file_id = osutils.safe_file_id(info[1][8:], warn=False)
582
393
 
583
394
            bundle_tree.note_id(file_id, path, kind)
584
395
            # this will be overridden in extra_info if executable is specified.
608
419
            'modified':modified
609
420
        }
610
421
        for action_line, lines in \
611
 
            self.info.get_revision_info(revision_id).tree_actions:
 
422
            self.get_revision_info(revision_id).tree_actions:
612
423
            first = action_line.find(' ')
613
424
            if first == -1:
614
425
                raise BzrError('Bogus action line'
629
440
                        ' (unrecognized action): %r' % action_line)
630
441
            valid_actions[action](kind, extra, lines)
631
442
 
 
443
    def install_revisions(self, target_repo, stream_input=True):
 
444
        """Install revisions and return the target revision
 
445
 
 
446
        :param target_repo: The repository to install into
 
447
        :param stream_input: Ignored by this implementation.
 
448
        """
 
449
        apply_bundle.install_bundle(target_repo, self)
 
450
        return self.target
 
451
 
 
452
    def get_merge_request(self, target_repo):
 
453
        """Provide data for performing a merge
 
454
 
 
455
        Returns suggested base, suggested target, and patch verification status
 
456
        """
 
457
        return None, self.target, 'inapplicable'
 
458
 
632
459
 
633
460
class BundleTree(Tree):
634
461
    def __init__(self, base_tree, revision_id):
652
479
 
653
480
    def note_rename(self, old_path, new_path):
654
481
        """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)
 
482
        if new_path in self._renamed:
 
483
            raise AssertionError(new_path)
 
484
        if old_path in self._renamed_r:
 
485
            raise AssertionError(old_path)
657
486
        self._renamed[new_path] = old_path
658
487
        self._renamed_r[old_path] = new_path
659
488
 
664
493
        self._kinds[new_id] = kind
665
494
 
666
495
    def note_last_changed(self, file_id, revision_id):
667
 
        if (self._last_changed.has_key(file_id)
 
496
        if (file_id in self._last_changed
668
497
                and self._last_changed[file_id] != revision_id):
669
498
            raise BzrError('Mismatched last-changed revision for file_id {%s}'
670
499
                    ': %s != %s' % (file_id,
689
518
 
690
519
    def old_path(self, new_path):
691
520
        """Get the old_path (path in the base_tree) for the file at new_path"""
692
 
        assert new_path[:1] not in ('\\', '/')
 
521
        if new_path[:1] in ('\\', '/'):
 
522
            raise ValueError(new_path)
693
523
        old_path = self._renamed.get(new_path)
694
524
        if old_path is not None:
695
525
            return old_path
702
532
            if old_dir is None:
703
533
                old_path = None
704
534
            else:
705
 
                old_path = os.path.join(old_dir, basename)
 
535
                old_path = pathjoin(old_dir, basename)
706
536
        else:
707
537
            old_path = new_path
708
538
        #If the new path wasn't in renamed, the old one shouldn't be in
709
539
        #renamed_r
710
 
        if self._renamed_r.has_key(old_path):
 
540
        if old_path in self._renamed_r:
711
541
            return None
712
542
        return old_path 
713
543
 
715
545
        """Get the new_path (path in the target_tree) for the file at old_path
716
546
        in the base tree.
717
547
        """
718
 
        assert old_path[:1] not in ('\\', '/')
 
548
        if old_path[:1] in ('\\', '/'):
 
549
            raise ValueError(old_path)
719
550
        new_path = self._renamed_r.get(old_path)
720
551
        if new_path is not None:
721
552
            return new_path
722
 
        if self._renamed.has_key(new_path):
 
553
        if new_path in self._renamed:
723
554
            return None
724
555
        dirname,basename = os.path.split(old_path)
725
556
        if dirname != '':
727
558
            if new_dir is None:
728
559
                new_path = None
729
560
            else:
730
 
                new_path = os.path.join(new_dir, basename)
 
561
                new_path = pathjoin(new_dir, basename)
731
562
        else:
732
563
            new_path = old_path
733
564
        #If the old path wasn't in renamed, the new one shouldn't be in
734
565
        #renamed_r
735
 
        if self._renamed.has_key(new_path):
 
566
        if new_path in self._renamed:
736
567
            return None
737
568
        return new_path 
738
569
 
746
577
            return None
747
578
        if old_path in self.deleted:
748
579
            return None
749
 
        if hasattr(self.base_tree, 'path2id'):
 
580
        if getattr(self.base_tree, 'path2id', None) is not None:
750
581
            return self.base_tree.path2id(old_path)
751
582
        else:
752
583
            return self.base_tree.inventory.path2id(old_path)
784
615
                then be cached.
785
616
        """
786
617
        base_id = self.old_contents_id(file_id)
787
 
        if base_id is not None:
 
618
        if (base_id is not None and
 
619
            base_id != self.base_tree.inventory.root.file_id):
788
620
            patch_original = self.base_tree.get_file(base_id)
789
621
        else:
790
622
            patch_original = None
793
625
            if (patch_original is None and 
794
626
                self.get_kind(file_id) == 'directory'):
795
627
                return StringIO()
796
 
            assert patch_original is not None, "None: %s" % file_id
 
628
            if patch_original is None:
 
629
                raise AssertionError("None: %s" % file_id)
797
630
            return patch_original
798
631
 
799
 
        assert not file_patch.startswith('\\'), \
800
 
            'Malformed patch for %s, %r' % (file_id, file_patch)
 
632
        if file_patch.startswith('\\'):
 
633
            raise ValueError(
 
634
                'Malformed patch for %s, %r' % (file_id, file_patch))
801
635
        return patched_file(file_patch, patch_original)
802
636
 
803
637
    def get_symlink_target(self, file_id):
850
684
        This need to be called before ever accessing self.inventory
851
685
        """
852
686
        from os.path import dirname, basename
853
 
 
854
 
        assert self.base_tree is not None
855
687
        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)
 
688
        inv = Inventory(None, self.revision_id)
862
689
 
863
690
        def add_entry(file_id):
864
691
            path = self.id2path(file_id)
865
692
            if path is None:
866
693
                return
867
 
            parent_path = dirname(path)
868
 
            if parent_path == u'':
869
 
                parent_id = root_id
 
694
            if path == '':
 
695
                parent_id = None
870
696
            else:
 
697
                parent_path = dirname(path)
871
698
                parent_id = self.path2id(parent_path)
872
699
 
873
700
            kind = self.get_kind(file_id)
894
721
 
895
722
        sorted_entries = self.sorted_path_id()
896
723
        for path, file_id in sorted_entries:
897
 
            if file_id == inv.root.file_id:
898
 
                continue
899
724
            add_entry(file_id)
900
725
 
901
726
        return inv
930
755
    from bzrlib.iterablefile import IterableFile
931
756
    if file_patch == "":
932
757
        return IterableFile(())
933
 
    return IterableFile(iter_patched(original, file_patch.splitlines(True)))
 
758
    # string.splitlines(True) also splits on '\r', but the iter_patched code
 
759
    # only expects to iterate over '\n' style lines
 
760
    return IterableFile(iter_patched(original,
 
761
                StringIO(file_patch).readlines()))