/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/bzr/dirstate.py

  • Committer: Breezy landing bot
  • Author(s): Jelmer Vernooij
  • Date: 2018-11-16 11:31:40 UTC
  • mfrom: (7143.12.3 annotated-tags)
  • Revision ID: breezy.the.bot@gmail.com-20181116113140-618u04763u0dyxnh
Fix fetching of revisions that are referenced by annotated tags.

Merged from https://code.launchpad.net/~jelmer/brz/annotated-tags/+merge/358536

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006-2011 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
20
20
lines by NL. The field delimiters are ommitted in the grammar, line delimiters
21
21
are not - this is done for clarity of reading. All string data is in utf8.
22
22
 
23
 
MINIKIND = "f" | "d" | "l" | "a" | "r" | "t";
24
 
NL = "\n";
25
 
NULL = "\0";
26
 
WHOLE_NUMBER = {digit}, digit;
27
 
BOOLEAN = "y" | "n";
28
 
REVISION_ID = a non-empty utf8 string;
29
 
 
30
 
dirstate format = header line, full checksum, row count, parent details,
31
 
 ghost_details, entries;
32
 
header line = "#bazaar dirstate flat format 3", NL;
33
 
full checksum = "crc32: ", ["-"], WHOLE_NUMBER, NL;
34
 
row count = "num_entries: ", WHOLE_NUMBER, NL;
35
 
parent_details = WHOLE NUMBER, {REVISION_ID}* NL;
36
 
ghost_details = WHOLE NUMBER, {REVISION_ID}*, NL;
37
 
entries = {entry};
38
 
entry = entry_key, current_entry_details, {parent_entry_details};
39
 
entry_key = dirname,  basename, fileid;
40
 
current_entry_details = common_entry_details, working_entry_details;
41
 
parent_entry_details = common_entry_details, history_entry_details;
42
 
common_entry_details = MINIKIND, fingerprint, size, executable
43
 
working_entry_details = packed_stat
44
 
history_entry_details = REVISION_ID;
45
 
executable = BOOLEAN;
46
 
size = WHOLE_NUMBER;
47
 
fingerprint = a nonempty utf8 sequence with meaning defined by minikind.
48
 
 
49
 
Given this definition, the following is useful to know:
50
 
entry (aka row) - all the data for a given key.
51
 
entry[0]: The key (dirname, basename, fileid)
52
 
entry[0][0]: dirname
53
 
entry[0][1]: basename
54
 
entry[0][2]: fileid
55
 
entry[1]: The tree(s) data for this path and id combination.
56
 
entry[1][0]: The current tree
57
 
entry[1][1]: The second tree
58
 
 
59
 
For an entry for a tree, we have (using tree 0 - current tree) to demonstrate:
60
 
entry[1][0][0]: minikind
61
 
entry[1][0][1]: fingerprint
62
 
entry[1][0][2]: size
63
 
entry[1][0][3]: executable
64
 
entry[1][0][4]: packed_stat
65
 
OR (for non tree-0)
66
 
entry[1][1][4]: revision_id
 
23
::
 
24
 
 
25
    MINIKIND = "f" | "d" | "l" | "a" | "r" | "t";
 
26
    NL = "\\n";
 
27
    NULL = "\\0";
 
28
    WHOLE_NUMBER = {digit}, digit;
 
29
    BOOLEAN = "y" | "n";
 
30
    REVISION_ID = a non-empty utf8 string;
 
31
    
 
32
    dirstate format = header line, full checksum, row count, parent details,
 
33
     ghost_details, entries;
 
34
    header line = "#bazaar dirstate flat format 3", NL;
 
35
    full checksum = "crc32: ", ["-"], WHOLE_NUMBER, NL;
 
36
    row count = "num_entries: ", WHOLE_NUMBER, NL;
 
37
    parent_details = WHOLE NUMBER, {REVISION_ID}* NL;
 
38
    ghost_details = WHOLE NUMBER, {REVISION_ID}*, NL;
 
39
    entries = {entry};
 
40
    entry = entry_key, current_entry_details, {parent_entry_details};
 
41
    entry_key = dirname,  basename, fileid;
 
42
    current_entry_details = common_entry_details, working_entry_details;
 
43
    parent_entry_details = common_entry_details, history_entry_details;
 
44
    common_entry_details = MINIKIND, fingerprint, size, executable
 
45
    working_entry_details = packed_stat
 
46
    history_entry_details = REVISION_ID;
 
47
    executable = BOOLEAN;
 
48
    size = WHOLE_NUMBER;
 
49
    fingerprint = a nonempty utf8 sequence with meaning defined by minikind.
 
50
 
 
51
Given this definition, the following is useful to know::
 
52
 
 
53
    entry (aka row) - all the data for a given key.
 
54
    entry[0]: The key (dirname, basename, fileid)
 
55
    entry[0][0]: dirname
 
56
    entry[0][1]: basename
 
57
    entry[0][2]: fileid
 
58
    entry[1]: The tree(s) data for this path and id combination.
 
59
    entry[1][0]: The current tree
 
60
    entry[1][1]: The second tree
 
61
 
 
62
For an entry for a tree, we have (using tree 0 - current tree) to demonstrate::
 
63
 
 
64
    entry[1][0][0]: minikind
 
65
    entry[1][0][1]: fingerprint
 
66
    entry[1][0][2]: size
 
67
    entry[1][0][3]: executable
 
68
    entry[1][0][4]: packed_stat
 
69
 
 
70
OR (for non tree-0)::
 
71
 
 
72
    entry[1][1][4]: revision_id
67
73
 
68
74
There may be multiple rows at the root, one per id present in the root, so the
69
 
in memory root row is now:
70
 
self._dirblocks[0] -> ('', [entry ...]),
71
 
and the entries in there are
72
 
entries[0][0]: ''
73
 
entries[0][1]: ''
74
 
entries[0][2]: file_id
75
 
entries[1][0]: The tree data for the current tree for this fileid at /
76
 
etc.
77
 
 
78
 
Kinds:
79
 
'r' is a relocated entry: This path is not present in this tree with this id,
80
 
    but the id can be found at another location. The fingerprint is used to
81
 
    point to the target location.
82
 
'a' is an absent entry: In that tree the id is not present at this path.
83
 
'd' is a directory entry: This path in this tree is a directory with the
84
 
    current file id. There is no fingerprint for directories.
85
 
'f' is a file entry: As for directory, but it's a file. The fingerprint is the
86
 
    sha1 value of the file's canonical form, i.e. after any read filters have
87
 
    been applied to the convenience form stored in the working tree.
88
 
'l' is a symlink entry: As for directory, but a symlink. The fingerprint is the
89
 
    link target.
90
 
't' is a reference to a nested subtree; the fingerprint is the referenced
91
 
    revision.
 
75
in memory root row is now::
 
76
 
 
77
    self._dirblocks[0] -> ('', [entry ...]),
 
78
 
 
79
and the entries in there are::
 
80
 
 
81
    entries[0][0]: b''
 
82
    entries[0][1]: b''
 
83
    entries[0][2]: file_id
 
84
    entries[1][0]: The tree data for the current tree for this fileid at /
 
85
    etc.
 
86
 
 
87
Kinds::
 
88
 
 
89
   b'r' is a relocated entry: This path is not present in this tree with this
 
90
        id, but the id can be found at another location. The fingerprint is
 
91
        used to point to the target location.
 
92
   b'a' is an absent entry: In that tree the id is not present at this path.
 
93
   b'd' is a directory entry: This path in this tree is a directory with the
 
94
        current file id. There is no fingerprint for directories.
 
95
   b'f' is a file entry: As for directory, but it's a file. The fingerprint is
 
96
        the sha1 value of the file's canonical form, i.e. after any read
 
97
        filters have been applied to the convenience form stored in the working
 
98
        tree.
 
99
   b'l' is a symlink entry: As for directory, but a symlink. The fingerprint is
 
100
        the link target.
 
101
   b't' is a reference to a nested subtree; the fingerprint is the referenced
 
102
        revision.
92
103
 
93
104
Ordering:
94
105
 
95
 
The entries on disk and in memory are ordered according to the following keys:
 
106
The entries on disk and in memory are ordered according to the following keys::
96
107
 
97
108
    directory, as a list of components
98
109
    filename
99
110
    file-id
100
111
 
101
112
--- Format 1 had the following different definition: ---
102
 
rows = dirname, NULL, basename, NULL, MINIKIND, NULL, fileid_utf8, NULL,
103
 
    WHOLE NUMBER (* size *), NULL, packed stat, NULL, sha1|symlink target,
104
 
    {PARENT ROW}
105
 
PARENT ROW = NULL, revision_utf8, NULL, MINIKIND, NULL, dirname, NULL,
106
 
    basename, NULL, WHOLE NUMBER (* size *), NULL, "y" | "n", NULL,
107
 
    SHA1
 
113
 
 
114
::
 
115
 
 
116
    rows = dirname, NULL, basename, NULL, MINIKIND, NULL, fileid_utf8, NULL,
 
117
        WHOLE NUMBER (* size *), NULL, packed stat, NULL, sha1|symlink target,
 
118
        {PARENT ROW}
 
119
    PARENT ROW = NULL, revision_utf8, NULL, MINIKIND, NULL, dirname, NULL,
 
120
        basename, NULL, WHOLE NUMBER (* size *), NULL, "y" | "n", NULL,
 
121
        SHA1
108
122
 
109
123
PARENT ROW's are emitted for every parent that is not in the ghosts details
110
124
line. That is, if the parents are foo, bar, baz, and the ghosts are bar, then
135
149
----
136
150
 
137
151
Design priorities:
138
 
 1) Fast end to end use for bzr's top 5 uses cases. (commmit/diff/status/merge/???)
139
 
 2) fall back current object model as needed.
140
 
 3) scale usably to the largest trees known today - say 50K entries. (mozilla
 
152
 1. Fast end to end use for bzr's top 5 uses cases. (commmit/diff/status/merge/???)
 
153
 2. fall back current object model as needed.
 
154
 3. scale usably to the largest trees known today - say 50K entries. (mozilla
141
155
    is an example of this)
142
156
 
143
157
 
144
158
Locking:
 
159
 
145
160
 Eventually reuse dirstate objects across locks IFF the dirstate file has not
146
161
 been modified, but will require that we flush/ignore cached stat-hit data
147
162
 because we won't want to restat all files on disk just because a lock was
148
163
 acquired, yet we cannot trust the data after the previous lock was released.
149
164
 
150
 
Memory representation:
 
165
Memory representation::
 
166
 
151
167
 vector of all directories, and vector of the childen ?
152
168
   i.e.
153
169
     root_entrie = (direntry for root, [parent_direntries_for_root]),
167
183
    - What's the risk of error here? Once we have the base format being processed
168
184
      we should have a net win regardless of optimality. So we are going to
169
185
      go with what seems reasonable.
 
186
 
170
187
open questions:
171
188
 
172
189
Maybe we should do a test profile of the core structure - 10K simulated
201
218
 
202
219
"""
203
220
 
 
221
from __future__ import absolute_import
 
222
 
204
223
import bisect
205
 
import binascii
206
224
import errno
207
225
import operator
208
226
import os
209
227
from stat import S_IEXEC
210
228
import stat
211
 
import struct
212
229
import sys
213
230
import time
214
231
import zlib
215
232
 
216
 
from bzrlib import (
 
233
from . import (
 
234
    inventory,
 
235
    )
 
236
from .. import (
217
237
    cache_utf8,
 
238
    config,
218
239
    debug,
219
240
    errors,
220
 
    inventory,
221
241
    lock,
222
242
    osutils,
 
243
    static_tuple,
223
244
    trace,
 
245
    urlutils,
 
246
    )
 
247
from ..sixish import (
 
248
    range,
 
249
    text_type,
 
250
    viewitems,
 
251
    viewvalues,
224
252
    )
225
253
 
226
254
 
231
259
ERROR_DIRECTORY = 267
232
260
 
233
261
 
234
 
if not getattr(struct, '_compile', None):
235
 
    # Cannot pre-compile the dirstate pack_stat
236
 
    def pack_stat(st, _encode=binascii.b2a_base64, _pack=struct.pack):
237
 
        """Convert stat values into a packed representation."""
238
 
        return _encode(_pack('>LLLLLL', st.st_size, int(st.st_mtime),
239
 
            int(st.st_ctime), st.st_dev, st.st_ino & 0xFFFFFFFF,
240
 
            st.st_mode))[:-1]
241
 
else:
242
 
    # compile the struct compiler we need, so as to only do it once
243
 
    from _struct import Struct
244
 
    _compiled_pack = Struct('>LLLLLL').pack
245
 
    def pack_stat(st, _encode=binascii.b2a_base64, _pack=_compiled_pack):
246
 
        """Convert stat values into a packed representation."""
247
 
        # jam 20060614 it isn't really worth removing more entries if we
248
 
        # are going to leave it in packed form.
249
 
        # With only st_mtime and st_mode filesize is 5.5M and read time is 275ms
250
 
        # With all entries, filesize is 5.9M and read time is maybe 280ms
251
 
        # well within the noise margin
252
 
 
253
 
        # base64 encoding always adds a final newline, so strip it off
254
 
        # The current version
255
 
        return _encode(_pack(st.st_size, int(st.st_mtime), int(st.st_ctime),
256
 
            st.st_dev, st.st_ino & 0xFFFFFFFF, st.st_mode))[:-1]
257
 
        # This is 0.060s / 1.520s faster by not encoding as much information
258
 
        # return _encode(_pack('>LL', int(st.st_mtime), st.st_mode))[:-1]
259
 
        # This is not strictly faster than _encode(_pack())[:-1]
260
 
        # return '%X.%X.%X.%X.%X.%X' % (
261
 
        #      st.st_size, int(st.st_mtime), int(st.st_ctime),
262
 
        #      st.st_dev, st.st_ino, st.st_mode)
263
 
        # Similar to the _encode(_pack('>LL'))
264
 
        # return '%X.%X' % (int(st.st_mtime), st.st_mode)
 
262
class DirstateCorrupt(errors.BzrError):
 
263
 
 
264
    _fmt = "The dirstate file (%(state)s) appears to be corrupt: %(msg)s"
 
265
 
 
266
    def __init__(self, state, msg):
 
267
        errors.BzrError.__init__(self)
 
268
        self.state = state
 
269
        self.msg = msg
265
270
 
266
271
 
267
272
class SHA1Provider(object):
296
301
 
297
302
    def stat_and_sha1(self, abspath):
298
303
        """Return the stat and sha1 of a file given its absolute path."""
299
 
        file_obj = file(abspath, 'rb')
300
 
        try:
 
304
        with open(abspath, 'rb') as file_obj:
301
305
            statvalue = os.fstat(file_obj.fileno())
302
306
            sha1 = osutils.sha_file(file_obj)
303
 
        finally:
304
 
            file_obj.close()
305
307
        return statvalue, sha1
306
308
 
307
309
 
322
324
    """
323
325
 
324
326
    _kind_to_minikind = {
325
 
            'absent': 'a',
326
 
            'file': 'f',
327
 
            'directory': 'd',
328
 
            'relocated': 'r',
329
 
            'symlink': 'l',
330
 
            'tree-reference': 't',
 
327
            'absent': b'a',
 
328
            'file': b'f',
 
329
            'directory': b'd',
 
330
            'relocated': b'r',
 
331
            'symlink': b'l',
 
332
            'tree-reference': b't',
331
333
        }
332
334
    _minikind_to_kind = {
333
 
            'a': 'absent',
334
 
            'f': 'file',
335
 
            'd': 'directory',
336
 
            'l':'symlink',
337
 
            'r': 'relocated',
338
 
            't': 'tree-reference',
 
335
            b'a': 'absent',
 
336
            b'f': 'file',
 
337
            b'd': 'directory',
 
338
            b'l': 'symlink',
 
339
            b'r': 'relocated',
 
340
            b't': 'tree-reference',
339
341
        }
340
342
    _stat_to_minikind = {
341
 
        stat.S_IFDIR:'d',
342
 
        stat.S_IFREG:'f',
343
 
        stat.S_IFLNK:'l',
 
343
        stat.S_IFDIR: b'd',
 
344
        stat.S_IFREG: b'f',
 
345
        stat.S_IFLNK: b'l',
344
346
    }
345
 
    _to_yesno = {True:'y', False: 'n'} # TODO profile the performance gain
 
347
    _to_yesno = {True: b'y', False: b'n'} # TODO profile the performance gain
346
348
     # of using int conversion rather than a dict here. AND BLAME ANDREW IF
347
349
     # it is faster.
348
350
 
354
356
    NOT_IN_MEMORY = 0
355
357
    IN_MEMORY_UNMODIFIED = 1
356
358
    IN_MEMORY_MODIFIED = 2
 
359
    IN_MEMORY_HASH_MODIFIED = 3 # Only hash-cache updates
357
360
 
358
361
    # A pack_stat (the x's) that is just noise and will never match the output
359
362
    # of base64 encode.
360
 
    NULLSTAT = 'x' * 32
361
 
    NULL_PARENT_DETAILS = ('a', '', 0, False, '')
362
 
 
363
 
    HEADER_FORMAT_2 = '#bazaar dirstate flat format 2\n'
364
 
    HEADER_FORMAT_3 = '#bazaar dirstate flat format 3\n'
365
 
 
366
 
    def __init__(self, path, sha1_provider):
 
363
    NULLSTAT = b'x' * 32
 
364
    NULL_PARENT_DETAILS = static_tuple.StaticTuple(b'a', b'', 0, False, b'')
 
365
 
 
366
    HEADER_FORMAT_2 = b'#bazaar dirstate flat format 2\n'
 
367
    HEADER_FORMAT_3 = b'#bazaar dirstate flat format 3\n'
 
368
 
 
369
    def __init__(self, path, sha1_provider, worth_saving_limit=0):
367
370
        """Create a  DirState object.
368
371
 
369
372
        :param path: The path at which the dirstate file on disk should live.
370
373
        :param sha1_provider: an object meeting the SHA1Provider interface.
 
374
        :param worth_saving_limit: when the exact number of hash changed
 
375
            entries is known, only bother saving the dirstate if more than
 
376
            this count of entries have changed.
 
377
            -1 means never save hash changes, 0 means always save hash changes.
371
378
        """
372
379
        # _header_state and _dirblock_state represent the current state
373
380
        # of the dirstate metadata and the per-row data respectiely.
410
417
        # during commit.
411
418
        self._last_block_index = None
412
419
        self._last_entry_index = None
 
420
        # The set of known hash changes
 
421
        self._known_hash_changes = set()
 
422
        # How many hash changed entries can we have without saving
 
423
        self._worth_saving_limit = worth_saving_limit
 
424
        self._config_stack = config.LocationStack(urlutils.local_path_to_url(
 
425
            path))
413
426
 
414
427
    def __repr__(self):
415
428
        return "%s(%r)" % \
416
429
            (self.__class__.__name__, self._filename)
417
430
 
 
431
    def _mark_modified(self, hash_changed_entries=None, header_modified=False):
 
432
        """Mark this dirstate as modified.
 
433
 
 
434
        :param hash_changed_entries: if non-None, mark just these entries as
 
435
            having their hash modified.
 
436
        :param header_modified: mark the header modified as well, not just the
 
437
            dirblocks.
 
438
        """
 
439
        #trace.mutter_callsite(3, "modified hash entries: %s", hash_changed_entries)
 
440
        if hash_changed_entries:
 
441
            self._known_hash_changes.update([e[0] for e in hash_changed_entries])
 
442
            if self._dirblock_state in (DirState.NOT_IN_MEMORY,
 
443
                                        DirState.IN_MEMORY_UNMODIFIED):
 
444
                # If the dirstate is already marked a IN_MEMORY_MODIFIED, then
 
445
                # that takes precedence.
 
446
                self._dirblock_state = DirState.IN_MEMORY_HASH_MODIFIED
 
447
        else:
 
448
            # TODO: Since we now have a IN_MEMORY_HASH_MODIFIED state, we
 
449
            #       should fail noisily if someone tries to set
 
450
            #       IN_MEMORY_MODIFIED but we don't have a write-lock!
 
451
            # We don't know exactly what changed so disable smart saving
 
452
            self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
453
        if header_modified:
 
454
            self._header_state = DirState.IN_MEMORY_MODIFIED
 
455
 
 
456
    def _mark_unmodified(self):
 
457
        """Mark this dirstate as unmodified."""
 
458
        self._header_state = DirState.IN_MEMORY_UNMODIFIED
 
459
        self._dirblock_state = DirState.IN_MEMORY_UNMODIFIED
 
460
        self._known_hash_changes = set()
 
461
 
418
462
    def add(self, path, file_id, kind, stat, fingerprint):
419
463
        """Add a path to be tracked.
420
464
 
421
 
        :param path: The path within the dirstate - '' is the root, 'foo' is the
 
465
        :param path: The path within the dirstate - b'' is the root, 'foo' is the
422
466
            path foo within the root, 'foo/bar' is the path bar within foo
423
467
            within the root.
424
468
        :param file_id: The file id of the path being added.
429
473
            after any read filters have been applied),
430
474
            or the target of a symlink,
431
475
            or the referenced revision id for tree-references,
432
 
            or '' for directories.
 
476
            or b'' for directories.
433
477
        """
434
478
        # adding a file:
435
479
        # find the block its in.
457
501
        utf8path = (dirname + '/' + basename).strip('/').encode('utf8')
458
502
        dirname, basename = osutils.split(utf8path)
459
503
        # uses __class__ for speed; the check is needed for safety
460
 
        if file_id.__class__ is not str:
 
504
        if file_id.__class__ is not bytes:
461
505
            raise AssertionError(
462
506
                "must be a utf8 file_id not %s" % (type(file_id), ))
463
507
        # Make sure the file_id does not exist in this tree
464
508
        rename_from = None
465
509
        file_id_entry = self._get_entry(0, fileid_utf8=file_id, include_deleted=True)
466
510
        if file_id_entry != (None, None):
467
 
            if file_id_entry[1][0][0] == 'a':
 
511
            if file_id_entry[1][0][0] == b'a':
468
512
                if file_id_entry[0] != (dirname, basename, file_id):
469
513
                    # set the old name's current operation to rename
470
514
                    self.update_minimal(file_id_entry[0],
471
 
                        'r',
472
 
                        path_utf8='',
473
 
                        packed_stat='',
 
515
                        b'r',
 
516
                        path_utf8=b'',
 
517
                        packed_stat=b'',
474
518
                        fingerprint=utf8path
475
519
                    )
476
520
                    rename_from = file_id_entry[0][0:2]
479
523
                kind = DirState._minikind_to_kind[file_id_entry[1][0][0]]
480
524
                info = '%s:%s' % (kind, path)
481
525
                raise errors.DuplicateFileId(file_id, info)
482
 
        first_key = (dirname, basename, '')
 
526
        first_key = (dirname, basename, b'')
483
527
        block_index, present = self._find_block_index_from_key(first_key)
484
528
        if present:
485
529
            # check the path is not in the tree
487
531
            entry_index, _ = self._find_entry_index(first_key, block)
488
532
            while (entry_index < len(block) and
489
533
                block[entry_index][0][0:2] == first_key[0:2]):
490
 
                if block[entry_index][1][0][0] not in 'ar':
 
534
                if block[entry_index][1][0][0] not in (b'a', b'r'):
491
535
                    # this path is in the dirstate in the current tree.
492
 
                    raise Exception, "adding already added path!"
 
536
                    raise Exception("adding already added path!")
493
537
                entry_index += 1
494
538
        else:
495
539
            # The block where we want to put the file is not present. But it
513
557
        minikind = DirState._kind_to_minikind[kind]
514
558
        if rename_from is not None:
515
559
            if rename_from[0]:
516
 
                old_path_utf8 = '%s/%s' % rename_from
 
560
                old_path_utf8 = b'%s/%s' % rename_from
517
561
            else:
518
562
                old_path_utf8 = rename_from[1]
519
 
            parent_info[0] = ('r', old_path_utf8, 0, False, '')
 
563
            parent_info[0] = (b'r', old_path_utf8, 0, False, b'')
520
564
        if kind == 'file':
521
565
            entry_data = entry_key, [
522
566
                (minikind, fingerprint, size, False, packed_stat),
523
567
                ] + parent_info
524
568
        elif kind == 'directory':
525
569
            entry_data = entry_key, [
526
 
                (minikind, '', 0, False, packed_stat),
 
570
                (minikind, b'', 0, False, packed_stat),
527
571
                ] + parent_info
528
572
        elif kind == 'symlink':
529
573
            entry_data = entry_key, [
539
583
        if not present:
540
584
            block.insert(entry_index, entry_data)
541
585
        else:
542
 
            if block[entry_index][1][0][0] != 'a':
 
586
            if block[entry_index][1][0][0] != b'a':
543
587
                raise AssertionError(" %r(%r) already added" % (basename, file_id))
544
588
            block[entry_index][1][0] = entry_data[1][0]
545
589
 
546
590
        if kind == 'directory':
547
591
           # insert a new dirblock
548
592
           self._ensure_block(block_index, entry_index, utf8path)
549
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
593
        self._mark_modified()
550
594
        if self._id_index:
551
 
            self._id_index.setdefault(entry_key[2], set()).add(entry_key)
 
595
            self._add_to_id_index(self._id_index, entry_key)
552
596
 
553
597
    def _bisect(self, paths):
554
598
        """Bisect through the disk structure for specific rows.
575
619
        file_size = os.fstat(state_file.fileno()).st_size
576
620
        # We end up with 2 extra fields, we should have a trailing '\n' to
577
621
        # ensure that we read the whole record, and we should have a precursur
578
 
        # '' which ensures that we start after the previous '\n'
 
622
        # b'' which ensures that we start after the previous '\n'
579
623
        entry_field_count = self._fields_per_entry() + 1
580
624
 
581
625
        low = self._end_of_header
609
653
            if count > max_count:
610
654
                raise errors.BzrError('Too many seeks, most likely a bug.')
611
655
 
612
 
            mid = max(low, (low+high-page_size)/2)
 
656
            mid = max(low, (low+high-page_size)//2)
613
657
 
614
658
            state_file.seek(mid)
615
659
            # limit the read size, so we don't end up reading data that we have
618
662
            block = state_file.read(read_size)
619
663
 
620
664
            start = mid
621
 
            entries = block.split('\n')
 
665
            entries = block.split(b'\n')
622
666
 
623
667
            if len(entries) < 2:
624
668
                # We didn't find a '\n', so we cannot have found any records.
632
676
            # Check the first and last entries, in case they are partial, or if
633
677
            # we don't care about the rest of this page
634
678
            first_entry_num = 0
635
 
            first_fields = entries[0].split('\0')
 
679
            first_fields = entries[0].split(b'\0')
636
680
            if len(first_fields) < entry_field_count:
637
681
                # We didn't get the complete first entry
638
682
                # so move start, and grab the next, which
639
683
                # should be a full entry
640
684
                start += len(entries[0])+1
641
 
                first_fields = entries[1].split('\0')
 
685
                first_fields = entries[1].split(b'\0')
642
686
                first_entry_num = 1
643
687
 
644
688
            if len(first_fields) <= 2:
652
696
                # after this first record.
653
697
                after = start
654
698
                if first_fields[1]:
655
 
                    first_path = first_fields[1] + '/' + first_fields[2]
 
699
                    first_path = first_fields[1] + b'/' + first_fields[2]
656
700
                else:
657
701
                    first_path = first_fields[2]
658
702
                first_loc = _bisect_path_left(cur_files, first_path)
668
712
 
669
713
                # Parse the last entry
670
714
                last_entry_num = len(entries)-1
671
 
                last_fields = entries[last_entry_num].split('\0')
 
715
                last_fields = entries[last_entry_num].split(b'\0')
672
716
                if len(last_fields) < entry_field_count:
673
717
                    # The very last hunk was not complete,
674
718
                    # read the previous hunk
675
719
                    after = mid + len(block) - len(entries[-1])
676
720
                    last_entry_num -= 1
677
 
                    last_fields = entries[last_entry_num].split('\0')
 
721
                    last_fields = entries[last_entry_num].split(b'\0')
678
722
                else:
679
723
                    after = mid + len(block)
680
724
 
681
725
                if last_fields[1]:
682
 
                    last_path = last_fields[1] + '/' + last_fields[2]
 
726
                    last_path = last_fields[1] + b'/' + last_fields[2]
683
727
                else:
684
728
                    last_path = last_fields[2]
685
729
                last_loc = _bisect_path_right(post, last_path)
705
749
                    # careful if we should append rather than overwrite
706
750
                    if last_entry_num != first_entry_num:
707
751
                        paths.setdefault(last_path, []).append(last_fields)
708
 
                    for num in xrange(first_entry_num+1, last_entry_num):
 
752
                    for num in range(first_entry_num+1, last_entry_num):
709
753
                        # TODO: jam 20070223 We are already splitting here, so
710
754
                        #       shouldn't we just split the whole thing rather
711
755
                        #       than doing the split again in add_one_record?
712
 
                        fields = entries[num].split('\0')
 
756
                        fields = entries[num].split(b'\0')
713
757
                        if fields[1]:
714
 
                            path = fields[1] + '/' + fields[2]
 
758
                            path = fields[1] + b'/' + fields[2]
715
759
                        else:
716
760
                            path = fields[2]
717
761
                        paths.setdefault(path, []).append(fields)
767
811
        file_size = os.fstat(state_file.fileno()).st_size
768
812
        # We end up with 2 extra fields, we should have a trailing '\n' to
769
813
        # ensure that we read the whole record, and we should have a precursur
770
 
        # '' which ensures that we start after the previous '\n'
 
814
        # b'' which ensures that we start after the previous '\n'
771
815
        entry_field_count = self._fields_per_entry() + 1
772
816
 
773
817
        low = self._end_of_header
801
845
            if count > max_count:
802
846
                raise errors.BzrError('Too many seeks, most likely a bug.')
803
847
 
804
 
            mid = max(low, (low+high-page_size)/2)
 
848
            mid = max(low, (low+high-page_size)//2)
805
849
 
806
850
            state_file.seek(mid)
807
851
            # limit the read size, so we don't end up reading data that we have
810
854
            block = state_file.read(read_size)
811
855
 
812
856
            start = mid
813
 
            entries = block.split('\n')
 
857
            entries = block.split(b'\n')
814
858
 
815
859
            if len(entries) < 2:
816
860
                # We didn't find a '\n', so we cannot have found any records.
824
868
            # Check the first and last entries, in case they are partial, or if
825
869
            # we don't care about the rest of this page
826
870
            first_entry_num = 0
827
 
            first_fields = entries[0].split('\0')
 
871
            first_fields = entries[0].split(b'\0')
828
872
            if len(first_fields) < entry_field_count:
829
873
                # We didn't get the complete first entry
830
874
                # so move start, and grab the next, which
831
875
                # should be a full entry
832
876
                start += len(entries[0])+1
833
 
                first_fields = entries[1].split('\0')
 
877
                first_fields = entries[1].split(b'\0')
834
878
                first_entry_num = 1
835
879
 
836
880
            if len(first_fields) <= 1:
857
901
 
858
902
                # Parse the last entry
859
903
                last_entry_num = len(entries)-1
860
 
                last_fields = entries[last_entry_num].split('\0')
 
904
                last_fields = entries[last_entry_num].split(b'\0')
861
905
                if len(last_fields) < entry_field_count:
862
906
                    # The very last hunk was not complete,
863
907
                    # read the previous hunk
864
908
                    after = mid + len(block) - len(entries[-1])
865
909
                    last_entry_num -= 1
866
 
                    last_fields = entries[last_entry_num].split('\0')
 
910
                    last_fields = entries[last_entry_num].split(b'\0')
867
911
                else:
868
912
                    after = mid + len(block)
869
913
 
891
935
                    # careful if we should append rather than overwrite
892
936
                    if last_entry_num != first_entry_num:
893
937
                        paths.setdefault(last_dir, []).append(last_fields)
894
 
                    for num in xrange(first_entry_num+1, last_entry_num):
 
938
                    for num in range(first_entry_num+1, last_entry_num):
895
939
                        # TODO: jam 20070223 We are already splitting here, so
896
940
                        #       shouldn't we just split the whole thing rather
897
941
                        #       than doing the split again in add_one_record?
898
 
                        fields = entries[num].split('\0')
 
942
                        fields = entries[num].split(b'\0')
899
943
                        paths.setdefault(fields[1], []).append(fields)
900
944
 
901
945
                    for cur_dir in middle_files:
926
970
        directories. (and renames?)
927
971
 
928
972
        :param paths: A sorted list of (dir, name) pairs
929
 
             eg: [('', 'a'), ('', 'f'), ('a/b', 'c')]
 
973
             eg: [('', b'a'), ('', b'f'), ('a/b', b'c')]
930
974
        :return: A dictionary mapping (dir, name, file_id) => [tree_info]
931
975
        """
932
976
        # Map from (dir, name, file_id) => [tree_info]
943
987
            # Directories that need to be read
944
988
            pending_dirs = set()
945
989
            paths_to_search = set()
946
 
            for entry_list in newly_found.itervalues():
 
990
            for entry_list in viewvalues(newly_found):
947
991
                for dir_name_id, trees_info in entry_list:
948
992
                    found[dir_name_id] = trees_info
949
993
                    found_dir_names.add(dir_name_id[:2])
950
994
                    is_dir = False
951
995
                    for tree_info in trees_info:
952
996
                        minikind = tree_info[0]
953
 
                        if minikind == 'd':
 
997
                        if minikind == b'd':
954
998
                            if is_dir:
955
999
                                # We already processed this one as a directory,
956
1000
                                # we don't need to do the extra work again.
960
1004
                            is_dir = True
961
1005
                            if path not in processed_dirs:
962
1006
                                pending_dirs.add(path)
963
 
                        elif minikind == 'r':
 
1007
                        elif minikind == b'r':
964
1008
                            # Rename, we need to directly search the target
965
1009
                            # which is contained in the fingerprint column
966
1010
                            dir_name = osutils.split(tree_info[1])
993
1037
            return
994
1038
        # only require all dirblocks if we are doing a full-pass removal.
995
1039
        self._read_dirblocks_if_needed()
996
 
        dead_patterns = set([('a', 'r'), ('a', 'a'), ('r', 'r'), ('r', 'a')])
 
1040
        dead_patterns = {(b'a', b'r'), (b'a', b'a'), (b'r', b'r'), (b'r', b'a')}
997
1041
        def iter_entries_removable():
998
1042
            for block in self._dirblocks:
999
1043
                deleted_positions = []
1018
1062
 
1019
1063
        self._ghosts = []
1020
1064
        self._parents = [parents[0]]
1021
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
1022
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
1065
        self._mark_modified(header_modified=True)
1023
1066
 
1024
1067
    def _empty_parent_info(self):
1025
1068
        return [DirState.NULL_PARENT_DETAILS] * (len(self._parents) -
1045
1088
        :param dirname: The utf8 dirname to ensure there is a block for.
1046
1089
        :return: The index for the block.
1047
1090
        """
1048
 
        if dirname == '' and parent_row_index == 0 and parent_block_index == 0:
 
1091
        if dirname == b'' and parent_row_index == 0 and parent_block_index == 0:
1049
1092
            # This is the signature of the root row, and the
1050
1093
            # contents-of-root row is always index 1
1051
1094
            return 1
1052
1095
        # the basename of the directory must be the end of its full name.
1053
1096
        if not (parent_block_index == -1 and
1054
 
            parent_block_index == -1 and dirname == ''):
 
1097
            parent_block_index == -1 and dirname == b''):
1055
1098
            if not dirname.endswith(
1056
1099
                    self._dirblocks[parent_block_index][1][parent_row_index][0][1]):
1057
1100
                raise AssertionError("bad dirname %r" % dirname)
1058
 
        block_index, present = self._find_block_index_from_key((dirname, '', ''))
 
1101
        block_index, present = self._find_block_index_from_key((dirname, b'', b''))
1059
1102
        if not present:
1060
1103
            ## In future, when doing partial parsing, this should load and
1061
1104
            # populate the entire block.
1072
1115
            to prevent unneeded overhead when callers have a sorted list already.
1073
1116
        :return: Nothing.
1074
1117
        """
1075
 
        if new_entries[0][0][0:2] != ('', ''):
 
1118
        if new_entries[0][0][0:2] != (b'', b''):
1076
1119
            raise AssertionError(
1077
1120
                "Missing root row %r" % (new_entries[0][0],))
1078
1121
        # The two blocks here are deliberate: the root block and the
1079
1122
        # contents-of-root block.
1080
 
        self._dirblocks = [('', []), ('', [])]
 
1123
        self._dirblocks = [(b'', []), (b'', [])]
1081
1124
        current_block = self._dirblocks[0][1]
1082
 
        current_dirname = ''
1083
 
        root_key = ('', '')
 
1125
        current_dirname = b''
 
1126
        root_key = (b'', b'')
1084
1127
        append_entry = current_block.append
1085
1128
        for entry in new_entries:
1086
1129
            if entry[0][0] != current_dirname:
1102
1145
        # The above loop leaves the "root block" entries mixed with the
1103
1146
        # "contents-of-root block". But we don't want an if check on
1104
1147
        # all entries, so instead we just fix it up here.
1105
 
        if self._dirblocks[1] != ('', []):
 
1148
        if self._dirblocks[1] != (b'', []):
1106
1149
            raise ValueError("bad dirblock start %r" % (self._dirblocks[1],))
1107
1150
        root_block = []
1108
1151
        contents_of_root_block = []
1111
1154
                root_block.append(entry)
1112
1155
            else:
1113
1156
                contents_of_root_block.append(entry)
1114
 
        self._dirblocks[0] = ('', root_block)
1115
 
        self._dirblocks[1] = ('', contents_of_root_block)
 
1157
        self._dirblocks[0] = (b'', root_block)
 
1158
        self._dirblocks[1] = (b'', contents_of_root_block)
1116
1159
 
1117
1160
    def _entries_for_path(self, path):
1118
1161
        """Return a list with all the entries that match path for all ids."""
1119
1162
        dirname, basename = os.path.split(path)
1120
 
        key = (dirname, basename, '')
 
1163
        key = (dirname, basename, b'')
1121
1164
        block_index, present = self._find_block_index_from_key(key)
1122
1165
        if not present:
1123
1166
            # the block which should contain path is absent.
1146
1189
            # minikind
1147
1190
            entire_entry[tree_offset + 0] = tree_data[0]
1148
1191
            # size
1149
 
            entire_entry[tree_offset + 2] = str(tree_data[2])
 
1192
            entire_entry[tree_offset + 2] = b'%d' % tree_data[2]
1150
1193
            # executable
1151
1194
            entire_entry[tree_offset + 3] = DirState._to_yesno[tree_data[3]]
1152
 
        return '\0'.join(entire_entry)
 
1195
        return b'\0'.join(entire_entry)
1153
1196
 
1154
1197
    def _fields_per_entry(self):
1155
1198
        """How many null separated fields should be in each entry row.
1156
1199
 
1157
 
        Each line now has an extra '\n' field which is not used
 
1200
        Each line now has an extra '\\n' field which is not used
1158
1201
        so we just skip over it
1159
 
        entry size:
 
1202
 
 
1203
        entry size::
1160
1204
            3 fields for the key
1161
1205
            + number of fields per tree_data (5) * tree count
1162
1206
            + newline
1191
1235
 
1192
1236
        :return: The block index, True if the block for the key is present.
1193
1237
        """
1194
 
        if key[0:2] == ('', ''):
 
1238
        if key[0:2] == (b'', b''):
1195
1239
            return 0, True
1196
1240
        try:
1197
1241
            if (self._last_block_index is not None and
1203
1247
                                      cache=self._split_path_cache)
1204
1248
        # _right returns one-past-where-key is so we have to subtract
1205
1249
        # one to use it. we use _right here because there are two
1206
 
        # '' blocks - the root, and the contents of root
 
1250
        # b'' blocks - the root, and the contents of root
1207
1251
        # we always have a minimum of 2 in self._dirblocks: root and
1208
 
        # root-contents, and for '', we get 2 back, so this is
 
1252
        # root-contents, and for b'', we get 2 back, so this is
1209
1253
        # simple and correct:
1210
1254
        present = (block_index < len(self._dirblocks) and
1211
1255
            self._dirblocks[block_index][0] == key[0])
1253
1297
        result = DirState.initialize(dir_state_filename,
1254
1298
            sha1_provider=sha1_provider)
1255
1299
        try:
1256
 
            tree.lock_read()
1257
 
            try:
1258
 
                parent_ids = tree.get_parent_ids()
1259
 
                num_parents = len(parent_ids)
1260
 
                parent_trees = []
1261
 
                for parent_id in parent_ids:
1262
 
                    parent_tree = tree.branch.repository.revision_tree(parent_id)
1263
 
                    parent_trees.append((parent_id, parent_tree))
1264
 
                    parent_tree.lock_read()
1265
 
                result.set_parent_trees(parent_trees, [])
1266
 
                result.set_state_from_inventory(tree.inventory)
1267
 
            finally:
1268
 
                for revid, parent_tree in parent_trees:
1269
 
                    parent_tree.unlock()
1270
 
                tree.unlock()
 
1300
            with tree.lock_read():
 
1301
                try:
 
1302
                    parent_ids = tree.get_parent_ids()
 
1303
                    num_parents = len(parent_ids)
 
1304
                    parent_trees = []
 
1305
                    for parent_id in parent_ids:
 
1306
                        parent_tree = tree.branch.repository.revision_tree(parent_id)
 
1307
                        parent_trees.append((parent_id, parent_tree))
 
1308
                        parent_tree.lock_read()
 
1309
                    result.set_parent_trees(parent_trees, [])
 
1310
                    result.set_state_from_inventory(tree.root_inventory)
 
1311
                finally:
 
1312
                    for revid, parent_tree in parent_trees:
 
1313
                        parent_tree.unlock()
1271
1314
        except:
1272
1315
            # The caller won't have a chance to unlock this, so make sure we
1273
1316
            # cleanup ourselves
1275
1318
            raise
1276
1319
        return result
1277
1320
 
 
1321
    def _check_delta_is_valid(self, delta):
 
1322
        delta = list(inventory._check_delta_unique_ids(
 
1323
                     inventory._check_delta_unique_old_paths(
 
1324
                     inventory._check_delta_unique_new_paths(
 
1325
                     inventory._check_delta_ids_match_entry(
 
1326
                     inventory._check_delta_ids_are_valid(
 
1327
                     inventory._check_delta_new_path_entry_both_or_None(delta)))))))
 
1328
        def delta_key(d):
 
1329
            (old_path, new_path, file_id, new_entry) = d
 
1330
            if old_path is None:
 
1331
                old_path = ''
 
1332
            if new_path is None:
 
1333
                new_path = ''
 
1334
            return (old_path, new_path, file_id, new_entry)
 
1335
        delta.sort(key=delta_key, reverse=True)
 
1336
        return delta
 
1337
 
1278
1338
    def update_by_delta(self, delta):
1279
1339
        """Apply an inventory delta to the dirstate for tree 0
1280
1340
 
1298
1358
        new_ids = set()
1299
1359
        # This loop transforms the delta to single atomic operations that can
1300
1360
        # be executed and validated.
1301
 
        for old_path, new_path, file_id, inv_entry in sorted(
1302
 
            inventory._check_delta_unique_old_paths(
1303
 
            inventory._check_delta_unique_new_paths(
1304
 
            inventory._check_delta_ids_match_entry(
1305
 
            inventory._check_delta_ids_are_valid(
1306
 
            inventory._check_delta_new_path_entry_both_or_None(delta))))),
1307
 
            reverse=True):
 
1361
        delta = self._check_delta_is_valid(delta)
 
1362
        for old_path, new_path, file_id, inv_entry in delta:
 
1363
            if not isinstance(file_id, bytes):
 
1364
                raise AssertionError(
 
1365
                    "must be a utf8 file_id not %s" % (type(file_id), ))
1308
1366
            if (file_id in insertions) or (file_id in removals):
1309
 
                raise errors.InconsistentDelta(old_path or new_path, file_id,
 
1367
                self._raise_invalid(old_path or new_path, file_id,
1310
1368
                    "repeated file_id")
1311
1369
            if old_path is not None:
1312
1370
                old_path = old_path.encode('utf-8')
1315
1373
                new_ids.add(file_id)
1316
1374
            if new_path is not None:
1317
1375
                if inv_entry is None:
1318
 
                    raise errors.InconsistentDelta(new_path, file_id,
 
1376
                    self._raise_invalid(new_path, file_id,
1319
1377
                        "new_path with no entry")
1320
1378
                new_path = new_path.encode('utf-8')
1321
1379
                dirname_utf8, basename = osutils.split(new_path)
1323
1381
                    parents.add((dirname_utf8, inv_entry.parent_id))
1324
1382
                key = (dirname_utf8, basename, file_id)
1325
1383
                minikind = DirState._kind_to_minikind[inv_entry.kind]
1326
 
                if minikind == 't':
1327
 
                    fingerprint = inv_entry.reference_revision or ''
 
1384
                if minikind == b't':
 
1385
                    fingerprint = inv_entry.reference_revision or b''
1328
1386
                else:
1329
 
                    fingerprint = ''
 
1387
                    fingerprint = b''
1330
1388
                insertions[file_id] = (key, minikind, inv_entry.executable,
1331
1389
                                       fingerprint, new_path)
1332
1390
            # Transform moves into delete+add pairs
1351
1409
                                               fingerprint, new_child_path)
1352
1410
        self._check_delta_ids_absent(new_ids, delta, 0)
1353
1411
        try:
1354
 
            self._apply_removals(removals.iteritems())
1355
 
            self._apply_insertions(insertions.values())
 
1412
            self._apply_removals(viewitems(removals))
 
1413
            self._apply_insertions(viewvalues(insertions))
1356
1414
            # Validate parents
1357
1415
            self._after_delta_check_parents(parents, 0)
1358
 
        except errors.BzrError, e:
 
1416
        except errors.BzrError as e:
1359
1417
            self._changes_aborted = True
1360
1418
            if 'integrity error' not in str(e):
1361
1419
                raise
1362
1420
            # _get_entry raises BzrError when a request is inconsistent; we
1363
1421
            # want such errors to be shown as InconsistentDelta - and that 
1364
1422
            # fits the behaviour we trigger.
1365
 
            raise errors.InconsistentDeltaDelta(delta, "error from _get_entry.")
 
1423
            raise errors.InconsistentDeltaDelta(delta,
 
1424
                "error from _get_entry. %s" % (e,))
1366
1425
 
1367
1426
    def _apply_removals(self, removals):
1368
1427
        for file_id, path in sorted(removals, reverse=True,
1373
1432
            try:
1374
1433
                entry = self._dirblocks[block_i][1][entry_i]
1375
1434
            except IndexError:
1376
 
                self._changes_aborted = True
1377
 
                raise errors.InconsistentDelta(path, file_id,
 
1435
                self._raise_invalid(path, file_id,
1378
1436
                    "Wrong path for old path.")
1379
 
            if not f_present or entry[1][0][0] in 'ar':
1380
 
                self._changes_aborted = True
1381
 
                raise errors.InconsistentDelta(path, file_id,
 
1437
            if not f_present or entry[1][0][0] in (b'a', b'r'):
 
1438
                self._raise_invalid(path, file_id,
1382
1439
                    "Wrong path for old path.")
1383
1440
            if file_id != entry[0][2]:
1384
 
                self._changes_aborted = True
1385
 
                raise errors.InconsistentDelta(path, file_id,
 
1441
                self._raise_invalid(path, file_id,
1386
1442
                    "Attempt to remove path has wrong id - found %r."
1387
1443
                    % entry[0][2])
1388
1444
            self._make_absent(entry)
1392
1448
            # is rare enough it shouldn't be an issue (famous last words?) RBC
1393
1449
            # 20080730.
1394
1450
            block_i, entry_i, d_present, f_present = \
1395
 
                self._get_block_entry_index(path, '', 0)
 
1451
                self._get_block_entry_index(path, b'', 0)
1396
1452
            if d_present:
1397
1453
                # The dir block is still present in the dirstate; this could
1398
1454
                # be due to it being in a parent tree, or a corrupt delta.
1399
1455
                for child_entry in self._dirblocks[block_i][1]:
1400
 
                    if child_entry[1][0][0] not in ('r', 'a'):
1401
 
                        self._changes_aborted = True
1402
 
                        raise errors.InconsistentDelta(path, entry[0][2],
 
1456
                    if child_entry[1][0][0] not in (b'r', b'a'):
 
1457
                        self._raise_invalid(path, entry[0][2],
1403
1458
                            "The file id was deleted but its children were "
1404
1459
                            "not deleted.")
1405
1460
 
1409
1464
                self.update_minimal(key, minikind, executable, fingerprint,
1410
1465
                                    path_utf8=path_utf8)
1411
1466
        except errors.NotVersionedError:
1412
 
            self._changes_aborted = True
1413
 
            raise errors.InconsistentDelta(path_utf8.decode('utf8'), key[2],
 
1467
            self._raise_invalid(path_utf8.decode('utf8'), key[2],
1414
1468
                "Missing parent")
1415
1469
 
1416
1470
    def update_basis_by_delta(self, delta, new_revid):
1424
1478
        Note that an exception during the operation of this method will leave
1425
1479
        the dirstate in a corrupt state where it should not be saved.
1426
1480
 
1427
 
        Finally, we expect all changes to be synchronising the basis tree with
1428
 
        the working tree.
1429
 
 
1430
1481
        :param new_revid: The new revision id for the trees parent.
1431
1482
        :param delta: An inventory delta (see apply_inventory_delta) describing
1432
1483
            the changes from the current left most parent revision to new_revid.
1444
1495
 
1445
1496
        self._parents[0] = new_revid
1446
1497
 
1447
 
        delta = sorted(delta, reverse=True)
 
1498
        delta = self._check_delta_is_valid(delta)
1448
1499
        adds = []
1449
1500
        changes = []
1450
1501
        deletes = []
1470
1521
        # ids.
1471
1522
        new_ids = set()
1472
1523
        for old_path, new_path, file_id, inv_entry in delta:
 
1524
            if file_id.__class__ is not bytes:
 
1525
                raise AssertionError(
 
1526
                    "must be a utf8 file_id not %s" % (type(file_id), ))
1473
1527
            if inv_entry is not None and file_id != inv_entry.file_id:
1474
 
                raise errors.InconsistentDelta(new_path, file_id,
 
1528
                self._raise_invalid(new_path, file_id,
1475
1529
                    "mismatched entry file_id %r" % inv_entry)
1476
 
            if new_path is not None:
 
1530
            if new_path is None:
 
1531
                new_path_utf8 = None
 
1532
            else:
1477
1533
                if inv_entry is None:
1478
 
                    raise errors.InconsistentDelta(new_path, file_id,
 
1534
                    self._raise_invalid(new_path, file_id,
1479
1535
                        "new_path with no entry")
1480
1536
                new_path_utf8 = encode(new_path)
1481
1537
                # note the parent for validation
1483
1539
                if basename_utf8:
1484
1540
                    parents.add((dirname_utf8, inv_entry.parent_id))
1485
1541
            if old_path is None:
1486
 
                adds.append((None, encode(new_path), file_id,
 
1542
                old_path_utf8 = None
 
1543
            else:
 
1544
                old_path_utf8 = encode(old_path)
 
1545
            if old_path is None:
 
1546
                adds.append((None, new_path_utf8, file_id,
1487
1547
                    inv_to_entry(inv_entry), True))
1488
1548
                new_ids.add(file_id)
1489
1549
            elif new_path is None:
1490
 
                deletes.append((encode(old_path), None, file_id, None, True))
1491
 
            elif (old_path, new_path) != root_only:
 
1550
                deletes.append((old_path_utf8, None, file_id, None, True))
 
1551
            elif (old_path, new_path) == root_only:
 
1552
                # change things in-place
 
1553
                # Note: the case of a parent directory changing its file_id
 
1554
                #       tends to break optimizations here, because officially
 
1555
                #       the file has actually been moved, it just happens to
 
1556
                #       end up at the same path. If we can figure out how to
 
1557
                #       handle that case, we can avoid a lot of add+delete
 
1558
                #       pairs for objects that stay put.
 
1559
                # elif old_path == new_path:
 
1560
                changes.append((old_path_utf8, new_path_utf8, file_id,
 
1561
                                inv_to_entry(inv_entry)))
 
1562
            else:
1492
1563
                # Renames:
1493
1564
                # Because renames must preserve their children we must have
1494
1565
                # processed all relocations and removes before hand. The sort
1497
1568
                # pair will result in the deleted item being reinserted, or
1498
1569
                # renamed items being reinserted twice - and possibly at the
1499
1570
                # wrong place. Splitting into a delete/add pair also simplifies
1500
 
                # the handling of entries with ('f', ...), ('r' ...) because
1501
 
                # the target of the 'r' is old_path here, and we add that to
 
1571
                # the handling of entries with (b'f', ...), (b'r' ...) because
 
1572
                # the target of the b'r' is old_path here, and we add that to
1502
1573
                # deletes, meaning that the add handler does not need to check
1503
 
                # for 'r' items on every pass.
 
1574
                # for b'r' items on every pass.
1504
1575
                self._update_basis_apply_deletes(deletes)
1505
1576
                deletes = []
1506
1577
                # Split into an add/delete pair recursively.
1507
 
                adds.append((None, new_path_utf8, file_id,
1508
 
                    inv_to_entry(inv_entry), False))
 
1578
                adds.append((old_path_utf8, new_path_utf8, file_id,
 
1579
                             inv_to_entry(inv_entry), False))
1509
1580
                # Expunge deletes that we've seen so that deleted/renamed
1510
1581
                # children of a rename directory are handled correctly.
1511
 
                new_deletes = reversed(list(self._iter_child_entries(1,
1512
 
                    encode(old_path))))
 
1582
                new_deletes = reversed(list(
 
1583
                    self._iter_child_entries(1, old_path_utf8)))
1513
1584
                # Remove the current contents of the tree at orig_path, and
1514
1585
                # reinsert at the correct new path.
1515
1586
                for entry in new_deletes:
1516
 
                    if entry[0][0]:
1517
 
                        source_path = entry[0][0] + '/' + entry[0][1]
 
1587
                    child_dirname, child_basename, child_file_id = entry[0]
 
1588
                    if child_dirname:
 
1589
                        source_path = child_dirname + b'/' + child_basename
1518
1590
                    else:
1519
 
                        source_path = entry[0][1]
 
1591
                        source_path = child_basename
1520
1592
                    if new_path_utf8:
1521
 
                        target_path = new_path_utf8 + source_path[len(old_path):]
 
1593
                        target_path = \
 
1594
                            new_path_utf8 + source_path[len(old_path_utf8):]
1522
1595
                    else:
1523
 
                        if old_path == '':
 
1596
                        if old_path_utf8 == b'':
1524
1597
                            raise AssertionError("cannot rename directory to"
1525
 
                                " itself")
1526
 
                        target_path = source_path[len(old_path) + 1:]
 
1598
                                                 " itself")
 
1599
                        target_path = source_path[len(old_path_utf8) + 1:]
1527
1600
                    adds.append((None, target_path, entry[0][2], entry[1][1], False))
1528
1601
                    deletes.append(
1529
1602
                        (source_path, target_path, entry[0][2], None, False))
1530
1603
                deletes.append(
1531
 
                    (encode(old_path), new_path, file_id, None, False))
1532
 
            else:
1533
 
                # changes to just the root should not require remove/insertion
1534
 
                # of everything.
1535
 
                changes.append((encode(old_path), encode(new_path), file_id,
1536
 
                    inv_to_entry(inv_entry)))
 
1604
                    (old_path_utf8, new_path_utf8, file_id, None, False))
 
1605
 
1537
1606
        self._check_delta_ids_absent(new_ids, delta, 1)
1538
1607
        try:
1539
1608
            # Finish expunging deletes/first half of renames.
1544
1613
            self._update_basis_apply_changes(changes)
1545
1614
            # Validate parents
1546
1615
            self._after_delta_check_parents(parents, 1)
1547
 
        except errors.BzrError, e:
 
1616
        except errors.BzrError as e:
1548
1617
            self._changes_aborted = True
1549
1618
            if 'integrity error' not in str(e):
1550
1619
                raise
1551
1620
            # _get_entry raises BzrError when a request is inconsistent; we
1552
 
            # want such errors to be shown as InconsistentDelta - and that 
1553
 
            # fits the behaviour we trigger. Partof this is driven by dirstate
1554
 
            # only supporting deltas that turn the basis into a closer fit to
1555
 
            # the active tree.
1556
 
            raise errors.InconsistentDeltaDelta(delta, "error from _get_entry.")
 
1621
            # want such errors to be shown as InconsistentDelta - and that
 
1622
            # fits the behaviour we trigger.
 
1623
            raise errors.InconsistentDeltaDelta(delta,
 
1624
                "error from _get_entry. %s" % (e,))
1557
1625
 
1558
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
1559
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
1626
        self._mark_modified(header_modified=True)
1560
1627
        self._id_index = None
1561
1628
        return
1562
1629
 
1566
1633
            return
1567
1634
        id_index = self._get_id_index()
1568
1635
        for file_id in new_ids:
1569
 
            for key in id_index.get(file_id, []):
 
1636
            for key in id_index.get(file_id, ()):
1570
1637
                block_i, entry_i, d_present, f_present = \
1571
1638
                    self._get_block_entry_index(key[0], key[1], tree_index)
1572
1639
                if not f_present:
1576
1643
                if entry[0][2] != file_id:
1577
1644
                    # Different file_id, so not what we want.
1578
1645
                    continue
1579
 
                # NB: No changes made before this helper is called, so no need
1580
 
                # to set the _changes_aborted flag.
1581
 
                raise errors.InconsistentDelta(
1582
 
                    ("%s/%s" % key[0:2]).decode('utf8'), file_id,
 
1646
                self._raise_invalid((b"%s/%s" % key[0:2]).decode('utf8'), file_id,
1583
1647
                    "This file_id is new in the delta but already present in "
1584
1648
                    "the target")
1585
1649
 
 
1650
    def _raise_invalid(self, path, file_id, reason):
 
1651
        self._changes_aborted = True
 
1652
        raise errors.InconsistentDelta(path, file_id, reason)
 
1653
 
1586
1654
    def _update_basis_apply_adds(self, adds):
1587
1655
        """Apply a sequence of adds to tree 1 during update_basis_by_delta.
1588
1656
 
1596
1664
        """
1597
1665
        # Adds are accumulated partly from renames, so can be in any input
1598
1666
        # order - sort it.
1599
 
        adds.sort()
 
1667
        # TODO: we may want to sort in dirblocks order. That way each entry
 
1668
        #       will end up in the same directory, allowing the _get_entry
 
1669
        #       fast-path for looking up 2 items in the same dir work.
 
1670
        adds.sort(key=lambda x: x[1])
1600
1671
        # adds is now in lexographic order, which places all parents before
1601
1672
        # their children, so we can process it linearly.
1602
 
        absent = 'ar'
 
1673
        st = static_tuple.StaticTuple
1603
1674
        for old_path, new_path, file_id, new_details, real_add in adds:
1604
 
            # the entry for this file_id must be in tree 0.
1605
 
            entry = self._get_entry(0, file_id, new_path)
1606
 
            if entry[0] is None or entry[0][2] != file_id:
1607
 
                self._changes_aborted = True
1608
 
                raise errors.InconsistentDelta(new_path, file_id,
1609
 
                    'working tree does not contain new entry')
1610
 
            if real_add and entry[1][1][0] not in absent:
1611
 
                self._changes_aborted = True
1612
 
                raise errors.InconsistentDelta(new_path, file_id,
1613
 
                    'The entry was considered to be a genuinely new record,'
1614
 
                    ' but there was already an old record for it.')
1615
 
            # We don't need to update the target of an 'r' because the handling
1616
 
            # of renames turns all 'r' situations into a delete at the original
1617
 
            # location.
1618
 
            entry[1][1] = new_details
 
1675
            dirname, basename = osutils.split(new_path)
 
1676
            entry_key = st(dirname, basename, file_id)
 
1677
            block_index, present = self._find_block_index_from_key(entry_key)
 
1678
            if not present:
 
1679
                # The block where we want to put the file is not present.
 
1680
                # However, it might have just been an empty directory. Look for
 
1681
                # the parent in the basis-so-far before throwing an error.
 
1682
                parent_dir, parent_base = osutils.split(dirname)
 
1683
                parent_block_idx, parent_entry_idx, _, parent_present = \
 
1684
                    self._get_block_entry_index(parent_dir, parent_base, 1)
 
1685
                if not parent_present:
 
1686
                    self._raise_invalid(new_path, file_id,
 
1687
                        "Unable to find block for this record."
 
1688
                        " Was the parent added?")
 
1689
                self._ensure_block(parent_block_idx, parent_entry_idx, dirname)
 
1690
 
 
1691
            block = self._dirblocks[block_index][1]
 
1692
            entry_index, present = self._find_entry_index(entry_key, block)
 
1693
            if real_add:
 
1694
                if old_path is not None:
 
1695
                    self._raise_invalid(new_path, file_id,
 
1696
                        'considered a real add but still had old_path at %s'
 
1697
                        % (old_path,))
 
1698
            if present:
 
1699
                entry = block[entry_index]
 
1700
                basis_kind = entry[1][1][0]
 
1701
                if basis_kind == b'a':
 
1702
                    entry[1][1] = new_details
 
1703
                elif basis_kind == b'r':
 
1704
                    raise NotImplementedError()
 
1705
                else:
 
1706
                    self._raise_invalid(new_path, file_id,
 
1707
                        "An entry was marked as a new add"
 
1708
                        " but the basis target already existed")
 
1709
            else:
 
1710
                # The exact key was not found in the block. However, we need to
 
1711
                # check if there is a key next to us that would have matched.
 
1712
                # We only need to check 2 locations, because there are only 2
 
1713
                # trees present.
 
1714
                for maybe_index in range(entry_index-1, entry_index+1):
 
1715
                    if maybe_index < 0 or maybe_index >= len(block):
 
1716
                        continue
 
1717
                    maybe_entry = block[maybe_index]
 
1718
                    if maybe_entry[0][:2] != (dirname, basename):
 
1719
                        # Just a random neighbor
 
1720
                        continue
 
1721
                    if maybe_entry[0][2] == file_id:
 
1722
                        raise AssertionError(
 
1723
                            '_find_entry_index didnt find a key match'
 
1724
                            ' but walking the data did, for %s'
 
1725
                            % (entry_key,))
 
1726
                    basis_kind = maybe_entry[1][1][0]
 
1727
                    if basis_kind not in (b'a', b'r'):
 
1728
                        self._raise_invalid(new_path, file_id,
 
1729
                            "we have an add record for path, but the path"
 
1730
                            " is already present with another file_id %s"
 
1731
                            % (maybe_entry[0][2],))
 
1732
 
 
1733
                entry = (entry_key, [DirState.NULL_PARENT_DETAILS,
 
1734
                                     new_details])
 
1735
                block.insert(entry_index, entry)
 
1736
 
 
1737
            active_kind = entry[1][0][0]
 
1738
            if active_kind == b'a':
 
1739
                # The active record shows up as absent, this could be genuine,
 
1740
                # or it could be present at some other location. We need to
 
1741
                # verify.
 
1742
                id_index = self._get_id_index()
 
1743
                # The id_index may not be perfectly accurate for tree1, because
 
1744
                # we haven't been keeping it updated. However, it should be
 
1745
                # fine for tree0, and that gives us enough info for what we
 
1746
                # need
 
1747
                keys = id_index.get(file_id, ())
 
1748
                for key in keys:
 
1749
                    block_i, entry_i, d_present, f_present = \
 
1750
                        self._get_block_entry_index(key[0], key[1], 0)
 
1751
                    if not f_present:
 
1752
                        continue
 
1753
                    active_entry = self._dirblocks[block_i][1][entry_i]
 
1754
                    if (active_entry[0][2] != file_id):
 
1755
                        # Some other file is at this path, we don't need to
 
1756
                        # link it.
 
1757
                        continue
 
1758
                    real_active_kind = active_entry[1][0][0]
 
1759
                    if real_active_kind in (b'a', b'r'):
 
1760
                        # We found a record, which was not *this* record,
 
1761
                        # which matches the file_id, but is not actually
 
1762
                        # present. Something seems *really* wrong.
 
1763
                        self._raise_invalid(new_path, file_id,
 
1764
                            "We found a tree0 entry that doesnt make sense")
 
1765
                    # Now, we've found a tree0 entry which matches the file_id
 
1766
                    # but is at a different location. So update them to be
 
1767
                    # rename records.
 
1768
                    active_dir, active_name = active_entry[0][:2]
 
1769
                    if active_dir:
 
1770
                        active_path = active_dir + b'/' + active_name
 
1771
                    else:
 
1772
                        active_path = active_name
 
1773
                    active_entry[1][1] = st(b'r', new_path, 0, False, b'')
 
1774
                    entry[1][0] = st(b'r', active_path, 0, False, b'')
 
1775
            elif active_kind == b'r':
 
1776
                raise NotImplementedError()
 
1777
 
 
1778
            new_kind = new_details[0]
 
1779
            if new_kind == b'd':
 
1780
                self._ensure_block(block_index, entry_index, new_path)
1619
1781
 
1620
1782
    def _update_basis_apply_changes(self, changes):
1621
1783
        """Apply a sequence of changes to tree 1 during update_basis_by_delta.
1623
1785
        :param adds: A sequence of changes. Each change is a tuple:
1624
1786
            (path_utf8, path_utf8, file_id, (entry_details))
1625
1787
        """
1626
 
        absent = 'ar'
1627
1788
        for old_path, new_path, file_id, new_details in changes:
1628
1789
            # the entry for this file_id must be in tree 0.
1629
 
            entry = self._get_entry(0, file_id, new_path)
1630
 
            if entry[0] is None or entry[0][2] != file_id:
1631
 
                self._changes_aborted = True
1632
 
                raise errors.InconsistentDelta(new_path, file_id,
1633
 
                    'working tree does not contain new entry')
1634
 
            if (entry[1][0][0] in absent or
1635
 
                entry[1][1][0] in absent):
1636
 
                self._changes_aborted = True
1637
 
                raise errors.InconsistentDelta(new_path, file_id,
1638
 
                    'changed considered absent')
 
1790
            entry = self._get_entry(1, file_id, new_path)
 
1791
            if entry[0] is None or entry[1][1][0] in (b'a', b'r'):
 
1792
                self._raise_invalid(new_path, file_id,
 
1793
                    'changed entry considered not present')
1639
1794
            entry[1][1] = new_details
1640
1795
 
1641
1796
    def _update_basis_apply_deletes(self, deletes):
1653
1808
        null = DirState.NULL_PARENT_DETAILS
1654
1809
        for old_path, new_path, file_id, _, real_delete in deletes:
1655
1810
            if real_delete != (new_path is None):
1656
 
                self._changes_aborted = True
1657
 
                raise AssertionError("bad delete delta")
 
1811
                self._raise_invalid(old_path, file_id, "bad delete delta")
1658
1812
            # the entry for this file_id must be in tree 1.
1659
1813
            dirname, basename = osutils.split(old_path)
1660
1814
            block_index, entry_index, dir_present, file_present = \
1661
1815
                self._get_block_entry_index(dirname, basename, 1)
1662
1816
            if not file_present:
1663
 
                self._changes_aborted = True
1664
 
                raise errors.InconsistentDelta(old_path, file_id,
 
1817
                self._raise_invalid(old_path, file_id,
1665
1818
                    'basis tree does not contain removed entry')
1666
1819
            entry = self._dirblocks[block_index][1][entry_index]
 
1820
            # The state of the entry in the 'active' WT
 
1821
            active_kind = entry[1][0][0]
1667
1822
            if entry[0][2] != file_id:
1668
 
                self._changes_aborted = True
1669
 
                raise errors.InconsistentDelta(old_path, file_id,
 
1823
                self._raise_invalid(old_path, file_id,
1670
1824
                    'mismatched file_id in tree 1')
1671
 
            if real_delete:
1672
 
                if entry[1][0][0] != 'a':
1673
 
                    self._changes_aborted = True
1674
 
                    raise errors.InconsistentDelta(old_path, file_id,
1675
 
                            'This was marked as a real delete, but the WT state'
1676
 
                            ' claims that it still exists and is versioned.')
 
1825
            dir_block = ()
 
1826
            old_kind = entry[1][1][0]
 
1827
            if active_kind in b'ar':
 
1828
                # The active tree doesn't have this file_id.
 
1829
                # The basis tree is changing this record. If this is a
 
1830
                # rename, then we don't want the record here at all
 
1831
                # anymore. If it is just an in-place change, we want the
 
1832
                # record here, but we'll add it if we need to. So we just
 
1833
                # delete it
 
1834
                if active_kind == b'r':
 
1835
                    active_path = entry[1][0][1]
 
1836
                    active_entry = self._get_entry(0, file_id, active_path)
 
1837
                    if active_entry[1][1][0] != b'r':
 
1838
                        self._raise_invalid(old_path, file_id,
 
1839
                            "Dirstate did not have matching rename entries")
 
1840
                    elif active_entry[1][0][0] in b'ar':
 
1841
                        self._raise_invalid(old_path, file_id,
 
1842
                            "Dirstate had a rename pointing at an inactive"
 
1843
                            " tree0")
 
1844
                    active_entry[1][1] = null
1677
1845
                del self._dirblocks[block_index][1][entry_index]
 
1846
                if old_kind == b'd':
 
1847
                    # This was a directory, and the active tree says it
 
1848
                    # doesn't exist, and now the basis tree says it doesn't
 
1849
                    # exist. Remove its dirblock if present
 
1850
                    (dir_block_index,
 
1851
                     present) = self._find_block_index_from_key(
 
1852
                        (old_path, b'', b''))
 
1853
                    if present:
 
1854
                        dir_block = self._dirblocks[dir_block_index][1]
 
1855
                        if not dir_block:
 
1856
                            # This entry is empty, go ahead and just remove it
 
1857
                            del self._dirblocks[dir_block_index]
1678
1858
            else:
1679
 
                if entry[1][0][0] == 'a':
1680
 
                    self._changes_aborted = True
1681
 
                    raise errors.InconsistentDelta(old_path, file_id,
1682
 
                        'The entry was considered a rename, but the source path'
1683
 
                        ' is marked as absent.')
1684
 
                    # For whatever reason, we were asked to rename an entry
1685
 
                    # that was originally marked as deleted. This could be
1686
 
                    # because we are renaming the parent directory, and the WT
1687
 
                    # current state has the file marked as deleted.
1688
 
                elif entry[1][0][0] == 'r':
1689
 
                    # implement the rename
1690
 
                    del self._dirblocks[block_index][1][entry_index]
1691
 
                else:
1692
 
                    # it is being resurrected here, so blank it out temporarily.
1693
 
                    self._dirblocks[block_index][1][entry_index][1][1] = null
 
1859
                # There is still an active record, so just mark this
 
1860
                # removed.
 
1861
                entry[1][1] = null
 
1862
                block_i, entry_i, d_present, f_present = \
 
1863
                    self._get_block_entry_index(old_path, b'', 1)
 
1864
                if d_present:
 
1865
                    dir_block = self._dirblocks[block_i][1]
 
1866
            for child_entry in dir_block:
 
1867
                child_basis_kind = child_entry[1][1][0]
 
1868
                if child_basis_kind not in b'ar':
 
1869
                    self._raise_invalid(old_path, file_id,
 
1870
                        "The file id was deleted but its children were "
 
1871
                        "not deleted.")
1694
1872
 
1695
1873
    def _after_delta_check_parents(self, parents, index):
1696
1874
        """Check that parents required by the delta are all intact.
1705
1883
            # has the right file id.
1706
1884
            entry = self._get_entry(index, file_id, dirname_utf8)
1707
1885
            if entry[1] is None:
1708
 
                self._changes_aborted = True
1709
 
                raise errors.InconsistentDelta(dirname_utf8.decode('utf8'),
 
1886
                self._raise_invalid(dirname_utf8.decode('utf8'),
1710
1887
                    file_id, "This parent is not present.")
1711
1888
            # Parents of things must be directories
1712
 
            if entry[1][index][0] != 'd':
1713
 
                self._changes_aborted = True
1714
 
                raise errors.InconsistentDelta(dirname_utf8.decode('utf8'),
 
1889
            if entry[1][index][0] != b'd':
 
1890
                self._raise_invalid(dirname_utf8.decode('utf8'),
1715
1891
                    file_id, "This parent is not a directory.")
1716
1892
 
1717
1893
    def _observed_sha1(self, entry, sha1, stat_value,
1718
 
        _stat_to_minikind=_stat_to_minikind, _pack_stat=pack_stat):
 
1894
        _stat_to_minikind=_stat_to_minikind):
1719
1895
        """Note the sha1 of a file.
1720
1896
 
1721
1897
        :param entry: The entry the sha1 is for.
1723
1899
        :param stat_value: The os.lstat for the file.
1724
1900
        """
1725
1901
        try:
1726
 
            minikind = _stat_to_minikind[stat_value.st_mode & 0170000]
 
1902
            minikind = _stat_to_minikind[stat_value.st_mode & 0o170000]
1727
1903
        except KeyError:
1728
1904
            # Unhandled kind
1729
1905
            return None
1730
 
        packed_stat = _pack_stat(stat_value)
1731
 
        if minikind == 'f':
 
1906
        if minikind == b'f':
1732
1907
            if self._cutoff_time is None:
1733
1908
                self._sha_cutoff_time()
1734
1909
            if (stat_value.st_mtime < self._cutoff_time
1735
1910
                and stat_value.st_ctime < self._cutoff_time):
1736
 
                entry[1][0] = ('f', sha1, entry[1][0][2], entry[1][0][3],
1737
 
                    packed_stat)
1738
 
                self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
1911
                entry[1][0] = (b'f', sha1, stat_value.st_size, entry[1][0][3],
 
1912
                               pack_stat(stat_value))
 
1913
                self._mark_modified([entry])
1739
1914
 
1740
1915
    def _sha_cutoff_time(self):
1741
1916
        """Return cutoff time.
1778
1953
        #       higher level, because there either won't be anything on disk,
1779
1954
        #       or the thing on disk will be a file.
1780
1955
        fs_encoding = osutils._fs_enc
1781
 
        if isinstance(abspath, unicode):
 
1956
        if isinstance(abspath, text_type):
1782
1957
            # abspath is defined as the path to pass to lstat. readlink is
1783
1958
            # buggy in python < 2.6 (it doesn't encode unicode path into FS
1784
1959
            # encoding), so we need to encode ourselves knowing that unicode
1785
1960
            # paths are produced by UnicodeDirReader on purpose.
1786
1961
            abspath = abspath.encode(fs_encoding)
1787
1962
        target = os.readlink(abspath)
1788
 
        if fs_encoding not in ('UTF-8', 'US-ASCII', 'ANSI_X3.4-1968'):
 
1963
        if fs_encoding not in ('utf-8', 'ascii'):
1789
1964
            # Change encoding if needed
1790
1965
            target = target.decode(fs_encoding).encode('UTF-8')
1791
1966
        return target
1799
1974
        """Serialise the entire dirstate to a sequence of lines."""
1800
1975
        if (self._header_state == DirState.IN_MEMORY_UNMODIFIED and
1801
1976
            self._dirblock_state == DirState.IN_MEMORY_UNMODIFIED):
1802
 
            # read whats on disk.
 
1977
            # read what's on disk.
1803
1978
            self._state_file.seek(0)
1804
1979
            return self._state_file.readlines()
1805
1980
        lines = []
1806
1981
        lines.append(self._get_parents_line(self.get_parent_ids()))
1807
1982
        lines.append(self._get_ghosts_line(self._ghosts))
1808
 
        # append the root line which is special cased
1809
 
        lines.extend(map(self._entry_to_line, self._iter_entries()))
 
1983
        lines.extend(self._iter_entry_lines())
1810
1984
        return self._get_output_lines(lines)
1811
1985
 
1812
1986
    def _get_ghosts_line(self, ghost_ids):
1813
1987
        """Create a line for the state file for ghost information."""
1814
 
        return '\0'.join([str(len(ghost_ids))] + ghost_ids)
 
1988
        return b'\0'.join([b'%d' % len(ghost_ids)] + ghost_ids)
1815
1989
 
1816
1990
    def _get_parents_line(self, parent_ids):
1817
1991
        """Create a line for the state file for parents information."""
1818
 
        return '\0'.join([str(len(parent_ids))] + parent_ids)
 
1992
        return b'\0'.join([b'%d' % len(parent_ids)] + parent_ids)
 
1993
 
 
1994
    def _iter_entry_lines(self):
 
1995
        """Create lines for entries."""
 
1996
        return map(self._entry_to_line, self._iter_entries())
1819
1997
 
1820
1998
    def _get_fields_to_entry(self):
1821
1999
        """Get a function which converts entry fields into a entry record.
1835
2013
                        fields[3],                # minikind
1836
2014
                        fields[4],                # fingerprint
1837
2015
                        _int(fields[5]),          # size
1838
 
                        fields[6] == 'y',         # executable
 
2016
                        fields[6] == b'y',         # executable
1839
2017
                        fields[7],                # packed_stat or revision_id
1840
2018
                    )])
1841
2019
            return fields_to_entry_0_parents
1847
2025
                        fields[3],                # minikind
1848
2026
                        fields[4],                # fingerprint
1849
2027
                        _int(fields[5]),          # size
1850
 
                        fields[6] == 'y',         # executable
 
2028
                        fields[6] == b'y',         # executable
1851
2029
                        fields[7],                # packed_stat or revision_id
1852
2030
                    ),
1853
2031
                    ( # Parent 1
1854
2032
                        fields[8],                # minikind
1855
2033
                        fields[9],                # fingerprint
1856
2034
                        _int(fields[10]),         # size
1857
 
                        fields[11] == 'y',        # executable
 
2035
                        fields[11] == b'y',        # executable
1858
2036
                        fields[12],               # packed_stat or revision_id
1859
2037
                    ),
1860
2038
                    ])
1867
2045
                        fields[3],                # minikind
1868
2046
                        fields[4],                # fingerprint
1869
2047
                        _int(fields[5]),          # size
1870
 
                        fields[6] == 'y',         # executable
 
2048
                        fields[6] == b'y',         # executable
1871
2049
                        fields[7],                # packed_stat or revision_id
1872
2050
                    ),
1873
2051
                    ( # Parent 1
1874
2052
                        fields[8],                # minikind
1875
2053
                        fields[9],                # fingerprint
1876
2054
                        _int(fields[10]),         # size
1877
 
                        fields[11] == 'y',        # executable
 
2055
                        fields[11] == b'y',        # executable
1878
2056
                        fields[12],               # packed_stat or revision_id
1879
2057
                    ),
1880
2058
                    ( # Parent 2
1881
2059
                        fields[13],               # minikind
1882
2060
                        fields[14],               # fingerprint
1883
2061
                        _int(fields[15]),         # size
1884
 
                        fields[16] == 'y',        # executable
 
2062
                        fields[16] == b'y',        # executable
1885
2063
                        fields[17],               # packed_stat or revision_id
1886
2064
                    ),
1887
2065
                    ])
1892
2070
                trees = [(fields[cur],                # minikind
1893
2071
                          fields[cur+1],              # fingerprint
1894
2072
                          _int(fields[cur+2]),        # size
1895
 
                          fields[cur+3] == 'y',       # executable
 
2073
                          fields[cur+3] == b'y',       # executable
1896
2074
                          fields[cur+4],              # stat or revision_id
1897
 
                         ) for cur in xrange(3, len(fields)-1, 5)]
 
2075
                         ) for cur in range(3, len(fields)-1, 5)]
1898
2076
                return path_name_file_id_key, trees
1899
2077
            return fields_to_entry_n_parents
1900
2078
 
1924
2102
            tree present there.
1925
2103
        """
1926
2104
        self._read_dirblocks_if_needed()
1927
 
        key = dirname, basename, ''
 
2105
        key = dirname, basename, b''
1928
2106
        block_index, present = self._find_block_index_from_key(key)
1929
2107
        if not present:
1930
2108
            # no such directory - return the dir index and 0 for the row.
1934
2112
        # linear search through entries at this path to find the one
1935
2113
        # requested.
1936
2114
        while entry_index < len(block) and block[entry_index][0][1] == basename:
1937
 
            if block[entry_index][1][tree_index][0] not in 'ar':
 
2115
            if block[entry_index][1][tree_index][0] not in (b'a', b'r'):
1938
2116
                # neither absent or relocated
1939
2117
                return block_index, entry_index, True, True
1940
2118
            entry_index += 1
1941
2119
        return block_index, entry_index, True, False
1942
2120
 
1943
 
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None, include_deleted=False):
 
2121
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None,
 
2122
                   include_deleted=False):
1944
2123
        """Get the dirstate entry for path in tree tree_index.
1945
2124
 
1946
2125
        If either file_id or path is supplied, it is used as the key to lookup.
1961
2140
        """
1962
2141
        self._read_dirblocks_if_needed()
1963
2142
        if path_utf8 is not None:
1964
 
            if type(path_utf8) is not str:
1965
 
                raise errors.BzrError('path_utf8 is not a str: %s %r'
 
2143
            if not isinstance(path_utf8, bytes):
 
2144
                raise errors.BzrError('path_utf8 is not bytes: %s %r'
1966
2145
                    % (type(path_utf8), path_utf8))
1967
2146
            # path lookups are faster
1968
2147
            dirname, basename = osutils.split(path_utf8)
1971
2150
            if not file_present:
1972
2151
                return None, None
1973
2152
            entry = self._dirblocks[block_index][1][entry_index]
1974
 
            if not (entry[0][2] and entry[1][tree_index][0] not in ('a', 'r')):
 
2153
            if not (entry[0][2] and entry[1][tree_index][0] not in (b'a', b'r')):
1975
2154
                raise AssertionError('unversioned entry?')
1976
2155
            if fileid_utf8:
1977
2156
                if entry[0][2] != fileid_utf8:
1980
2159
                                          ' tree_index, file_id and path')
1981
2160
            return entry
1982
2161
        else:
1983
 
            possible_keys = self._get_id_index().get(fileid_utf8, None)
 
2162
            possible_keys = self._get_id_index().get(fileid_utf8, ())
1984
2163
            if not possible_keys:
1985
2164
                return None, None
1986
2165
            for key in possible_keys:
1999
2178
                    entry = self._dirblocks[block_index][1][entry_index]
2000
2179
                    # TODO: We might want to assert that entry[0][2] ==
2001
2180
                    #       fileid_utf8.
2002
 
                    if entry[1][tree_index][0] in 'fdlt':
 
2181
                    # GZ 2017-06-09: Hoist set of minkinds somewhere
 
2182
                    if entry[1][tree_index][0] in {b'f', b'd', b'l', b't'}:
2003
2183
                        # this is the result we are looking for: the
2004
2184
                        # real home of this file_id in this tree.
2005
2185
                        return entry
2006
 
                    if entry[1][tree_index][0] == 'a':
 
2186
                    if entry[1][tree_index][0] == b'a':
2007
2187
                        # there is no home for this entry in this tree
2008
2188
                        if include_deleted:
2009
2189
                            return entry
2010
2190
                        return None, None
2011
 
                    if entry[1][tree_index][0] != 'r':
 
2191
                    if entry[1][tree_index][0] != b'r':
2012
2192
                        raise AssertionError(
2013
2193
                            "entry %r has invalid minikind %r for tree %r" \
2014
2194
                            % (entry,
2040
2220
            sha1_provider = DefaultSHA1Provider()
2041
2221
        result = cls(path, sha1_provider)
2042
2222
        # root dir and root dir contents with no children.
2043
 
        empty_tree_dirblocks = [('', []), ('', [])]
 
2223
        empty_tree_dirblocks = [(b'', []), (b'', [])]
2044
2224
        # a new root directory, with a NULLSTAT.
2045
2225
        empty_tree_dirblocks[0][1].append(
2046
 
            (('', '', inventory.ROOT_ID), [
2047
 
                ('d', '', 0, False, DirState.NULLSTAT),
 
2226
            ((b'', b'', inventory.ROOT_ID), [
 
2227
                (b'd', b'', 0, False, DirState.NULLSTAT),
2048
2228
            ]))
2049
2229
        result.lock_write()
2050
2230
        try:
2068
2248
        minikind = DirState._kind_to_minikind[kind]
2069
2249
        tree_data = inv_entry.revision
2070
2250
        if kind == 'directory':
2071
 
            fingerprint = ''
 
2251
            fingerprint = b''
2072
2252
            size = 0
2073
2253
            executable = False
2074
2254
        elif kind == 'symlink':
2075
2255
            if inv_entry.symlink_target is None:
2076
 
                fingerprint = ''
 
2256
                fingerprint = b''
2077
2257
            else:
2078
2258
                fingerprint = inv_entry.symlink_target.encode('utf8')
2079
2259
            size = 0
2080
2260
            executable = False
2081
2261
        elif kind == 'file':
2082
 
            fingerprint = inv_entry.text_sha1 or ''
 
2262
            fingerprint = inv_entry.text_sha1 or b''
2083
2263
            size = inv_entry.text_size or 0
2084
2264
            executable = inv_entry.executable
2085
2265
        elif kind == 'tree-reference':
2086
 
            fingerprint = inv_entry.reference_revision or ''
 
2266
            fingerprint = inv_entry.reference_revision or b''
2087
2267
            size = 0
2088
2268
            executable = False
2089
2269
        else:
2090
2270
            raise Exception("can't pack %s" % inv_entry)
2091
 
        return (minikind, fingerprint, size, executable, tree_data)
 
2271
        return static_tuple.StaticTuple(minikind, fingerprint, size,
 
2272
                                        executable, tree_data)
2092
2273
 
2093
2274
    def _iter_child_entries(self, tree_index, path_utf8):
2094
2275
        """Iterate over all the entries that are children of path_utf.
2095
2276
 
2096
 
        This only returns entries that are present (not in 'a', 'r') in
 
2277
        This only returns entries that are present (not in b'a', b'r') in
2097
2278
        tree_index. tree_index data is not refreshed, so if tree 0 is used,
2098
2279
        results may differ from that obtained if paths were statted to
2099
2280
        determine what ones were directories.
2103
2284
        """
2104
2285
        pending_dirs = []
2105
2286
        next_pending_dirs = [path_utf8]
2106
 
        absent = 'ar'
 
2287
        absent = (b'a', b'r')
2107
2288
        while next_pending_dirs:
2108
2289
            pending_dirs = next_pending_dirs
2109
2290
            next_pending_dirs = []
2110
2291
            for path in pending_dirs:
2111
2292
                block_index, present = self._find_block_index_from_key(
2112
 
                    (path, '', ''))
 
2293
                    (path, b'', b''))
2113
2294
                if block_index == 0:
2114
2295
                    block_index = 1
2115
2296
                    if len(self._dirblocks) == 1:
2124
2305
                    kind = entry[1][tree_index][0]
2125
2306
                    if kind not in absent:
2126
2307
                        yield entry
2127
 
                    if kind == 'd':
 
2308
                    if kind == b'd':
2128
2309
                        if entry[0][0]:
2129
 
                            path = entry[0][0] + '/' + entry[0][1]
 
2310
                            path = entry[0][0] + b'/' + entry[0][1]
2130
2311
                        else:
2131
2312
                            path = entry[0][1]
2132
2313
                        next_pending_dirs.append(path)
2135
2316
        """Iterate over all the entries in the dirstate.
2136
2317
 
2137
2318
        Each yelt item is an entry in the standard format described in the
2138
 
        docstring of bzrlib.dirstate.
 
2319
        docstring of breezy.dirstate.
2139
2320
        """
2140
2321
        self._read_dirblocks_if_needed()
2141
2322
        for directory in self._dirblocks:
2143
2324
                yield entry
2144
2325
 
2145
2326
    def _get_id_index(self):
2146
 
        """Get an id index of self._dirblocks."""
 
2327
        """Get an id index of self._dirblocks.
 
2328
 
 
2329
        This maps from file_id => [(directory, name, file_id)] entries where
 
2330
        that file_id appears in one of the trees.
 
2331
        """
2147
2332
        if self._id_index is None:
2148
2333
            id_index = {}
2149
2334
            for key, tree_details in self._iter_entries():
2150
 
                id_index.setdefault(key[2], set()).add(key)
 
2335
                self._add_to_id_index(id_index, key)
2151
2336
            self._id_index = id_index
2152
2337
        return self._id_index
2153
2338
 
 
2339
    def _add_to_id_index(self, id_index, entry_key):
 
2340
        """Add this entry to the _id_index mapping."""
 
2341
        # This code used to use a set for every entry in the id_index. However,
 
2342
        # it is *rare* to have more than one entry. So a set is a large
 
2343
        # overkill. And even when we do, we won't ever have more than the
 
2344
        # number of parent trees. Which is still a small number (rarely >2). As
 
2345
        # such, we use a simple tuple, and do our own uniqueness checks. While
 
2346
        # the 'in' check is O(N) since N is nicely bounded it shouldn't ever
 
2347
        # cause quadratic failure.
 
2348
        file_id = entry_key[2]
 
2349
        entry_key = static_tuple.StaticTuple.from_sequence(entry_key)
 
2350
        if file_id not in id_index:
 
2351
            id_index[file_id] = static_tuple.StaticTuple(entry_key,)
 
2352
        else:
 
2353
            entry_keys = id_index[file_id]
 
2354
            if entry_key not in entry_keys:
 
2355
                id_index[file_id] = entry_keys + (entry_key,)
 
2356
 
 
2357
    def _remove_from_id_index(self, id_index, entry_key):
 
2358
        """Remove this entry from the _id_index mapping.
 
2359
 
 
2360
        It is an programming error to call this when the entry_key is not
 
2361
        already present.
 
2362
        """
 
2363
        file_id = entry_key[2]
 
2364
        entry_keys = list(id_index[file_id])
 
2365
        entry_keys.remove(entry_key)
 
2366
        id_index[file_id] = static_tuple.StaticTuple.from_sequence(entry_keys)
 
2367
 
2154
2368
    def _get_output_lines(self, lines):
2155
2369
        """Format lines for final output.
2156
2370
 
2158
2372
            path lines.
2159
2373
        """
2160
2374
        output_lines = [DirState.HEADER_FORMAT_3]
2161
 
        lines.append('') # a final newline
2162
 
        inventory_text = '\0\n\0'.join(lines)
2163
 
        output_lines.append('crc32: %s\n' % (zlib.crc32(inventory_text),))
 
2375
        lines.append(b'') # a final newline
 
2376
        inventory_text = b'\0\n\0'.join(lines)
 
2377
        output_lines.append(b'crc32: %d\n' % (zlib.crc32(inventory_text),))
2164
2378
        # -3, 1 for num parents, 1 for ghosts, 1 for final newline
2165
2379
        num_entries = len(lines)-3
2166
 
        output_lines.append('num_entries: %s\n' % (num_entries,))
 
2380
        output_lines.append(b'num_entries: %d\n' % (num_entries,))
2167
2381
        output_lines.append(inventory_text)
2168
2382
        return output_lines
2169
2383
 
2170
2384
    def _make_deleted_row(self, fileid_utf8, parents):
2171
2385
        """Return a deleted row for fileid_utf8."""
2172
 
        return ('/', 'RECYCLED.BIN', 'file', fileid_utf8, 0, DirState.NULLSTAT,
2173
 
            ''), parents
 
2386
        return (b'/', b'RECYCLED.BIN', b'file', fileid_utf8, 0, DirState.NULLSTAT,
 
2387
            b''), parents
2174
2388
 
2175
2389
    def _num_present_parents(self):
2176
2390
        """The number of parent entries in each record row."""
2177
2391
        return len(self._parents) - len(self._ghosts)
2178
2392
 
2179
 
    @staticmethod
2180
 
    def on_file(path, sha1_provider=None):
 
2393
    @classmethod
 
2394
    def on_file(cls, path, sha1_provider=None, worth_saving_limit=0):
2181
2395
        """Construct a DirState on the file at path "path".
2182
2396
 
2183
2397
        :param path: The path at which the dirstate file on disk should live.
2184
2398
        :param sha1_provider: an object meeting the SHA1Provider interface.
2185
2399
            If None, a DefaultSHA1Provider is used.
 
2400
        :param worth_saving_limit: when the exact number of hash changed
 
2401
            entries is known, only bother saving the dirstate if more than
 
2402
            this count of entries have changed. -1 means never save.
2186
2403
        :return: An unlocked DirState object, associated with the given path.
2187
2404
        """
2188
2405
        if sha1_provider is None:
2189
2406
            sha1_provider = DefaultSHA1Provider()
2190
 
        result = DirState(path, sha1_provider)
 
2407
        result = cls(path, sha1_provider,
 
2408
                     worth_saving_limit=worth_saving_limit)
2191
2409
        return result
2192
2410
 
2193
2411
    def _read_dirblocks_if_needed(self):
2211
2429
        """
2212
2430
        self._read_prelude()
2213
2431
        parent_line = self._state_file.readline()
2214
 
        info = parent_line.split('\0')
 
2432
        info = parent_line.split(b'\0')
2215
2433
        num_parents = int(info[0])
2216
2434
        self._parents = info[1:-1]
2217
2435
        ghost_line = self._state_file.readline()
2218
 
        info = ghost_line.split('\0')
 
2436
        info = ghost_line.split(b'\0')
2219
2437
        num_ghosts = int(info[1])
2220
2438
        self._ghosts = info[2:-1]
2221
2439
        self._header_state = DirState.IN_MEMORY_UNMODIFIED
2243
2461
            raise errors.BzrError(
2244
2462
                'invalid header line: %r' % (header,))
2245
2463
        crc_line = self._state_file.readline()
2246
 
        if not crc_line.startswith('crc32: '):
 
2464
        if not crc_line.startswith(b'crc32: '):
2247
2465
            raise errors.BzrError('missing crc32 checksum: %r' % crc_line)
2248
 
        self.crc_expected = int(crc_line[len('crc32: '):-1])
 
2466
        self.crc_expected = int(crc_line[len(b'crc32: '):-1])
2249
2467
        num_entries_line = self._state_file.readline()
2250
 
        if not num_entries_line.startswith('num_entries: '):
 
2468
        if not num_entries_line.startswith(b'num_entries: '):
2251
2469
            raise errors.BzrError('missing num_entries line')
2252
 
        self._num_entries = int(num_entries_line[len('num_entries: '):-1])
 
2470
        self._num_entries = int(num_entries_line[len(b'num_entries: '):-1])
2253
2471
 
2254
 
    def sha1_from_stat(self, path, stat_result, _pack_stat=pack_stat):
 
2472
    def sha1_from_stat(self, path, stat_result):
2255
2473
        """Find a sha1 given a stat lookup."""
2256
 
        return self._get_packed_stat_index().get(_pack_stat(stat_result), None)
 
2474
        return self._get_packed_stat_index().get(pack_stat(stat_result), None)
2257
2475
 
2258
2476
    def _get_packed_stat_index(self):
2259
2477
        """Get a packed_stat index of self._dirblocks."""
2260
2478
        if self._packed_stat_index is None:
2261
2479
            index = {}
2262
2480
            for key, tree_details in self._iter_entries():
2263
 
                if tree_details[0][0] == 'f':
 
2481
                if tree_details[0][0] == b'f':
2264
2482
                    index[tree_details[0][4]] = tree_details[0][1]
2265
2483
            self._packed_stat_index = index
2266
2484
        return self._packed_stat_index
2285
2503
            trace.mutter('Not saving DirState because '
2286
2504
                    '_changes_aborted is set.')
2287
2505
            return
2288
 
        if (self._header_state == DirState.IN_MEMORY_MODIFIED or
2289
 
            self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
 
2506
        # TODO: Since we now distinguish IN_MEMORY_MODIFIED from
 
2507
        #       IN_MEMORY_HASH_MODIFIED, we should only fail quietly if we fail
 
2508
        #       to save an IN_MEMORY_HASH_MODIFIED, and fail *noisily* if we
 
2509
        #       fail to save IN_MEMORY_MODIFIED
 
2510
        if not self._worth_saving():
 
2511
            return
2290
2512
 
2291
 
            grabbed_write_lock = False
2292
 
            if self._lock_state != 'w':
2293
 
                grabbed_write_lock, new_lock = self._lock_token.temporary_write_lock()
2294
 
                # Switch over to the new lock, as the old one may be closed.
 
2513
        grabbed_write_lock = False
 
2514
        if self._lock_state != 'w':
 
2515
            grabbed_write_lock, new_lock = self._lock_token.temporary_write_lock()
 
2516
            # Switch over to the new lock, as the old one may be closed.
 
2517
            # TODO: jam 20070315 We should validate the disk file has
 
2518
            #       not changed contents, since temporary_write_lock may
 
2519
            #       not be an atomic operation.
 
2520
            self._lock_token = new_lock
 
2521
            self._state_file = new_lock.f
 
2522
            if not grabbed_write_lock:
 
2523
                # We couldn't grab a write lock, so we switch back to a read one
 
2524
                return
 
2525
        try:
 
2526
            lines = self.get_lines()
 
2527
            self._state_file.seek(0)
 
2528
            self._state_file.writelines(lines)
 
2529
            self._state_file.truncate()
 
2530
            self._state_file.flush()
 
2531
            self._maybe_fdatasync()
 
2532
            self._mark_unmodified()
 
2533
        finally:
 
2534
            if grabbed_write_lock:
 
2535
                self._lock_token = self._lock_token.restore_read_lock()
 
2536
                self._state_file = self._lock_token.f
2295
2537
                # TODO: jam 20070315 We should validate the disk file has
2296
 
                #       not changed contents. Since temporary_write_lock may
 
2538
                #       not changed contents. Since restore_read_lock may
2297
2539
                #       not be an atomic operation.
2298
 
                self._lock_token = new_lock
2299
 
                self._state_file = new_lock.f
2300
 
                if not grabbed_write_lock:
2301
 
                    # We couldn't grab a write lock, so we switch back to a read one
2302
 
                    return
2303
 
            try:
2304
 
                self._state_file.seek(0)
2305
 
                self._state_file.writelines(self.get_lines())
2306
 
                self._state_file.truncate()
2307
 
                self._state_file.flush()
2308
 
                self._header_state = DirState.IN_MEMORY_UNMODIFIED
2309
 
                self._dirblock_state = DirState.IN_MEMORY_UNMODIFIED
2310
 
            finally:
2311
 
                if grabbed_write_lock:
2312
 
                    self._lock_token = self._lock_token.restore_read_lock()
2313
 
                    self._state_file = self._lock_token.f
2314
 
                    # TODO: jam 20070315 We should validate the disk file has
2315
 
                    #       not changed contents. Since restore_read_lock may
2316
 
                    #       not be an atomic operation.
 
2540
 
 
2541
    def _maybe_fdatasync(self):
 
2542
        """Flush to disk if possible and if not configured off."""
 
2543
        if self._config_stack.get('dirstate.fdatasync'):
 
2544
            osutils.fdatasync(self._state_file.fileno())
 
2545
 
 
2546
    def _worth_saving(self):
 
2547
        """Is it worth saving the dirstate or not?"""
 
2548
        if (self._header_state == DirState.IN_MEMORY_MODIFIED
 
2549
            or self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
 
2550
            return True
 
2551
        if self._dirblock_state == DirState.IN_MEMORY_HASH_MODIFIED:
 
2552
            if self._worth_saving_limit == -1:
 
2553
                # We never save hash changes when the limit is -1
 
2554
                return False
 
2555
            # If we're using smart saving and only a small number of
 
2556
            # entries have changed their hash, don't bother saving. John has
 
2557
            # suggested using a heuristic here based on the size of the
 
2558
            # changed files and/or tree. For now, we go with a configurable
 
2559
            # number of changes, keeping the calculation time
 
2560
            # as low overhead as possible. (This also keeps all existing
 
2561
            # tests passing as the default is 0, i.e. always save.)
 
2562
            if len(self._known_hash_changes) >= self._worth_saving_limit:
 
2563
                return True
 
2564
        return False
2317
2565
 
2318
2566
    def _set_data(self, parent_ids, dirblocks):
2319
2567
        """Set the full dirstate data in memory.
2328
2576
        """
2329
2577
        # our memory copy is now authoritative.
2330
2578
        self._dirblocks = dirblocks
2331
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
2332
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2579
        self._mark_modified(header_modified=True)
2333
2580
        self._parents = list(parent_ids)
2334
2581
        self._id_index = None
2335
2582
        self._packed_stat_index = None
2337
2584
    def set_path_id(self, path, new_id):
2338
2585
        """Change the id of path to new_id in the current working tree.
2339
2586
 
2340
 
        :param path: The path inside the tree to set - '' is the root, 'foo'
 
2587
        :param path: The path inside the tree to set - b'' is the root, 'foo'
2341
2588
            is the path foo in the root.
2342
2589
        :param new_id: The new id to assign to the path. This must be a utf8
2343
2590
            file id (not unicode, and not None).
2351
2598
        if entry[0][2] == new_id:
2352
2599
            # Nothing to change.
2353
2600
            return
 
2601
        if new_id.__class__ != bytes:
 
2602
            raise AssertionError(
 
2603
                "must be a utf8 file_id not %s" % (type(new_id), ))
2354
2604
        # mark the old path absent, and insert a new root path
2355
2605
        self._make_absent(entry)
2356
 
        self.update_minimal(('', '', new_id), 'd',
2357
 
            path_utf8='', packed_stat=entry[1][0][4])
2358
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2606
        self.update_minimal((b'', b'', new_id), b'd',
 
2607
            path_utf8=b'', packed_stat=entry[1][0][4])
 
2608
        self._mark_modified()
2359
2609
 
2360
2610
    def set_parent_trees(self, trees, ghosts):
2361
2611
        """Set the parent trees for the dirstate.
2406
2656
        parent_trees = [tree for rev_id, tree in trees if rev_id not in ghosts]
2407
2657
        # how many trees do we end up with
2408
2658
        parent_count = len(parent_trees)
 
2659
        st = static_tuple.StaticTuple
2409
2660
 
2410
2661
        # one: the current tree
2411
2662
        for entry in self._iter_entries():
2412
2663
            # skip entries not in the current tree
2413
 
            if entry[1][0][0] in 'ar': # absent, relocated
 
2664
            if entry[1][0][0] in (b'a', b'r'): # absent, relocated
2414
2665
                continue
2415
2666
            by_path[entry[0]] = [entry[1][0]] + \
2416
2667
                [DirState.NULL_PARENT_DETAILS] * parent_count
2417
 
            id_index[entry[0][2]] = set([entry[0]])
 
2668
            # TODO: Possibly inline this, since we know it isn't present yet
 
2669
            #       id_index[entry[0][2]] = (entry[0],)
 
2670
            self._add_to_id_index(id_index, entry[0])
2418
2671
 
2419
2672
        # now the parent trees:
2420
2673
        for tree_index, tree in enumerate(parent_trees):
2426
2679
            # the suffix is from tree_index+1:parent_count+1.
2427
2680
            new_location_suffix = [DirState.NULL_PARENT_DETAILS] * (parent_count - tree_index)
2428
2681
            # now stitch in all the entries from this tree
2429
 
            for path, entry in tree.inventory.iter_entries_by_dir():
 
2682
            last_dirname = None
 
2683
            for path, entry in tree.iter_entries_by_dir():
2430
2684
                # here we process each trees details for each item in the tree.
2431
2685
                # we first update any existing entries for the id at other paths,
2432
2686
                # then we either create or update the entry for the id at the
2439
2693
                file_id = entry.file_id
2440
2694
                path_utf8 = path.encode('utf8')
2441
2695
                dirname, basename = osutils.split(path_utf8)
2442
 
                new_entry_key = (dirname, basename, file_id)
 
2696
                if dirname == last_dirname:
 
2697
                    # Try to re-use objects as much as possible
 
2698
                    dirname = last_dirname
 
2699
                else:
 
2700
                    last_dirname = dirname
 
2701
                new_entry_key = st(dirname, basename, file_id)
2443
2702
                # tree index consistency: All other paths for this id in this tree
2444
2703
                # index must point to the correct path.
2445
 
                for entry_key in id_index.setdefault(file_id, set()):
 
2704
                entry_keys = id_index.get(file_id, ())
 
2705
                for entry_key in entry_keys:
2446
2706
                    # TODO:PROFILING: It might be faster to just update
2447
2707
                    # rather than checking if we need to, and then overwrite
2448
2708
                    # the one we are located at.
2451
2711
                        # other trees, so put absent pointers there
2452
2712
                        # This is the vertical axis in the matrix, all pointing
2453
2713
                        # to the real path.
2454
 
                        by_path[entry_key][tree_index] = ('r', path_utf8, 0, False, '')
2455
 
                # by path consistency: Insert into an existing path record (trivial), or
2456
 
                # add a new one with relocation pointers for the other tree indexes.
2457
 
                if new_entry_key in id_index[file_id]:
2458
 
                    # there is already an entry where this data belongs, just insert it.
 
2714
                        by_path[entry_key][tree_index] = st(b'r', path_utf8, 0,
 
2715
                                                            False, b'')
 
2716
                # by path consistency: Insert into an existing path record
 
2717
                # (trivial), or add a new one with relocation pointers for the
 
2718
                # other tree indexes.
 
2719
                if new_entry_key in entry_keys:
 
2720
                    # there is already an entry where this data belongs, just
 
2721
                    # insert it.
2459
2722
                    by_path[new_entry_key][tree_index] = \
2460
2723
                        self._inv_entry_to_details(entry)
2461
2724
                else:
2463
2726
                    # mapping from path,id. We need to look up the correct path
2464
2727
                    # for the indexes from 0 to tree_index -1
2465
2728
                    new_details = []
2466
 
                    for lookup_index in xrange(tree_index):
 
2729
                    for lookup_index in range(tree_index):
2467
2730
                        # boundary case: this is the first occurence of file_id
2468
 
                        # so there are no id_indexs, possibly take this out of
 
2731
                        # so there are no id_indexes, possibly take this out of
2469
2732
                        # the loop?
2470
 
                        if not len(id_index[file_id]):
 
2733
                        if not len(entry_keys):
2471
2734
                            new_details.append(DirState.NULL_PARENT_DETAILS)
2472
2735
                        else:
2473
2736
                            # grab any one entry, use it to find the right path.
2474
 
                            # TODO: optimise this to reduce memory use in highly
2475
 
                            # fragmented situations by reusing the relocation
2476
 
                            # records.
2477
 
                            a_key = iter(id_index[file_id]).next()
2478
 
                            if by_path[a_key][lookup_index][0] in ('r', 'a'):
2479
 
                                # its a pointer or missing statement, use it as is.
 
2737
                            a_key = next(iter(entry_keys))
 
2738
                            if by_path[a_key][lookup_index][0] in (b'r', b'a'):
 
2739
                                # its a pointer or missing statement, use it as
 
2740
                                # is.
2480
2741
                                new_details.append(by_path[a_key][lookup_index])
2481
2742
                            else:
2482
2743
                                # we have the right key, make a pointer to it.
2483
 
                                real_path = ('/'.join(a_key[0:2])).strip('/')
2484
 
                                new_details.append(('r', real_path, 0, False, ''))
 
2744
                                real_path = (b'/'.join(a_key[0:2])).strip(b'/')
 
2745
                                new_details.append(st(b'r', real_path, 0, False,
 
2746
                                                      b''))
2485
2747
                    new_details.append(self._inv_entry_to_details(entry))
2486
2748
                    new_details.extend(new_location_suffix)
2487
2749
                    by_path[new_entry_key] = new_details
2488
 
                    id_index[file_id].add(new_entry_key)
 
2750
                    self._add_to_id_index(id_index, new_entry_key)
2489
2751
        # --- end generation of full tree mappings
2490
2752
 
2491
2753
        # sort and output all the entries
2492
 
        new_entries = self._sort_entries(by_path.items())
 
2754
        new_entries = self._sort_entries(viewitems(by_path))
2493
2755
        self._entries_to_current_state(new_entries)
2494
2756
        self._parents = [rev_id for rev_id, tree in trees]
2495
2757
        self._ghosts = list(ghosts)
2496
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
2497
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2758
        self._mark_modified(header_modified=True)
2498
2759
        self._id_index = id_index
2499
2760
 
2500
2761
    def _sort_entries(self, entry_list):
2504
2765
        try to keep everything in sorted blocks all the time, but sometimes
2505
2766
        it's easier to sort after the fact.
2506
2767
        """
2507
 
        def _key(entry):
 
2768
        # When sorting, we usually have 10x more entries than directories. (69k
 
2769
        # total entries, 4k directories). So cache the results of splitting.
 
2770
        # Saving time and objects. Also, use StaticTuple to avoid putting all
 
2771
        # of these object into python's garbage collector.
 
2772
        split_dirs = {}
 
2773
        def _key(entry, _split_dirs=split_dirs, _st=static_tuple.StaticTuple):
2508
2774
            # sort by: directory parts, file name, file id
2509
 
            return entry[0][0].split('/'), entry[0][1], entry[0][2]
 
2775
            dirpath, fname, file_id = entry[0]
 
2776
            try:
 
2777
                split = _split_dirs[dirpath]
 
2778
            except KeyError:
 
2779
                split = _st.from_sequence(dirpath.split(b'/'))
 
2780
                _split_dirs[dirpath] = split
 
2781
            return _st(split, fname, file_id)
2510
2782
        return sorted(entry_list, key=_key)
2511
2783
 
2512
2784
    def set_state_from_inventory(self, new_inv):
2542
2814
        # underlying dirstate.
2543
2815
        old_iterator = iter(list(self._iter_entries()))
2544
2816
        # both must have roots so this is safe:
2545
 
        current_new = new_iterator.next()
2546
 
        current_old = old_iterator.next()
 
2817
        current_new = next(new_iterator)
 
2818
        current_old = next(old_iterator)
2547
2819
        def advance(iterator):
2548
2820
            try:
2549
 
                return iterator.next()
 
2821
                return next(iterator)
2550
2822
            except StopIteration:
2551
2823
                return None
2552
2824
        while current_new or current_old:
2553
2825
            # skip entries in old that are not really there
2554
 
            if current_old and current_old[1][0][0] in 'ar':
 
2826
            if current_old and current_old[1][0][0] in (b'a', b'r'):
2555
2827
                # relocated or absent
2556
2828
                current_old = advance(old_iterator)
2557
2829
                continue
2563
2835
                new_entry_key = (new_dirname, new_basename, new_id)
2564
2836
                current_new_minikind = \
2565
2837
                    DirState._kind_to_minikind[current_new[1].kind]
2566
 
                if current_new_minikind == 't':
2567
 
                    fingerprint = current_new[1].reference_revision or ''
 
2838
                if current_new_minikind == b't':
 
2839
                    fingerprint = current_new[1].reference_revision or b''
2568
2840
                else:
2569
2841
                    # We normally only insert or remove records, or update
2570
2842
                    # them when it has significantly changed.  Then we want to
2571
2843
                    # erase its fingerprint.  Unaffected records should
2572
2844
                    # normally not be updated at all.
2573
 
                    fingerprint = ''
 
2845
                    fingerprint = b''
2574
2846
            else:
2575
2847
                # for safety disable variables
2576
2848
                new_path_utf8 = new_dirname = new_basename = new_id = \
2615
2887
                # both sides are dealt with, move on
2616
2888
                current_old = advance(old_iterator)
2617
2889
                current_new = advance(new_iterator)
2618
 
            elif (cmp_by_dirs(new_dirname, current_old[0][0]) < 0
 
2890
            elif (lt_by_dirs(new_dirname, current_old[0][0])
2619
2891
                  or (new_dirname == current_old[0][0]
2620
2892
                      and new_entry_key[1:] < current_old[0][1:])):
2621
2893
                # new comes before:
2637
2909
                        current_old[0][1].decode('utf8'))
2638
2910
                self._make_absent(current_old)
2639
2911
                current_old = advance(old_iterator)
2640
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2912
        self._mark_modified()
2641
2913
        self._id_index = None
2642
2914
        self._packed_stat_index = None
2643
2915
        if tracing:
2644
2916
            trace.mutter("set_state_from_inventory complete.")
2645
2917
 
 
2918
    def set_state_from_scratch(self, working_inv, parent_trees, parent_ghosts):
 
2919
        """Wipe the currently stored state and set it to something new.
 
2920
 
 
2921
        This is a hard-reset for the data we are working with.
 
2922
        """
 
2923
        # Technically, we really want a write lock, but until we write, we
 
2924
        # don't really need it.
 
2925
        self._requires_lock()
 
2926
        # root dir and root dir contents with no children. We have to have a
 
2927
        # root for set_state_from_inventory to work correctly.
 
2928
        empty_root = ((b'', b'', inventory.ROOT_ID),
 
2929
                      [(b'd', b'', 0, False, DirState.NULLSTAT)])
 
2930
        empty_tree_dirblocks = [(b'', [empty_root]), (b'', [])]
 
2931
        self._set_data([], empty_tree_dirblocks)
 
2932
        self.set_state_from_inventory(working_inv)
 
2933
        self.set_parent_trees(parent_trees, parent_ghosts)
 
2934
 
2646
2935
    def _make_absent(self, current_old):
2647
2936
        """Mark current_old - an entry - as absent for tree 0.
2648
2937
 
2655
2944
        all_remaining_keys = set()
2656
2945
        # Dont check the working tree, because it's going.
2657
2946
        for details in current_old[1][1:]:
2658
 
            if details[0] not in 'ar': # absent, relocated
 
2947
            if details[0] not in (b'a', b'r'): # absent, relocated
2659
2948
                all_remaining_keys.add(current_old[0])
2660
 
            elif details[0] == 'r': # relocated
 
2949
            elif details[0] == b'r': # relocated
2661
2950
                # record the key for the real path.
2662
2951
                all_remaining_keys.add(tuple(osutils.split(details[1])) + (current_old[0][2],))
2663
2952
            # absent rows are not present at any path.
2673
2962
            block[1].pop(entry_index)
2674
2963
            # if we have an id_index in use, remove this key from it for this id.
2675
2964
            if self._id_index is not None:
2676
 
                self._id_index[current_old[0][2]].remove(current_old[0])
 
2965
                self._remove_from_id_index(self._id_index, current_old[0])
2677
2966
        # update all remaining keys for this id to record it as absent. The
2678
2967
        # existing details may either be the record we are marking as deleted
2679
2968
        # (if there were other trees with the id present at this path), or may
2689
2978
                raise AssertionError('could not find entry for %s' % (update_key,))
2690
2979
            update_tree_details = self._dirblocks[update_block_index][1][update_entry_index][1]
2691
2980
            # it must not be absent at the moment
2692
 
            if update_tree_details[0][0] == 'a': # absent
 
2981
            if update_tree_details[0][0] == b'a': # absent
2693
2982
                raise AssertionError('bad row %r' % (update_tree_details,))
2694
2983
            update_tree_details[0] = DirState.NULL_PARENT_DETAILS
2695
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2984
        self._mark_modified()
2696
2985
        return last_reference
2697
2986
 
2698
 
    def update_minimal(self, key, minikind, executable=False, fingerprint='',
 
2987
    def update_minimal(self, key, minikind, executable=False, fingerprint=b'',
2699
2988
        packed_stat=None, size=0, path_utf8=None, fullscan=False):
2700
2989
        """Update an entry to the state in tree 0.
2701
2990
 
2704
2993
        updated as well.
2705
2994
 
2706
2995
        :param key: (dir, name, file_id) for the new entry
2707
 
        :param minikind: The type for the entry ('f' == 'file', 'd' ==
 
2996
        :param minikind: The type for the entry (b'f' == 'file', b'd' ==
2708
2997
                'directory'), etc.
2709
2998
        :param executable: Should the executable bit be set?
2710
2999
        :param fingerprint: Simple fingerprint for new entry: canonical-form
2723
3012
        block = self._find_block(key)[1]
2724
3013
        if packed_stat is None:
2725
3014
            packed_stat = DirState.NULLSTAT
2726
 
        # XXX: Some callers pass '' as the packed_stat, and it seems to be
 
3015
        # XXX: Some callers pass b'' as the packed_stat, and it seems to be
2727
3016
        # sometimes present in the dirstate - this seems oddly inconsistent.
2728
3017
        # mbp 20071008
2729
3018
        entry_index, present = self._find_entry_index(key, block)
2732
3021
        if not present:
2733
3022
            # New record. Check there isn't a entry at this path already.
2734
3023
            if not fullscan:
2735
 
                low_index, _ = self._find_entry_index(key[0:2] + ('',), block)
 
3024
                low_index, _ = self._find_entry_index(key[0:2] + (b'',), block)
2736
3025
                while low_index < len(block):
2737
3026
                    entry = block[low_index]
2738
3027
                    if entry[0][0:2] == key[0:2]:
2739
 
                        if entry[1][0][0] not in 'ar':
 
3028
                        if entry[1][0][0] not in (b'a', b'r'):
2740
3029
                            # This entry has the same path (but a different id) as
2741
3030
                            # the new entry we're adding, and is present in ths
2742
3031
                            # tree.
2743
 
                            raise errors.InconsistentDelta(
2744
 
                                ("%s/%s" % key[0:2]).decode('utf8'), key[2],
 
3032
                            self._raise_invalid(
 
3033
                                (b"%s/%s" % key[0:2]).decode('utf8'), key[2],
2745
3034
                                "Attempt to add item at path already occupied by "
2746
3035
                                "id %r" % entry[0][2])
2747
3036
                        low_index += 1
2748
3037
                    else:
2749
3038
                        break
2750
3039
            # new entry, synthesis cross reference here,
2751
 
            existing_keys = id_index.setdefault(key[2], set())
 
3040
            existing_keys = id_index.get(key[2], ())
2752
3041
            if not existing_keys:
2753
3042
                # not currently in the state, simplest case
2754
3043
                new_entry = key, [new_details] + self._empty_parent_info()
2784
3073
                    # entry, if not already examined, is skipped over by that
2785
3074
                    # loop.
2786
3075
                    other_entry = other_block[other_entry_index]
2787
 
                    other_entry[1][0] = ('r', path_utf8, 0, False, '')
2788
 
                    self._maybe_remove_row(other_block, other_entry_index,
2789
 
                        id_index)
 
3076
                    other_entry[1][0] = (b'r', path_utf8, 0, False, b'')
 
3077
                    if self._maybe_remove_row(other_block, other_entry_index,
 
3078
                                              id_index):
 
3079
                        # If the row holding this was removed, we need to
 
3080
                        # recompute where this entry goes
 
3081
                        entry_index, _ = self._find_entry_index(key, block)
2790
3082
 
2791
3083
                # This loop:
2792
3084
                # adds a tuple to the new details for each column
2794
3086
                #  - or by creating a new pointer to the right row inside that column
2795
3087
                num_present_parents = self._num_present_parents()
2796
3088
                if num_present_parents:
 
3089
                    # TODO: This re-evaluates the existing_keys set, do we need
 
3090
                    #       to do that ourselves?
2797
3091
                    other_key = list(existing_keys)[0]
2798
 
                for lookup_index in xrange(1, num_present_parents + 1):
 
3092
                for lookup_index in range(1, num_present_parents + 1):
2799
3093
                    # grab any one entry, use it to find the right path.
2800
3094
                    # TODO: optimise this to reduce memory use in highly
2801
3095
                    # fragmented situations by reusing the relocation
2809
3103
                    if not present:
2810
3104
                        raise AssertionError('update_minimal: could not find entry for %s' % (other_key,))
2811
3105
                    update_details = self._dirblocks[update_block_index][1][update_entry_index][1][lookup_index]
2812
 
                    if update_details[0] in 'ar': # relocated, absent
 
3106
                    if update_details[0] in (b'a', b'r'): # relocated, absent
2813
3107
                        # its a pointer or absent in lookup_index's tree, use
2814
3108
                        # it as is.
2815
3109
                        new_entry[1].append(update_details)
2816
3110
                    else:
2817
3111
                        # we have the right key, make a pointer to it.
2818
3112
                        pointer_path = osutils.pathjoin(*other_key[0:2])
2819
 
                        new_entry[1].append(('r', pointer_path, 0, False, ''))
 
3113
                        new_entry[1].append((b'r', pointer_path, 0, False, b''))
2820
3114
            block.insert(entry_index, new_entry)
2821
 
            existing_keys.add(key)
 
3115
            self._add_to_id_index(id_index, key)
2822
3116
        else:
2823
3117
            # Does the new state matter?
2824
3118
            block[entry_index][1][0] = new_details
2833
3127
            # converted to relocated.
2834
3128
            if path_utf8 is None:
2835
3129
                raise AssertionError('no path')
2836
 
            for entry_key in id_index.setdefault(key[2], set()):
 
3130
            existing_keys = id_index.get(key[2], ())
 
3131
            if key not in existing_keys:
 
3132
                raise AssertionError('We found the entry in the blocks, but'
 
3133
                    ' the key is not in the id_index.'
 
3134
                    ' key: %s, existing_keys: %s' % (key, existing_keys))
 
3135
            for entry_key in existing_keys:
2837
3136
                # TODO:PROFILING: It might be faster to just update
2838
3137
                # rather than checking if we need to, and then overwrite
2839
3138
                # the one we are located at.
2849
3148
                    if not present:
2850
3149
                        raise AssertionError('not present: %r', entry_key)
2851
3150
                    self._dirblocks[block_index][1][entry_index][1][0] = \
2852
 
                        ('r', path_utf8, 0, False, '')
 
3151
                        (b'r', path_utf8, 0, False, b'')
2853
3152
        # add a containing dirblock if needed.
2854
 
        if new_details[0] == 'd':
2855
 
            subdir_key = (osutils.pathjoin(*key[0:2]), '', '')
 
3153
        if new_details[0] == b'd':
 
3154
            # GZ 2017-06-09: Using pathjoin why?
 
3155
            subdir_key = (osutils.pathjoin(*key[0:2]), b'', b'')
2856
3156
            block_index, present = self._find_block_index_from_key(subdir_key)
2857
3157
            if not present:
2858
3158
                self._dirblocks.insert(block_index, (subdir_key[0], []))
2859
3159
 
2860
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
3160
        self._mark_modified()
2861
3161
 
2862
3162
    def _maybe_remove_row(self, block, index, id_index):
2863
3163
        """Remove index if it is absent or relocated across the row.
2864
3164
        
2865
3165
        id_index is updated accordingly.
 
3166
        :return: True if we removed the row, False otherwise
2866
3167
        """
2867
3168
        present_in_row = False
2868
3169
        entry = block[index]
2869
3170
        for column in entry[1]:
2870
 
            if column[0] not in 'ar':
 
3171
            if column[0] not in (b'a', b'r'):
2871
3172
                present_in_row = True
2872
3173
                break
2873
3174
        if not present_in_row:
2874
3175
            block.pop(index)
2875
 
            id_index[entry[0][2]].remove(entry[0])
 
3176
            self._remove_from_id_index(id_index, entry[0])
 
3177
            return True
 
3178
        return False
2876
3179
 
2877
3180
    def _validate(self):
2878
3181
        """Check that invariants on the dirblock are correct.
2885
3188
        # NOTE: This must always raise AssertionError not just assert,
2886
3189
        # otherwise it may not behave properly under python -O
2887
3190
        #
2888
 
        # TODO: All entries must have some content that's not 'a' or 'r',
 
3191
        # TODO: All entries must have some content that's not b'a' or b'r',
2889
3192
        # otherwise it could just be removed.
2890
3193
        #
2891
3194
        # TODO: All relocations must point directly to a real entry.
2896
3199
        from pprint import pformat
2897
3200
        self._read_dirblocks_if_needed()
2898
3201
        if len(self._dirblocks) > 0:
2899
 
            if not self._dirblocks[0][0] == '':
 
3202
            if not self._dirblocks[0][0] == b'':
2900
3203
                raise AssertionError(
2901
3204
                    "dirblocks don't start with root block:\n" + \
2902
3205
                    pformat(self._dirblocks))
2903
3206
        if len(self._dirblocks) > 1:
2904
 
            if not self._dirblocks[1][0] == '':
 
3207
            if not self._dirblocks[1][0] == b'':
2905
3208
                raise AssertionError(
2906
3209
                    "dirblocks missing root directory:\n" + \
2907
3210
                    pformat(self._dirblocks))
2908
3211
        # the dirblocks are sorted by their path components, name, and dir id
2909
 
        dir_names = [d[0].split('/')
 
3212
        dir_names = [d[0].split(b'/')
2910
3213
                for d in self._dirblocks[1:]]
2911
3214
        if dir_names != sorted(dir_names):
2912
3215
            raise AssertionError(
2936
3239
            current tree. (It is invalid to have a non-absent file in an absent
2937
3240
            directory.)
2938
3241
            """
2939
 
            if entry[0][0:2] == ('', ''):
 
3242
            if entry[0][0:2] == (b'', b''):
2940
3243
                # There should be no parent for the root row
2941
3244
                return
2942
3245
            parent_entry = self._get_entry(tree_index, path_utf8=entry[0][0])
2944
3247
                raise AssertionError(
2945
3248
                    "no parent entry for: %s in tree %s"
2946
3249
                    % (this_path, tree_index))
2947
 
            if parent_entry[1][tree_index][0] != 'd':
 
3250
            if parent_entry[1][tree_index][0] != b'd':
2948
3251
                raise AssertionError(
2949
3252
                    "Parent entry for %s is not marked as a valid"
2950
3253
                    " directory. %s" % (this_path, parent_entry,))
2958
3261
        # We check this with a dict per tree pointing either to the present
2959
3262
        # name, or None if absent.
2960
3263
        tree_count = self._num_present_parents() + 1
2961
 
        id_path_maps = [dict() for i in range(tree_count)]
 
3264
        id_path_maps = [{} for _ in range(tree_count)]
2962
3265
        # Make sure that all renamed entries point to the correct location.
2963
3266
        for entry in self._iter_entries():
2964
3267
            file_id = entry[0][2]
2972
3275
            for tree_index, tree_state in enumerate(entry[1]):
2973
3276
                this_tree_map = id_path_maps[tree_index]
2974
3277
                minikind = tree_state[0]
2975
 
                if minikind in 'ar':
 
3278
                if minikind in (b'a', b'r'):
2976
3279
                    absent_positions += 1
2977
3280
                # have we seen this id before in this column?
2978
3281
                if file_id in this_tree_map:
2979
3282
                    previous_path, previous_loc = this_tree_map[file_id]
2980
3283
                    # any later mention of this file must be consistent with
2981
3284
                    # what was said before
2982
 
                    if minikind == 'a':
 
3285
                    if minikind == b'a':
2983
3286
                        if previous_path is not None:
2984
3287
                            raise AssertionError(
2985
3288
                            "file %s is absent in row %r but also present " \
2986
3289
                            "at %r"% \
2987
 
                            (file_id, entry, previous_path))
2988
 
                    elif minikind == 'r':
 
3290
                            (file_id.decode('utf-8'), entry, previous_path))
 
3291
                    elif minikind == b'r':
2989
3292
                        target_location = tree_state[1]
2990
3293
                        if previous_path != target_location:
2991
3294
                            raise AssertionError(
3001
3304
                                (entry, previous_path, previous_loc))
3002
3305
                        check_valid_parent()
3003
3306
                else:
3004
 
                    if minikind == 'a':
 
3307
                    if minikind == b'a':
3005
3308
                        # absent; should not occur anywhere else
3006
3309
                        this_tree_map[file_id] = None, this_path
3007
 
                    elif minikind == 'r':
 
3310
                    elif minikind == b'r':
3008
3311
                        # relocation, must occur at expected location
3009
3312
                        this_tree_map[file_id] = tree_state[1], this_path
3010
3313
                    else:
3014
3317
                raise AssertionError(
3015
3318
                    "entry %r has no data for any tree." % (entry,))
3016
3319
        if self._id_index is not None:
3017
 
            for file_id, entry_keys in self._id_index.iteritems():
 
3320
            for file_id, entry_keys in viewitems(self._id_index):
3018
3321
                for entry_key in entry_keys:
 
3322
                    # Check that the entry in the map is pointing to the same
 
3323
                    # file_id
3019
3324
                    if entry_key[2] != file_id:
3020
3325
                        raise AssertionError(
3021
3326
                            'file_id %r did not match entry key %s'
3022
3327
                            % (file_id, entry_key))
 
3328
                    # And that from this entry key, we can look up the original
 
3329
                    # record
 
3330
                    block_index, present = self._find_block_index_from_key(entry_key)
 
3331
                    if not present:
 
3332
                        raise AssertionError('missing block for entry key: %r', entry_key)
 
3333
                    entry_index, present = self._find_entry_index(entry_key, self._dirblocks[block_index][1])
 
3334
                    if not present:
 
3335
                        raise AssertionError('missing entry for key: %r', entry_key)
 
3336
                if len(entry_keys) != len(set(entry_keys)):
 
3337
                    raise AssertionError(
 
3338
                        'id_index contained non-unique data for %s'
 
3339
                        % (entry_keys,))
3023
3340
 
3024
3341
    def _wipe_state(self):
3025
3342
        """Forget all state information about the dirstate."""
3047
3364
        self._lock_state = 'r'
3048
3365
        self._state_file = self._lock_token.f
3049
3366
        self._wipe_state()
 
3367
        return lock.LogicalLockResult(self.unlock)
3050
3368
 
3051
3369
    def lock_write(self):
3052
3370
        """Acquire a write lock on the dirstate."""
3060
3378
        self._lock_state = 'w'
3061
3379
        self._state_file = self._lock_token.f
3062
3380
        self._wipe_state()
 
3381
        return lock.LogicalLockResult(self.unlock, self._lock_token)
3063
3382
 
3064
3383
    def unlock(self):
3065
3384
        """Drop any locks held on the dirstate."""
3082
3401
 
3083
3402
 
3084
3403
def py_update_entry(state, entry, abspath, stat_value,
3085
 
                 _stat_to_minikind=DirState._stat_to_minikind,
3086
 
                 _pack_stat=pack_stat):
 
3404
                    _stat_to_minikind=DirState._stat_to_minikind):
3087
3405
    """Update the entry based on what is actually on disk.
3088
3406
 
3089
3407
    This function only calculates the sha if it needs to - if the entry is
3098
3416
        target of a symlink.
3099
3417
    """
3100
3418
    try:
3101
 
        minikind = _stat_to_minikind[stat_value.st_mode & 0170000]
 
3419
        minikind = _stat_to_minikind[stat_value.st_mode & 0o170000]
3102
3420
    except KeyError:
3103
3421
        # Unhandled kind
3104
3422
        return None
3105
 
    packed_stat = _pack_stat(stat_value)
 
3423
    packed_stat = pack_stat(stat_value)
3106
3424
    (saved_minikind, saved_link_or_sha1, saved_file_size,
3107
3425
     saved_executable, saved_packed_stat) = entry[1][0]
 
3426
    if not isinstance(saved_minikind, bytes):
 
3427
        raise TypeError(saved_minikind)
3108
3428
 
3109
 
    if minikind == 'd' and saved_minikind == 't':
3110
 
        minikind = 't'
 
3429
    if minikind == b'd' and saved_minikind == b't':
 
3430
        minikind = b't'
3111
3431
    if (minikind == saved_minikind
3112
3432
        and packed_stat == saved_packed_stat):
3113
3433
        # The stat hasn't changed since we saved, so we can re-use the
3114
3434
        # saved sha hash.
3115
 
        if minikind == 'd':
 
3435
        if minikind == b'd':
3116
3436
            return None
3117
3437
 
3118
3438
        # size should also be in packed_stat
3122
3442
    # If we have gotten this far, that means that we need to actually
3123
3443
    # process this entry.
3124
3444
    link_or_sha1 = None
3125
 
    if minikind == 'f':
 
3445
    worth_saving = True
 
3446
    if minikind == b'f':
3126
3447
        executable = state._is_executable(stat_value.st_mode,
3127
3448
                                         saved_executable)
3128
3449
        if state._cutoff_time is None:
3130
3451
        if (stat_value.st_mtime < state._cutoff_time
3131
3452
            and stat_value.st_ctime < state._cutoff_time
3132
3453
            and len(entry[1]) > 1
3133
 
            and entry[1][1][0] != 'a'):
 
3454
            and entry[1][1][0] != b'a'):
3134
3455
            # Could check for size changes for further optimised
3135
3456
            # avoidance of sha1's. However the most prominent case of
3136
3457
            # over-shaing is during initial add, which this catches.
3138
3459
            # are calculated at the same time, so checking just the size
3139
3460
            # gains nothing w.r.t. performance.
3140
3461
            link_or_sha1 = state._sha1_file(abspath)
3141
 
            entry[1][0] = ('f', link_or_sha1, stat_value.st_size,
 
3462
            entry[1][0] = (b'f', link_or_sha1, stat_value.st_size,
3142
3463
                           executable, packed_stat)
3143
3464
        else:
3144
 
            entry[1][0] = ('f', '', stat_value.st_size,
 
3465
            entry[1][0] = (b'f', b'', stat_value.st_size,
3145
3466
                           executable, DirState.NULLSTAT)
3146
 
    elif minikind == 'd':
 
3467
            worth_saving = False
 
3468
    elif minikind == b'd':
3147
3469
        link_or_sha1 = None
3148
 
        entry[1][0] = ('d', '', 0, False, packed_stat)
3149
 
        if saved_minikind != 'd':
 
3470
        entry[1][0] = (b'd', b'', 0, False, packed_stat)
 
3471
        if saved_minikind != b'd':
3150
3472
            # This changed from something into a directory. Make sure we
3151
3473
            # have a directory block for it. This doesn't happen very
3152
3474
            # often, so this doesn't have to be super fast.
3154
3476
                state._get_block_entry_index(entry[0][0], entry[0][1], 0)
3155
3477
            state._ensure_block(block_index, entry_index,
3156
3478
                               osutils.pathjoin(entry[0][0], entry[0][1]))
3157
 
    elif minikind == 'l':
 
3479
        else:
 
3480
            worth_saving = False
 
3481
    elif minikind == b'l':
 
3482
        if saved_minikind == b'l':
 
3483
            worth_saving = False
3158
3484
        link_or_sha1 = state._read_link(abspath, saved_link_or_sha1)
3159
3485
        if state._cutoff_time is None:
3160
3486
            state._sha_cutoff_time()
3161
3487
        if (stat_value.st_mtime < state._cutoff_time
3162
3488
            and stat_value.st_ctime < state._cutoff_time):
3163
 
            entry[1][0] = ('l', link_or_sha1, stat_value.st_size,
 
3489
            entry[1][0] = (b'l', link_or_sha1, stat_value.st_size,
3164
3490
                           False, packed_stat)
3165
3491
        else:
3166
 
            entry[1][0] = ('l', '', stat_value.st_size,
 
3492
            entry[1][0] = (b'l', b'', stat_value.st_size,
3167
3493
                           False, DirState.NULLSTAT)
3168
 
    state._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
3494
    if worth_saving:
 
3495
        state._mark_modified([entry])
3169
3496
    return link_or_sha1
3170
3497
 
3171
3498
 
3184
3511
        self.old_dirname_to_file_id = {}
3185
3512
        self.new_dirname_to_file_id = {}
3186
3513
        # Are we doing a partial iter_changes?
3187
 
        self.partial = search_specific_files != set([''])
 
3514
        self.partial = search_specific_files != {''}
3188
3515
        # Using a list so that we can access the values and change them in
3189
3516
        # nested scope. Each one is [path, file_id, entry]
3190
3517
        self.last_source_parent = [None, None]
3236
3563
            source_details = DirState.NULL_PARENT_DETAILS
3237
3564
        else:
3238
3565
            source_details = entry[1][self.source_index]
 
3566
        # GZ 2017-06-09: Eck, more sets.
 
3567
        _fdltr = {b'f', b'd', b'l', b't', b'r'}
 
3568
        _fdlt = {b'f', b'd', b'l', b't'}
 
3569
        _ra = (b'r', b'a')
3239
3570
        target_details = entry[1][self.target_index]
3240
3571
        target_minikind = target_details[0]
3241
 
        if path_info is not None and target_minikind in 'fdlt':
 
3572
        if path_info is not None and target_minikind in _fdlt:
3242
3573
            if not (self.target_index == 0):
3243
3574
                raise AssertionError()
3244
3575
            link_or_sha1 = update_entry(self.state, entry,
3250
3581
            link_or_sha1 = None
3251
3582
        file_id = entry[0][2]
3252
3583
        source_minikind = source_details[0]
3253
 
        if source_minikind in 'fdltr' and target_minikind in 'fdlt':
 
3584
        if source_minikind in _fdltr and target_minikind in _fdlt:
3254
3585
            # claimed content in both: diff
3255
3586
            #   r    | fdlt   |      | add source to search, add id path move and perform
3256
3587
            #        |        |      | diff check on source-target
3257
3588
            #   r    | fdlt   |  a   | dangling file that was present in the basis.
3258
3589
            #        |        |      | ???
3259
 
            if source_minikind in 'r':
 
3590
            if source_minikind == b'r':
3260
3591
                # add the source to the search path to find any children it
3261
3592
                # has.  TODO ? : only add if it is a container ?
3262
3593
                if not osutils.is_inside_any(self.searched_specific_files,
3272
3603
                # update the source details variable to be the real
3273
3604
                # location.
3274
3605
                if old_entry == (None, None):
3275
 
                    raise errors.CorruptDirstate(self.state._filename,
 
3606
                    raise DirstateCorrupt(self.state._filename,
3276
3607
                        "entry '%s/%s' is considered renamed from %r"
3277
3608
                        " but source does not exist\n"
3278
3609
                        "entry: %s" % (entry[0][0], entry[0][1], old_path, entry))
3294
3625
                    if path is None:
3295
3626
                        old_path = path = pathjoin(old_dirname, old_basename)
3296
3627
                    self.new_dirname_to_file_id[path] = file_id
3297
 
                    if source_minikind != 'd':
 
3628
                    if source_minikind != b'd':
3298
3629
                        content_change = True
3299
3630
                    else:
3300
3631
                        # directories have no fingerprint
3301
3632
                        content_change = False
3302
3633
                    target_exec = False
3303
3634
                elif target_kind == 'file':
3304
 
                    if source_minikind != 'f':
 
3635
                    if source_minikind != b'f':
3305
3636
                        content_change = True
3306
3637
                    else:
3307
3638
                        # Check the sha. We can't just rely on the size as
3323
3654
                    else:
3324
3655
                        target_exec = target_details[3]
3325
3656
                elif target_kind == 'symlink':
3326
 
                    if source_minikind != 'l':
 
3657
                    if source_minikind != b'l':
3327
3658
                        content_change = True
3328
3659
                    else:
3329
3660
                        content_change = (link_or_sha1 != source_details[1])
3330
3661
                    target_exec = False
3331
3662
                elif target_kind == 'tree-reference':
3332
 
                    if source_minikind != 't':
 
3663
                    if source_minikind != b't':
3333
3664
                        content_change = True
3334
3665
                    else:
3335
3666
                        content_change = False
3338
3669
                    if path is None:
3339
3670
                        path = pathjoin(old_dirname, old_basename)
3340
3671
                    raise errors.BadFileKindError(path, path_info[2])
3341
 
            if source_minikind == 'd':
 
3672
            if source_minikind == b'd':
3342
3673
                if path is None:
3343
3674
                    old_path = path = pathjoin(old_dirname, old_basename)
3344
3675
                self.old_dirname_to_file_id[old_path] = file_id
3409
3740
                       (self.utf8_decode(old_basename)[0], self.utf8_decode(entry[0][1])[0]),
3410
3741
                       (source_kind, target_kind),
3411
3742
                       (source_exec, target_exec)), changed
3412
 
        elif source_minikind in 'a' and target_minikind in 'fdlt':
 
3743
        elif source_minikind in b'a' and target_minikind in _fdlt:
3413
3744
            # looks like a new file
3414
3745
            path = pathjoin(entry[0][0], entry[0][1])
3415
3746
            # parent id is the entry for the path in the target tree
3446
3777
                       (None, self.utf8_decode(entry[0][1])[0]),
3447
3778
                       (None, None),
3448
3779
                       (None, False)), True
3449
 
        elif source_minikind in 'fdlt' and target_minikind in 'a':
 
3780
        elif source_minikind in _fdlt and target_minikind in b'a':
3450
3781
            # unversioned, possibly, or possibly not deleted: we dont care.
3451
3782
            # if its still on disk, *and* theres no other entry at this
3452
3783
            # path [we dont know this in this routine at the moment -
3464
3795
                   (self.utf8_decode(entry[0][1])[0], None),
3465
3796
                   (DirState._minikind_to_kind[source_minikind], None),
3466
3797
                   (source_details[3], None)), True
3467
 
        elif source_minikind in 'fdlt' and target_minikind in 'r':
 
3798
        elif source_minikind in _fdlt and target_minikind in b'r':
3468
3799
            # a rename; could be a true rename, or a rename inherited from
3469
3800
            # a renamed parent. TODO: handle this efficiently. Its not
3470
3801
            # common case to rename dirs though, so a correct but slow
3471
3802
            # implementation will do.
3472
 
            if not osutils.is_inside_any(self.searched_specific_files, target_details[1]):
 
3803
            if not osutils.is_inside_any(self.searched_specific_files,
 
3804
                                         target_details[1]):
3473
3805
                self.search_specific_files.add(target_details[1])
3474
 
        elif source_minikind in 'ra' and target_minikind in 'ra':
 
3806
        elif source_minikind in _ra and target_minikind in _ra:
3475
3807
            # neither of the selected trees contain this file,
3476
3808
            # so skip over it. This is not currently directly tested, but
3477
3809
            # is indirectly via test_too_much.TestCommands.test_conflicts.
3480
3812
            raise AssertionError("don't know how to compare "
3481
3813
                "source_minikind=%r, target_minikind=%r"
3482
3814
                % (source_minikind, target_minikind))
3483
 
            ## import pdb;pdb.set_trace()
3484
3815
        return None, None
3485
3816
 
3486
3817
    def __iter__(self):
3500
3831
        if new_path:
3501
3832
            # Not the root and not a delete: queue up the parents of the path.
3502
3833
            self.search_specific_file_parents.update(
3503
 
                osutils.parent_directories(new_path.encode('utf8')))
 
3834
                p.encode('utf8') for p in osutils.parent_directories(new_path))
3504
3835
            # Add the root directory which parent_directories does not
3505
3836
            # provide.
3506
 
            self.search_specific_file_parents.add('')
 
3837
            self.search_specific_file_parents.add(b'')
3507
3838
 
3508
3839
    def iter_changes(self):
3509
3840
        """Iterate over the changes."""
3510
3841
        utf8_decode = cache_utf8._utf8_decode
3511
 
        _cmp_by_dirs = cmp_by_dirs
 
3842
        _lt_by_dirs = lt_by_dirs
3512
3843
        _process_entry = self._process_entry
3513
3844
        search_specific_files = self.search_specific_files
3514
3845
        searched_specific_files = self.searched_specific_files
3563
3894
            root_abspath = self.tree.abspath(current_root_unicode)
3564
3895
            try:
3565
3896
                root_stat = os.lstat(root_abspath)
3566
 
            except OSError, e:
 
3897
            except OSError as e:
3567
3898
                if e.errno == errno.ENOENT:
3568
3899
                    # the path does not exist: let _process_entry know that.
3569
3900
                    root_dir_info = None
3571
3902
                    # some other random error: hand it up.
3572
3903
                    raise
3573
3904
            else:
3574
 
                root_dir_info = ('', current_root,
 
3905
                root_dir_info = (b'', current_root,
3575
3906
                    osutils.file_kind_from_stat_mode(root_stat.st_mode), root_stat,
3576
3907
                    root_abspath)
3577
3908
                if root_dir_info[2] == 'directory':
3605
3936
                       (None, root_dir_info[2]),
3606
3937
                       (None, new_executable)
3607
3938
                      )
3608
 
            initial_key = (current_root, '', '')
 
3939
            initial_key = (current_root, b'', b'')
3609
3940
            block_index, _ = self.state._find_block_index_from_key(initial_key)
3610
3941
            if block_index == 0:
3611
3942
                # we have processed the total root already, but because the
3616
3947
            else:
3617
3948
                dir_iterator = osutils._walkdirs_utf8(root_abspath, prefix=current_root)
3618
3949
                try:
3619
 
                    current_dir_info = dir_iterator.next()
3620
 
                except OSError, e:
 
3950
                    current_dir_info = next(dir_iterator)
 
3951
                except OSError as e:
3621
3952
                    # on win32, python2.4 has e.errno == ERROR_DIRECTORY, but
3622
3953
                    # python 2.5 has e.errno == EINVAL,
3623
3954
                    #            and e.winerror == ERROR_DIRECTORY
3635
3966
                    else:
3636
3967
                        raise
3637
3968
                else:
3638
 
                    if current_dir_info[0][0] == '':
 
3969
                    if current_dir_info[0][0] == b'':
3639
3970
                        # remove .bzr from iteration
3640
 
                        bzr_index = bisect.bisect_left(current_dir_info[1], ('.bzr',))
3641
 
                        if current_dir_info[1][bzr_index][0] != '.bzr':
 
3971
                        bzr_index = bisect.bisect_left(current_dir_info[1], (b'.bzr',))
 
3972
                        if current_dir_info[1][bzr_index][0] != b'.bzr':
3642
3973
                            raise AssertionError()
3643
3974
                        del current_dir_info[1][bzr_index]
3644
3975
            # walk until both the directory listing and the versioned metadata
3645
3976
            # are exhausted.
3646
3977
            if (block_index < len(self.state._dirblocks) and
3647
 
                osutils.is_inside(current_root, self.state._dirblocks[block_index][0])):
 
3978
                osutils.is_inside(current_root,
 
3979
                                  self.state._dirblocks[block_index][0])):
3648
3980
                current_block = self.state._dirblocks[block_index]
3649
3981
            else:
3650
3982
                current_block = None
3652
3984
                   current_block is not None):
3653
3985
                if (current_dir_info and current_block
3654
3986
                    and current_dir_info[0][0] != current_block[0]):
3655
 
                    if _cmp_by_dirs(current_dir_info[0][0], current_block[0]) < 0:
 
3987
                    if _lt_by_dirs(current_dir_info[0][0], current_block[0]):
3656
3988
                        # filesystem data refers to paths not covered by the dirblock.
3657
3989
                        # this has two possibilities:
3658
3990
                        # A) it is versioned but empty, so there is no block for it
3692
4024
 
3693
4025
                        # This dir info has been handled, go to the next
3694
4026
                        try:
3695
 
                            current_dir_info = dir_iterator.next()
 
4027
                            current_dir_info = next(dir_iterator)
3696
4028
                        except StopIteration:
3697
4029
                            current_dir_info = None
3698
4030
                    else:
3753
4085
                            if changed or self.include_unchanged:
3754
4086
                                yield result
3755
4087
                    elif (current_entry[0][1] != current_path_info[1]
3756
 
                          or current_entry[1][self.target_index][0] in 'ar'):
 
4088
                          or current_entry[1][self.target_index][0] in (b'a', b'r')):
3757
4089
                        # The current path on disk doesn't match the dirblock
3758
4090
                        # record. Either the dirblock is marked as absent, or
3759
4091
                        # the file on disk is not present at all in the
3838
4170
                if current_block is not None:
3839
4171
                    block_index += 1
3840
4172
                    if (block_index < len(self.state._dirblocks) and
3841
 
                        osutils.is_inside(current_root, self.state._dirblocks[block_index][0])):
 
4173
                        osutils.is_inside(current_root,
 
4174
                                          self.state._dirblocks[block_index][0])):
3842
4175
                        current_block = self.state._dirblocks[block_index]
3843
4176
                    else:
3844
4177
                        current_block = None
3845
4178
                if current_dir_info is not None:
3846
4179
                    try:
3847
 
                        current_dir_info = dir_iterator.next()
 
4180
                        current_dir_info = next(dir_iterator)
3848
4181
                    except StopIteration:
3849
4182
                        current_dir_info = None
3850
4183
        for result in self._iter_specific_file_parents():
3872
4205
            found_item = False
3873
4206
            for candidate_entry in path_entries:
3874
4207
                # Find entries present in target at this path:
3875
 
                if candidate_entry[1][self.target_index][0] not in 'ar':
 
4208
                if candidate_entry[1][self.target_index][0] not in (b'a', b'r'):
3876
4209
                    found_item = True
3877
4210
                    selected_entries.append(candidate_entry)
3878
4211
                # Find entries present in source at this path:
3879
4212
                elif (self.source_index is not None and
3880
 
                    candidate_entry[1][self.source_index][0] not in 'ar'):
 
4213
                    candidate_entry[1][self.source_index][0] not in (b'a', b'r')):
3881
4214
                    found_item = True
3882
 
                    if candidate_entry[1][self.target_index][0] == 'a':
 
4215
                    if candidate_entry[1][self.target_index][0] == b'a':
3883
4216
                        # Deleted, emit it here.
3884
4217
                        selected_entries.append(candidate_entry)
3885
4218
                    else:
3909
4242
                        result[6][1] != 'directory'):
3910
4243
                        # This stopped being a directory, the old children have
3911
4244
                        # to be included.
3912
 
                        if entry[1][self.source_index][0] == 'r':
 
4245
                        if entry[1][self.source_index][0] == b'r':
3913
4246
                            # renamed, take the source path
3914
4247
                            entry_path_utf8 = entry[1][self.source_index][1]
3915
4248
                        else:
3916
4249
                            entry_path_utf8 = path_utf8
3917
 
                        initial_key = (entry_path_utf8, '', '')
 
4250
                        initial_key = (entry_path_utf8, b'', b'')
3918
4251
                        block_index, _ = self.state._find_block_index_from_key(
3919
4252
                            initial_key)
3920
4253
                        if block_index == 0:
3929
4262
                                current_block = None
3930
4263
                        if current_block is not None:
3931
4264
                            for entry in current_block[1]:
3932
 
                                if entry[1][self.source_index][0] in 'ar':
 
4265
                                if entry[1][self.source_index][0] in (b'a', b'r'):
3933
4266
                                    # Not in the source tree, so doesn't have to be
3934
4267
                                    # included.
3935
4268
                                    continue
3949
4282
        abspath = self.tree.abspath(unicode_path)
3950
4283
        try:
3951
4284
            stat = os.lstat(abspath)
3952
 
        except OSError, e:
 
4285
        except OSError as e:
3953
4286
            if e.errno == errno.ENOENT:
3954
4287
                # the path does not exist.
3955
4288
                return None
3956
4289
            else:
3957
4290
                raise
3958
 
        utf8_basename = utf8_path.rsplit('/', 1)[-1]
 
4291
        utf8_basename = utf8_path.rsplit(b'/', 1)[-1]
3959
4292
        dir_info = (utf8_path, utf8_basename,
3960
4293
            osutils.file_kind_from_stat_mode(stat.st_mode), stat,
3961
4294
            abspath)
3969
4302
 
3970
4303
# Try to load the compiled form if possible
3971
4304
try:
3972
 
    from bzrlib._dirstate_helpers_pyx import (
 
4305
    from ._dirstate_helpers_pyx import (
3973
4306
        _read_dirblocks,
3974
4307
        bisect_dirblock,
3975
4308
        _bisect_path_left,
3976
4309
        _bisect_path_right,
3977
 
        cmp_by_dirs,
 
4310
        lt_by_dirs,
 
4311
        pack_stat,
3978
4312
        ProcessEntryC as _process_entry,
3979
4313
        update_entry as update_entry,
3980
4314
        )
3981
 
except ImportError, e:
 
4315
except ImportError as e:
3982
4316
    osutils.failed_to_load_extension(e)
3983
 
    from bzrlib._dirstate_helpers_py import (
 
4317
    from ._dirstate_helpers_py import (
3984
4318
        _read_dirblocks,
3985
4319
        bisect_dirblock,
3986
4320
        _bisect_path_left,
3987
4321
        _bisect_path_right,
3988
 
        cmp_by_dirs,
 
4322
        lt_by_dirs,
 
4323
        pack_stat,
3989
4324
        )
3990
4325
    # FIXME: It would be nice to be able to track moved lines so that the
3991
4326
    # corresponding python code can be moved to the _dirstate_helpers_py