/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: Jelmer Vernooij
  • Date: 2018-07-17 21:20:55 UTC
  • mto: (7045.3.3 python3-r)
  • mto: This revision was merged to the branch mainline in revision 7050.
  • Revision ID: jelmer@jelmer.uk-20180717212055-apkv1s70oo6wpi6o
Fix a few more tests.

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
 
 
310
def is_inside(dir, fname):
 
311
    """True if fname is inside dir.
 
312
 
 
313
    The parameters should typically be passed to osutils.normpath first, so
 
314
    that . and .. and repeated slashes are eliminated, and the separators
 
315
    are canonical for the platform.
 
316
 
 
317
    The empty string as a dir name is taken as top-of-tree and matches
 
318
    everything.
 
319
 
 
320
    This is based on breezy.osutils.is_inside, which uses filesystem strings.
 
321
    """
 
322
    if dir == fname:
 
323
        return True
 
324
 
 
325
    if dir == b'':
 
326
        return True
 
327
 
 
328
    if not dir.endswith(b'/'):
 
329
        dir += b'/'
 
330
 
 
331
    return fname.startswith(dir)
 
332
 
 
333
 
 
334
def is_inside_any(dir_list, fname):
 
335
    """True if fname is inside any of given dirs."""
 
336
    for dirname in dir_list:
 
337
        if is_inside(dirname, fname):
 
338
            return True
 
339
    return False
 
340
 
 
341
 
308
342
class DirState(object):
309
343
    """Record directory and metadata state for fast access.
310
344
 
322
356
    """
323
357
 
324
358
    _kind_to_minikind = {
325
 
            'absent': 'a',
326
 
            'file': 'f',
327
 
            'directory': 'd',
328
 
            'relocated': 'r',
329
 
            'symlink': 'l',
330
 
            'tree-reference': 't',
 
359
            'absent': b'a',
 
360
            'file': b'f',
 
361
            'directory': b'd',
 
362
            'relocated': b'r',
 
363
            'symlink': b'l',
 
364
            'tree-reference': b't',
331
365
        }
332
366
    _minikind_to_kind = {
333
 
            'a': 'absent',
334
 
            'f': 'file',
335
 
            'd': 'directory',
336
 
            'l':'symlink',
337
 
            'r': 'relocated',
338
 
            't': 'tree-reference',
 
367
            b'a': 'absent',
 
368
            b'f': 'file',
 
369
            b'd': 'directory',
 
370
            b'l': 'symlink',
 
371
            b'r': 'relocated',
 
372
            b't': 'tree-reference',
339
373
        }
340
374
    _stat_to_minikind = {
341
 
        stat.S_IFDIR:'d',
342
 
        stat.S_IFREG:'f',
343
 
        stat.S_IFLNK:'l',
 
375
        stat.S_IFDIR: b'd',
 
376
        stat.S_IFREG: b'f',
 
377
        stat.S_IFLNK: b'l',
344
378
    }
345
 
    _to_yesno = {True:'y', False: 'n'} # TODO profile the performance gain
 
379
    _to_yesno = {True: b'y', False: b'n'} # TODO profile the performance gain
346
380
     # of using int conversion rather than a dict here. AND BLAME ANDREW IF
347
381
     # it is faster.
348
382
 
354
388
    NOT_IN_MEMORY = 0
355
389
    IN_MEMORY_UNMODIFIED = 1
356
390
    IN_MEMORY_MODIFIED = 2
 
391
    IN_MEMORY_HASH_MODIFIED = 3 # Only hash-cache updates
357
392
 
358
393
    # A pack_stat (the x's) that is just noise and will never match the output
359
394
    # 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):
 
395
    NULLSTAT = b'x' * 32
 
396
    NULL_PARENT_DETAILS = static_tuple.StaticTuple(b'a', b'', 0, False, b'')
 
397
 
 
398
    HEADER_FORMAT_2 = b'#bazaar dirstate flat format 2\n'
 
399
    HEADER_FORMAT_3 = b'#bazaar dirstate flat format 3\n'
 
400
 
 
401
    def __init__(self, path, sha1_provider, worth_saving_limit=0):
367
402
        """Create a  DirState object.
368
403
 
369
404
        :param path: The path at which the dirstate file on disk should live.
370
405
        :param sha1_provider: an object meeting the SHA1Provider interface.
 
406
        :param worth_saving_limit: when the exact number of hash changed
 
407
            entries is known, only bother saving the dirstate if more than
 
408
            this count of entries have changed.
 
409
            -1 means never save hash changes, 0 means always save hash changes.
371
410
        """
372
411
        # _header_state and _dirblock_state represent the current state
373
412
        # of the dirstate metadata and the per-row data respectiely.
410
449
        # during commit.
411
450
        self._last_block_index = None
412
451
        self._last_entry_index = None
 
452
        # The set of known hash changes
 
453
        self._known_hash_changes = set()
 
454
        # How many hash changed entries can we have without saving
 
455
        self._worth_saving_limit = worth_saving_limit
 
456
        self._config_stack = config.LocationStack(urlutils.local_path_to_url(
 
457
            path))
413
458
 
414
459
    def __repr__(self):
415
460
        return "%s(%r)" % \
416
461
            (self.__class__.__name__, self._filename)
417
462
 
 
463
    def _mark_modified(self, hash_changed_entries=None, header_modified=False):
 
464
        """Mark this dirstate as modified.
 
465
 
 
466
        :param hash_changed_entries: if non-None, mark just these entries as
 
467
            having their hash modified.
 
468
        :param header_modified: mark the header modified as well, not just the
 
469
            dirblocks.
 
470
        """
 
471
        #trace.mutter_callsite(3, "modified hash entries: %s", hash_changed_entries)
 
472
        if hash_changed_entries:
 
473
            self._known_hash_changes.update([e[0] for e in hash_changed_entries])
 
474
            if self._dirblock_state in (DirState.NOT_IN_MEMORY,
 
475
                                        DirState.IN_MEMORY_UNMODIFIED):
 
476
                # If the dirstate is already marked a IN_MEMORY_MODIFIED, then
 
477
                # that takes precedence.
 
478
                self._dirblock_state = DirState.IN_MEMORY_HASH_MODIFIED
 
479
        else:
 
480
            # TODO: Since we now have a IN_MEMORY_HASH_MODIFIED state, we
 
481
            #       should fail noisily if someone tries to set
 
482
            #       IN_MEMORY_MODIFIED but we don't have a write-lock!
 
483
            # We don't know exactly what changed so disable smart saving
 
484
            self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
485
        if header_modified:
 
486
            self._header_state = DirState.IN_MEMORY_MODIFIED
 
487
 
 
488
    def _mark_unmodified(self):
 
489
        """Mark this dirstate as unmodified."""
 
490
        self._header_state = DirState.IN_MEMORY_UNMODIFIED
 
491
        self._dirblock_state = DirState.IN_MEMORY_UNMODIFIED
 
492
        self._known_hash_changes = set()
 
493
 
418
494
    def add(self, path, file_id, kind, stat, fingerprint):
419
495
        """Add a path to be tracked.
420
496
 
421
 
        :param path: The path within the dirstate - '' is the root, 'foo' is the
 
497
        :param path: The path within the dirstate - b'' is the root, 'foo' is the
422
498
            path foo within the root, 'foo/bar' is the path bar within foo
423
499
            within the root.
424
500
        :param file_id: The file id of the path being added.
429
505
            after any read filters have been applied),
430
506
            or the target of a symlink,
431
507
            or the referenced revision id for tree-references,
432
 
            or '' for directories.
 
508
            or b'' for directories.
433
509
        """
434
510
        # adding a file:
435
511
        # find the block its in.
457
533
        utf8path = (dirname + '/' + basename).strip('/').encode('utf8')
458
534
        dirname, basename = osutils.split(utf8path)
459
535
        # uses __class__ for speed; the check is needed for safety
460
 
        if file_id.__class__ is not str:
 
536
        if file_id.__class__ is not bytes:
461
537
            raise AssertionError(
462
538
                "must be a utf8 file_id not %s" % (type(file_id), ))
463
539
        # Make sure the file_id does not exist in this tree
464
540
        rename_from = None
465
541
        file_id_entry = self._get_entry(0, fileid_utf8=file_id, include_deleted=True)
466
542
        if file_id_entry != (None, None):
467
 
            if file_id_entry[1][0][0] == 'a':
 
543
            if file_id_entry[1][0][0] == b'a':
468
544
                if file_id_entry[0] != (dirname, basename, file_id):
469
545
                    # set the old name's current operation to rename
470
546
                    self.update_minimal(file_id_entry[0],
471
 
                        'r',
472
 
                        path_utf8='',
473
 
                        packed_stat='',
 
547
                        b'r',
 
548
                        path_utf8=b'',
 
549
                        packed_stat=b'',
474
550
                        fingerprint=utf8path
475
551
                    )
476
552
                    rename_from = file_id_entry[0][0:2]
479
555
                kind = DirState._minikind_to_kind[file_id_entry[1][0][0]]
480
556
                info = '%s:%s' % (kind, path)
481
557
                raise errors.DuplicateFileId(file_id, info)
482
 
        first_key = (dirname, basename, '')
 
558
        first_key = (dirname, basename, b'')
483
559
        block_index, present = self._find_block_index_from_key(first_key)
484
560
        if present:
485
561
            # check the path is not in the tree
487
563
            entry_index, _ = self._find_entry_index(first_key, block)
488
564
            while (entry_index < len(block) and
489
565
                block[entry_index][0][0:2] == first_key[0:2]):
490
 
                if block[entry_index][1][0][0] not in 'ar':
 
566
                if block[entry_index][1][0][0] not in (b'a', b'r'):
491
567
                    # this path is in the dirstate in the current tree.
492
 
                    raise Exception, "adding already added path!"
 
568
                    raise Exception("adding already added path!")
493
569
                entry_index += 1
494
570
        else:
495
571
            # The block where we want to put the file is not present. But it
513
589
        minikind = DirState._kind_to_minikind[kind]
514
590
        if rename_from is not None:
515
591
            if rename_from[0]:
516
 
                old_path_utf8 = '%s/%s' % rename_from
 
592
                old_path_utf8 = b'%s/%s' % rename_from
517
593
            else:
518
594
                old_path_utf8 = rename_from[1]
519
 
            parent_info[0] = ('r', old_path_utf8, 0, False, '')
 
595
            parent_info[0] = (b'r', old_path_utf8, 0, False, b'')
520
596
        if kind == 'file':
521
597
            entry_data = entry_key, [
522
598
                (minikind, fingerprint, size, False, packed_stat),
523
599
                ] + parent_info
524
600
        elif kind == 'directory':
525
601
            entry_data = entry_key, [
526
 
                (minikind, '', 0, False, packed_stat),
 
602
                (minikind, b'', 0, False, packed_stat),
527
603
                ] + parent_info
528
604
        elif kind == 'symlink':
529
605
            entry_data = entry_key, [
539
615
        if not present:
540
616
            block.insert(entry_index, entry_data)
541
617
        else:
542
 
            if block[entry_index][1][0][0] != 'a':
 
618
            if block[entry_index][1][0][0] != b'a':
543
619
                raise AssertionError(" %r(%r) already added" % (basename, file_id))
544
620
            block[entry_index][1][0] = entry_data[1][0]
545
621
 
546
622
        if kind == 'directory':
547
623
           # insert a new dirblock
548
624
           self._ensure_block(block_index, entry_index, utf8path)
549
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
625
        self._mark_modified()
550
626
        if self._id_index:
551
 
            self._id_index.setdefault(entry_key[2], set()).add(entry_key)
 
627
            self._add_to_id_index(self._id_index, entry_key)
552
628
 
553
629
    def _bisect(self, paths):
554
630
        """Bisect through the disk structure for specific rows.
575
651
        file_size = os.fstat(state_file.fileno()).st_size
576
652
        # We end up with 2 extra fields, we should have a trailing '\n' to
577
653
        # ensure that we read the whole record, and we should have a precursur
578
 
        # '' which ensures that we start after the previous '\n'
 
654
        # b'' which ensures that we start after the previous '\n'
579
655
        entry_field_count = self._fields_per_entry() + 1
580
656
 
581
657
        low = self._end_of_header
609
685
            if count > max_count:
610
686
                raise errors.BzrError('Too many seeks, most likely a bug.')
611
687
 
612
 
            mid = max(low, (low+high-page_size)/2)
 
688
            mid = max(low, (low+high-page_size)//2)
613
689
 
614
690
            state_file.seek(mid)
615
691
            # limit the read size, so we don't end up reading data that we have
618
694
            block = state_file.read(read_size)
619
695
 
620
696
            start = mid
621
 
            entries = block.split('\n')
 
697
            entries = block.split(b'\n')
622
698
 
623
699
            if len(entries) < 2:
624
700
                # We didn't find a '\n', so we cannot have found any records.
632
708
            # Check the first and last entries, in case they are partial, or if
633
709
            # we don't care about the rest of this page
634
710
            first_entry_num = 0
635
 
            first_fields = entries[0].split('\0')
 
711
            first_fields = entries[0].split(b'\0')
636
712
            if len(first_fields) < entry_field_count:
637
713
                # We didn't get the complete first entry
638
714
                # so move start, and grab the next, which
639
715
                # should be a full entry
640
716
                start += len(entries[0])+1
641
 
                first_fields = entries[1].split('\0')
 
717
                first_fields = entries[1].split(b'\0')
642
718
                first_entry_num = 1
643
719
 
644
720
            if len(first_fields) <= 2:
652
728
                # after this first record.
653
729
                after = start
654
730
                if first_fields[1]:
655
 
                    first_path = first_fields[1] + '/' + first_fields[2]
 
731
                    first_path = first_fields[1] + b'/' + first_fields[2]
656
732
                else:
657
733
                    first_path = first_fields[2]
658
734
                first_loc = _bisect_path_left(cur_files, first_path)
668
744
 
669
745
                # Parse the last entry
670
746
                last_entry_num = len(entries)-1
671
 
                last_fields = entries[last_entry_num].split('\0')
 
747
                last_fields = entries[last_entry_num].split(b'\0')
672
748
                if len(last_fields) < entry_field_count:
673
749
                    # The very last hunk was not complete,
674
750
                    # read the previous hunk
675
751
                    after = mid + len(block) - len(entries[-1])
676
752
                    last_entry_num -= 1
677
 
                    last_fields = entries[last_entry_num].split('\0')
 
753
                    last_fields = entries[last_entry_num].split(b'\0')
678
754
                else:
679
755
                    after = mid + len(block)
680
756
 
681
757
                if last_fields[1]:
682
 
                    last_path = last_fields[1] + '/' + last_fields[2]
 
758
                    last_path = last_fields[1] + b'/' + last_fields[2]
683
759
                else:
684
760
                    last_path = last_fields[2]
685
761
                last_loc = _bisect_path_right(post, last_path)
705
781
                    # careful if we should append rather than overwrite
706
782
                    if last_entry_num != first_entry_num:
707
783
                        paths.setdefault(last_path, []).append(last_fields)
708
 
                    for num in xrange(first_entry_num+1, last_entry_num):
 
784
                    for num in range(first_entry_num+1, last_entry_num):
709
785
                        # TODO: jam 20070223 We are already splitting here, so
710
786
                        #       shouldn't we just split the whole thing rather
711
787
                        #       than doing the split again in add_one_record?
712
 
                        fields = entries[num].split('\0')
 
788
                        fields = entries[num].split(b'\0')
713
789
                        if fields[1]:
714
 
                            path = fields[1] + '/' + fields[2]
 
790
                            path = fields[1] + b'/' + fields[2]
715
791
                        else:
716
792
                            path = fields[2]
717
793
                        paths.setdefault(path, []).append(fields)
767
843
        file_size = os.fstat(state_file.fileno()).st_size
768
844
        # We end up with 2 extra fields, we should have a trailing '\n' to
769
845
        # ensure that we read the whole record, and we should have a precursur
770
 
        # '' which ensures that we start after the previous '\n'
 
846
        # b'' which ensures that we start after the previous '\n'
771
847
        entry_field_count = self._fields_per_entry() + 1
772
848
 
773
849
        low = self._end_of_header
801
877
            if count > max_count:
802
878
                raise errors.BzrError('Too many seeks, most likely a bug.')
803
879
 
804
 
            mid = max(low, (low+high-page_size)/2)
 
880
            mid = max(low, (low+high-page_size)//2)
805
881
 
806
882
            state_file.seek(mid)
807
883
            # limit the read size, so we don't end up reading data that we have
810
886
            block = state_file.read(read_size)
811
887
 
812
888
            start = mid
813
 
            entries = block.split('\n')
 
889
            entries = block.split(b'\n')
814
890
 
815
891
            if len(entries) < 2:
816
892
                # We didn't find a '\n', so we cannot have found any records.
824
900
            # Check the first and last entries, in case they are partial, or if
825
901
            # we don't care about the rest of this page
826
902
            first_entry_num = 0
827
 
            first_fields = entries[0].split('\0')
 
903
            first_fields = entries[0].split(b'\0')
828
904
            if len(first_fields) < entry_field_count:
829
905
                # We didn't get the complete first entry
830
906
                # so move start, and grab the next, which
831
907
                # should be a full entry
832
908
                start += len(entries[0])+1
833
 
                first_fields = entries[1].split('\0')
 
909
                first_fields = entries[1].split(b'\0')
834
910
                first_entry_num = 1
835
911
 
836
912
            if len(first_fields) <= 1:
857
933
 
858
934
                # Parse the last entry
859
935
                last_entry_num = len(entries)-1
860
 
                last_fields = entries[last_entry_num].split('\0')
 
936
                last_fields = entries[last_entry_num].split(b'\0')
861
937
                if len(last_fields) < entry_field_count:
862
938
                    # The very last hunk was not complete,
863
939
                    # read the previous hunk
864
940
                    after = mid + len(block) - len(entries[-1])
865
941
                    last_entry_num -= 1
866
 
                    last_fields = entries[last_entry_num].split('\0')
 
942
                    last_fields = entries[last_entry_num].split(b'\0')
867
943
                else:
868
944
                    after = mid + len(block)
869
945
 
891
967
                    # careful if we should append rather than overwrite
892
968
                    if last_entry_num != first_entry_num:
893
969
                        paths.setdefault(last_dir, []).append(last_fields)
894
 
                    for num in xrange(first_entry_num+1, last_entry_num):
 
970
                    for num in range(first_entry_num+1, last_entry_num):
895
971
                        # TODO: jam 20070223 We are already splitting here, so
896
972
                        #       shouldn't we just split the whole thing rather
897
973
                        #       than doing the split again in add_one_record?
898
 
                        fields = entries[num].split('\0')
 
974
                        fields = entries[num].split(b'\0')
899
975
                        paths.setdefault(fields[1], []).append(fields)
900
976
 
901
977
                    for cur_dir in middle_files:
926
1002
        directories. (and renames?)
927
1003
 
928
1004
        :param paths: A sorted list of (dir, name) pairs
929
 
             eg: [('', 'a'), ('', 'f'), ('a/b', 'c')]
 
1005
             eg: [('', b'a'), ('', b'f'), ('a/b', b'c')]
930
1006
        :return: A dictionary mapping (dir, name, file_id) => [tree_info]
931
1007
        """
932
1008
        # Map from (dir, name, file_id) => [tree_info]
943
1019
            # Directories that need to be read
944
1020
            pending_dirs = set()
945
1021
            paths_to_search = set()
946
 
            for entry_list in newly_found.itervalues():
 
1022
            for entry_list in viewvalues(newly_found):
947
1023
                for dir_name_id, trees_info in entry_list:
948
1024
                    found[dir_name_id] = trees_info
949
1025
                    found_dir_names.add(dir_name_id[:2])
950
1026
                    is_dir = False
951
1027
                    for tree_info in trees_info:
952
1028
                        minikind = tree_info[0]
953
 
                        if minikind == 'd':
 
1029
                        if minikind == b'd':
954
1030
                            if is_dir:
955
1031
                                # We already processed this one as a directory,
956
1032
                                # we don't need to do the extra work again.
960
1036
                            is_dir = True
961
1037
                            if path not in processed_dirs:
962
1038
                                pending_dirs.add(path)
963
 
                        elif minikind == 'r':
 
1039
                        elif minikind == b'r':
964
1040
                            # Rename, we need to directly search the target
965
1041
                            # which is contained in the fingerprint column
966
1042
                            dir_name = osutils.split(tree_info[1])
993
1069
            return
994
1070
        # only require all dirblocks if we are doing a full-pass removal.
995
1071
        self._read_dirblocks_if_needed()
996
 
        dead_patterns = set([('a', 'r'), ('a', 'a'), ('r', 'r'), ('r', 'a')])
 
1072
        dead_patterns = {(b'a', b'r'), (b'a', b'a'), (b'r', b'r'), (b'r', b'a')}
997
1073
        def iter_entries_removable():
998
1074
            for block in self._dirblocks:
999
1075
                deleted_positions = []
1018
1094
 
1019
1095
        self._ghosts = []
1020
1096
        self._parents = [parents[0]]
1021
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
1022
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
1097
        self._mark_modified(header_modified=True)
1023
1098
 
1024
1099
    def _empty_parent_info(self):
1025
1100
        return [DirState.NULL_PARENT_DETAILS] * (len(self._parents) -
1045
1120
        :param dirname: The utf8 dirname to ensure there is a block for.
1046
1121
        :return: The index for the block.
1047
1122
        """
1048
 
        if dirname == '' and parent_row_index == 0 and parent_block_index == 0:
 
1123
        if dirname == b'' and parent_row_index == 0 and parent_block_index == 0:
1049
1124
            # This is the signature of the root row, and the
1050
1125
            # contents-of-root row is always index 1
1051
1126
            return 1
1052
1127
        # the basename of the directory must be the end of its full name.
1053
1128
        if not (parent_block_index == -1 and
1054
 
            parent_block_index == -1 and dirname == ''):
 
1129
            parent_block_index == -1 and dirname == b''):
1055
1130
            if not dirname.endswith(
1056
1131
                    self._dirblocks[parent_block_index][1][parent_row_index][0][1]):
1057
1132
                raise AssertionError("bad dirname %r" % dirname)
1058
 
        block_index, present = self._find_block_index_from_key((dirname, '', ''))
 
1133
        block_index, present = self._find_block_index_from_key((dirname, b'', b''))
1059
1134
        if not present:
1060
1135
            ## In future, when doing partial parsing, this should load and
1061
1136
            # populate the entire block.
1072
1147
            to prevent unneeded overhead when callers have a sorted list already.
1073
1148
        :return: Nothing.
1074
1149
        """
1075
 
        if new_entries[0][0][0:2] != ('', ''):
 
1150
        if new_entries[0][0][0:2] != (b'', b''):
1076
1151
            raise AssertionError(
1077
1152
                "Missing root row %r" % (new_entries[0][0],))
1078
1153
        # The two blocks here are deliberate: the root block and the
1079
1154
        # contents-of-root block.
1080
 
        self._dirblocks = [('', []), ('', [])]
 
1155
        self._dirblocks = [(b'', []), (b'', [])]
1081
1156
        current_block = self._dirblocks[0][1]
1082
 
        current_dirname = ''
1083
 
        root_key = ('', '')
 
1157
        current_dirname = b''
 
1158
        root_key = (b'', b'')
1084
1159
        append_entry = current_block.append
1085
1160
        for entry in new_entries:
1086
1161
            if entry[0][0] != current_dirname:
1102
1177
        # The above loop leaves the "root block" entries mixed with the
1103
1178
        # "contents-of-root block". But we don't want an if check on
1104
1179
        # all entries, so instead we just fix it up here.
1105
 
        if self._dirblocks[1] != ('', []):
 
1180
        if self._dirblocks[1] != (b'', []):
1106
1181
            raise ValueError("bad dirblock start %r" % (self._dirblocks[1],))
1107
1182
        root_block = []
1108
1183
        contents_of_root_block = []
1111
1186
                root_block.append(entry)
1112
1187
            else:
1113
1188
                contents_of_root_block.append(entry)
1114
 
        self._dirblocks[0] = ('', root_block)
1115
 
        self._dirblocks[1] = ('', contents_of_root_block)
 
1189
        self._dirblocks[0] = (b'', root_block)
 
1190
        self._dirblocks[1] = (b'', contents_of_root_block)
1116
1191
 
1117
1192
    def _entries_for_path(self, path):
1118
1193
        """Return a list with all the entries that match path for all ids."""
1119
1194
        dirname, basename = os.path.split(path)
1120
 
        key = (dirname, basename, '')
 
1195
        key = (dirname, basename, b'')
1121
1196
        block_index, present = self._find_block_index_from_key(key)
1122
1197
        if not present:
1123
1198
            # the block which should contain path is absent.
1146
1221
            # minikind
1147
1222
            entire_entry[tree_offset + 0] = tree_data[0]
1148
1223
            # size
1149
 
            entire_entry[tree_offset + 2] = str(tree_data[2])
 
1224
            entire_entry[tree_offset + 2] = b'%d' % tree_data[2]
1150
1225
            # executable
1151
1226
            entire_entry[tree_offset + 3] = DirState._to_yesno[tree_data[3]]
1152
 
        return '\0'.join(entire_entry)
 
1227
        return b'\0'.join(entire_entry)
1153
1228
 
1154
1229
    def _fields_per_entry(self):
1155
1230
        """How many null separated fields should be in each entry row.
1156
1231
 
1157
 
        Each line now has an extra '\n' field which is not used
 
1232
        Each line now has an extra '\\n' field which is not used
1158
1233
        so we just skip over it
1159
 
        entry size:
 
1234
 
 
1235
        entry size::
1160
1236
            3 fields for the key
1161
1237
            + number of fields per tree_data (5) * tree count
1162
1238
            + newline
1191
1267
 
1192
1268
        :return: The block index, True if the block for the key is present.
1193
1269
        """
1194
 
        if key[0:2] == ('', ''):
 
1270
        if key[0:2] == (b'', b''):
1195
1271
            return 0, True
1196
1272
        try:
1197
1273
            if (self._last_block_index is not None and
1203
1279
                                      cache=self._split_path_cache)
1204
1280
        # _right returns one-past-where-key is so we have to subtract
1205
1281
        # one to use it. we use _right here because there are two
1206
 
        # '' blocks - the root, and the contents of root
 
1282
        # b'' blocks - the root, and the contents of root
1207
1283
        # we always have a minimum of 2 in self._dirblocks: root and
1208
 
        # root-contents, and for '', we get 2 back, so this is
 
1284
        # root-contents, and for b'', we get 2 back, so this is
1209
1285
        # simple and correct:
1210
1286
        present = (block_index < len(self._dirblocks) and
1211
1287
            self._dirblocks[block_index][0] == key[0])
1253
1329
        result = DirState.initialize(dir_state_filename,
1254
1330
            sha1_provider=sha1_provider)
1255
1331
        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()
 
1332
            with tree.lock_read():
 
1333
                try:
 
1334
                    parent_ids = tree.get_parent_ids()
 
1335
                    num_parents = len(parent_ids)
 
1336
                    parent_trees = []
 
1337
                    for parent_id in parent_ids:
 
1338
                        parent_tree = tree.branch.repository.revision_tree(parent_id)
 
1339
                        parent_trees.append((parent_id, parent_tree))
 
1340
                        parent_tree.lock_read()
 
1341
                    result.set_parent_trees(parent_trees, [])
 
1342
                    result.set_state_from_inventory(tree.root_inventory)
 
1343
                finally:
 
1344
                    for revid, parent_tree in parent_trees:
 
1345
                        parent_tree.unlock()
1271
1346
        except:
1272
1347
            # The caller won't have a chance to unlock this, so make sure we
1273
1348
            # cleanup ourselves
1275
1350
            raise
1276
1351
        return result
1277
1352
 
 
1353
    def _check_delta_is_valid(self, delta):
 
1354
        delta = list(inventory._check_delta_unique_ids(
 
1355
                     inventory._check_delta_unique_old_paths(
 
1356
                     inventory._check_delta_unique_new_paths(
 
1357
                     inventory._check_delta_ids_match_entry(
 
1358
                     inventory._check_delta_ids_are_valid(
 
1359
                     inventory._check_delta_new_path_entry_both_or_None(delta)))))))
 
1360
        def delta_key(d):
 
1361
            (old_path, new_path, file_id, new_entry) = d
 
1362
            if old_path is None:
 
1363
                old_path = ''
 
1364
            if new_path is None:
 
1365
                new_path = ''
 
1366
            return (old_path, new_path, file_id, new_entry)
 
1367
        delta.sort(key=delta_key, reverse=True)
 
1368
        return delta
 
1369
 
1278
1370
    def update_by_delta(self, delta):
1279
1371
        """Apply an inventory delta to the dirstate for tree 0
1280
1372
 
1298
1390
        new_ids = set()
1299
1391
        # This loop transforms the delta to single atomic operations that can
1300
1392
        # 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):
 
1393
        delta = self._check_delta_is_valid(delta)
 
1394
        for old_path, new_path, file_id, inv_entry in delta:
 
1395
            if not isinstance(file_id, bytes):
 
1396
                raise AssertionError(
 
1397
                    "must be a utf8 file_id not %s" % (type(file_id), ))
1308
1398
            if (file_id in insertions) or (file_id in removals):
1309
 
                raise errors.InconsistentDelta(old_path or new_path, file_id,
 
1399
                self._raise_invalid(old_path or new_path, file_id,
1310
1400
                    "repeated file_id")
1311
1401
            if old_path is not None:
1312
1402
                old_path = old_path.encode('utf-8')
1315
1405
                new_ids.add(file_id)
1316
1406
            if new_path is not None:
1317
1407
                if inv_entry is None:
1318
 
                    raise errors.InconsistentDelta(new_path, file_id,
 
1408
                    self._raise_invalid(new_path, file_id,
1319
1409
                        "new_path with no entry")
1320
1410
                new_path = new_path.encode('utf-8')
1321
1411
                dirname_utf8, basename = osutils.split(new_path)
1323
1413
                    parents.add((dirname_utf8, inv_entry.parent_id))
1324
1414
                key = (dirname_utf8, basename, file_id)
1325
1415
                minikind = DirState._kind_to_minikind[inv_entry.kind]
1326
 
                if minikind == 't':
1327
 
                    fingerprint = inv_entry.reference_revision or ''
 
1416
                if minikind == b't':
 
1417
                    fingerprint = inv_entry.reference_revision or b''
1328
1418
                else:
1329
 
                    fingerprint = ''
 
1419
                    fingerprint = b''
1330
1420
                insertions[file_id] = (key, minikind, inv_entry.executable,
1331
1421
                                       fingerprint, new_path)
1332
1422
            # Transform moves into delete+add pairs
1351
1441
                                               fingerprint, new_child_path)
1352
1442
        self._check_delta_ids_absent(new_ids, delta, 0)
1353
1443
        try:
1354
 
            self._apply_removals(removals.iteritems())
1355
 
            self._apply_insertions(insertions.values())
 
1444
            self._apply_removals(viewitems(removals))
 
1445
            self._apply_insertions(viewvalues(insertions))
1356
1446
            # Validate parents
1357
1447
            self._after_delta_check_parents(parents, 0)
1358
 
        except errors.BzrError, e:
 
1448
        except errors.BzrError as e:
1359
1449
            self._changes_aborted = True
1360
1450
            if 'integrity error' not in str(e):
1361
1451
                raise
1362
1452
            # _get_entry raises BzrError when a request is inconsistent; we
1363
1453
            # want such errors to be shown as InconsistentDelta - and that 
1364
1454
            # fits the behaviour we trigger.
1365
 
            raise errors.InconsistentDeltaDelta(delta, "error from _get_entry.")
 
1455
            raise errors.InconsistentDeltaDelta(delta,
 
1456
                "error from _get_entry. %s" % (e,))
1366
1457
 
1367
1458
    def _apply_removals(self, removals):
1368
1459
        for file_id, path in sorted(removals, reverse=True,
1373
1464
            try:
1374
1465
                entry = self._dirblocks[block_i][1][entry_i]
1375
1466
            except IndexError:
1376
 
                self._changes_aborted = True
1377
 
                raise errors.InconsistentDelta(path, file_id,
 
1467
                self._raise_invalid(path, file_id,
1378
1468
                    "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,
 
1469
            if not f_present or entry[1][0][0] in (b'a', b'r'):
 
1470
                self._raise_invalid(path, file_id,
1382
1471
                    "Wrong path for old path.")
1383
1472
            if file_id != entry[0][2]:
1384
 
                self._changes_aborted = True
1385
 
                raise errors.InconsistentDelta(path, file_id,
 
1473
                self._raise_invalid(path, file_id,
1386
1474
                    "Attempt to remove path has wrong id - found %r."
1387
1475
                    % entry[0][2])
1388
1476
            self._make_absent(entry)
1392
1480
            # is rare enough it shouldn't be an issue (famous last words?) RBC
1393
1481
            # 20080730.
1394
1482
            block_i, entry_i, d_present, f_present = \
1395
 
                self._get_block_entry_index(path, '', 0)
 
1483
                self._get_block_entry_index(path, b'', 0)
1396
1484
            if d_present:
1397
1485
                # The dir block is still present in the dirstate; this could
1398
1486
                # be due to it being in a parent tree, or a corrupt delta.
1399
1487
                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],
 
1488
                    if child_entry[1][0][0] not in (b'r', b'a'):
 
1489
                        self._raise_invalid(path, entry[0][2],
1403
1490
                            "The file id was deleted but its children were "
1404
1491
                            "not deleted.")
1405
1492
 
1409
1496
                self.update_minimal(key, minikind, executable, fingerprint,
1410
1497
                                    path_utf8=path_utf8)
1411
1498
        except errors.NotVersionedError:
1412
 
            self._changes_aborted = True
1413
 
            raise errors.InconsistentDelta(path_utf8.decode('utf8'), key[2],
 
1499
            self._raise_invalid(path_utf8.decode('utf8'), key[2],
1414
1500
                "Missing parent")
1415
1501
 
1416
1502
    def update_basis_by_delta(self, delta, new_revid):
1424
1510
        Note that an exception during the operation of this method will leave
1425
1511
        the dirstate in a corrupt state where it should not be saved.
1426
1512
 
1427
 
        Finally, we expect all changes to be synchronising the basis tree with
1428
 
        the working tree.
1429
 
 
1430
1513
        :param new_revid: The new revision id for the trees parent.
1431
1514
        :param delta: An inventory delta (see apply_inventory_delta) describing
1432
1515
            the changes from the current left most parent revision to new_revid.
1444
1527
 
1445
1528
        self._parents[0] = new_revid
1446
1529
 
1447
 
        delta = sorted(delta, reverse=True)
 
1530
        delta = self._check_delta_is_valid(delta)
1448
1531
        adds = []
1449
1532
        changes = []
1450
1533
        deletes = []
1460
1543
        # expanding them recursively as needed.
1461
1544
        # At the same time, to reduce interface friction we convert the input
1462
1545
        # inventory entries to dirstate.
1463
 
        root_only = ('', '')
 
1546
        root_only = (b'', b'')
1464
1547
        # Accumulate parent references (path_utf8, id), to check for parentless
1465
1548
        # items or items placed under files/links/tree-references. We get
1466
1549
        # references from every item in the delta that is not a deletion and
1470
1553
        # ids.
1471
1554
        new_ids = set()
1472
1555
        for old_path, new_path, file_id, inv_entry in delta:
 
1556
            if file_id.__class__ is not bytes:
 
1557
                raise AssertionError(
 
1558
                    "must be a utf8 file_id not %s" % (type(file_id), ))
1473
1559
            if inv_entry is not None and file_id != inv_entry.file_id:
1474
 
                raise errors.InconsistentDelta(new_path, file_id,
 
1560
                self._raise_invalid(new_path, file_id,
1475
1561
                    "mismatched entry file_id %r" % inv_entry)
1476
 
            if new_path is not None:
 
1562
            if new_path is None:
 
1563
                new_path_utf8 = None
 
1564
            else:
1477
1565
                if inv_entry is None:
1478
 
                    raise errors.InconsistentDelta(new_path, file_id,
 
1566
                    self._raise_invalid(new_path, file_id,
1479
1567
                        "new_path with no entry")
1480
1568
                new_path_utf8 = encode(new_path)
1481
1569
                # note the parent for validation
1483
1571
                if basename_utf8:
1484
1572
                    parents.add((dirname_utf8, inv_entry.parent_id))
1485
1573
            if old_path is None:
1486
 
                adds.append((None, encode(new_path), file_id,
 
1574
                old_path_utf8 = None
 
1575
            else:
 
1576
                old_path_utf8 = encode(old_path)
 
1577
            if old_path is None:
 
1578
                adds.append((None, new_path_utf8, file_id,
1487
1579
                    inv_to_entry(inv_entry), True))
1488
1580
                new_ids.add(file_id)
1489
1581
            elif new_path is None:
1490
 
                deletes.append((encode(old_path), None, file_id, None, True))
1491
 
            elif (old_path, new_path) != root_only:
 
1582
                deletes.append((old_path_utf8, None, file_id, None, True))
 
1583
            elif (old_path, new_path) == root_only:
 
1584
                # change things in-place
 
1585
                # Note: the case of a parent directory changing its file_id
 
1586
                #       tends to break optimizations here, because officially
 
1587
                #       the file has actually been moved, it just happens to
 
1588
                #       end up at the same path. If we can figure out how to
 
1589
                #       handle that case, we can avoid a lot of add+delete
 
1590
                #       pairs for objects that stay put.
 
1591
                # elif old_path == new_path:
 
1592
                changes.append((old_path_utf8, new_path_utf8, file_id,
 
1593
                                inv_to_entry(inv_entry)))
 
1594
            else:
1492
1595
                # Renames:
1493
1596
                # Because renames must preserve their children we must have
1494
1597
                # processed all relocations and removes before hand. The sort
1497
1600
                # pair will result in the deleted item being reinserted, or
1498
1601
                # renamed items being reinserted twice - and possibly at the
1499
1602
                # 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
 
1603
                # the handling of entries with (b'f', ...), (b'r' ...) because
 
1604
                # the target of the b'r' is old_path here, and we add that to
1502
1605
                # deletes, meaning that the add handler does not need to check
1503
 
                # for 'r' items on every pass.
 
1606
                # for b'r' items on every pass.
1504
1607
                self._update_basis_apply_deletes(deletes)
1505
1608
                deletes = []
1506
1609
                # Split into an add/delete pair recursively.
1507
 
                adds.append((None, new_path_utf8, file_id,
1508
 
                    inv_to_entry(inv_entry), False))
 
1610
                adds.append((old_path_utf8, new_path_utf8, file_id,
 
1611
                             inv_to_entry(inv_entry), False))
1509
1612
                # Expunge deletes that we've seen so that deleted/renamed
1510
1613
                # children of a rename directory are handled correctly.
1511
 
                new_deletes = reversed(list(self._iter_child_entries(1,
1512
 
                    encode(old_path))))
 
1614
                new_deletes = reversed(list(
 
1615
                    self._iter_child_entries(1, old_path_utf8)))
1513
1616
                # Remove the current contents of the tree at orig_path, and
1514
1617
                # reinsert at the correct new path.
1515
1618
                for entry in new_deletes:
1516
 
                    if entry[0][0]:
1517
 
                        source_path = entry[0][0] + '/' + entry[0][1]
 
1619
                    child_dirname, child_basename, child_file_id = entry[0]
 
1620
                    if child_dirname:
 
1621
                        source_path = child_dirname + b'/' + child_basename
1518
1622
                    else:
1519
 
                        source_path = entry[0][1]
 
1623
                        source_path = child_basename
1520
1624
                    if new_path_utf8:
1521
 
                        target_path = new_path_utf8 + source_path[len(old_path):]
 
1625
                        target_path = \
 
1626
                            new_path_utf8 + source_path[len(old_path_utf8):]
1522
1627
                    else:
1523
 
                        if old_path == '':
 
1628
                        if old_path_utf8 == b'':
1524
1629
                            raise AssertionError("cannot rename directory to"
1525
 
                                " itself")
1526
 
                        target_path = source_path[len(old_path) + 1:]
 
1630
                                                 " itself")
 
1631
                        target_path = source_path[len(old_path_utf8) + 1:]
1527
1632
                    adds.append((None, target_path, entry[0][2], entry[1][1], False))
1528
1633
                    deletes.append(
1529
1634
                        (source_path, target_path, entry[0][2], None, False))
1530
1635
                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)))
 
1636
                    (old_path_utf8, new_path_utf8, file_id, None, False))
 
1637
 
1537
1638
        self._check_delta_ids_absent(new_ids, delta, 1)
1538
1639
        try:
1539
1640
            # Finish expunging deletes/first half of renames.
1544
1645
            self._update_basis_apply_changes(changes)
1545
1646
            # Validate parents
1546
1647
            self._after_delta_check_parents(parents, 1)
1547
 
        except errors.BzrError, e:
 
1648
        except errors.BzrError as e:
1548
1649
            self._changes_aborted = True
1549
1650
            if 'integrity error' not in str(e):
1550
1651
                raise
1551
1652
            # _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.")
 
1653
            # want such errors to be shown as InconsistentDelta - and that
 
1654
            # fits the behaviour we trigger.
 
1655
            raise errors.InconsistentDeltaDelta(delta,
 
1656
                "error from _get_entry. %s" % (e,))
1557
1657
 
1558
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
1559
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
1658
        self._mark_modified(header_modified=True)
1560
1659
        self._id_index = None
1561
1660
        return
1562
1661
 
1566
1665
            return
1567
1666
        id_index = self._get_id_index()
1568
1667
        for file_id in new_ids:
1569
 
            for key in id_index.get(file_id, []):
 
1668
            for key in id_index.get(file_id, ()):
1570
1669
                block_i, entry_i, d_present, f_present = \
1571
1670
                    self._get_block_entry_index(key[0], key[1], tree_index)
1572
1671
                if not f_present:
1576
1675
                if entry[0][2] != file_id:
1577
1676
                    # Different file_id, so not what we want.
1578
1677
                    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,
 
1678
                self._raise_invalid((b"%s/%s" % key[0:2]).decode('utf8'), file_id,
1583
1679
                    "This file_id is new in the delta but already present in "
1584
1680
                    "the target")
1585
1681
 
 
1682
    def _raise_invalid(self, path, file_id, reason):
 
1683
        self._changes_aborted = True
 
1684
        raise errors.InconsistentDelta(path, file_id, reason)
 
1685
 
1586
1686
    def _update_basis_apply_adds(self, adds):
1587
1687
        """Apply a sequence of adds to tree 1 during update_basis_by_delta.
1588
1688
 
1596
1696
        """
1597
1697
        # Adds are accumulated partly from renames, so can be in any input
1598
1698
        # order - sort it.
1599
 
        adds.sort()
 
1699
        # TODO: we may want to sort in dirblocks order. That way each entry
 
1700
        #       will end up in the same directory, allowing the _get_entry
 
1701
        #       fast-path for looking up 2 items in the same dir work.
 
1702
        adds.sort(key=lambda x: x[1])
1600
1703
        # adds is now in lexographic order, which places all parents before
1601
1704
        # their children, so we can process it linearly.
1602
 
        absent = 'ar'
 
1705
        st = static_tuple.StaticTuple
1603
1706
        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
 
1707
            dirname, basename = osutils.split(new_path)
 
1708
            entry_key = st(dirname, basename, file_id)
 
1709
            block_index, present = self._find_block_index_from_key(entry_key)
 
1710
            if not present:
 
1711
                # The block where we want to put the file is not present.
 
1712
                # However, it might have just been an empty directory. Look for
 
1713
                # the parent in the basis-so-far before throwing an error.
 
1714
                parent_dir, parent_base = osutils.split(dirname)
 
1715
                parent_block_idx, parent_entry_idx, _, parent_present = \
 
1716
                    self._get_block_entry_index(parent_dir, parent_base, 1)
 
1717
                if not parent_present:
 
1718
                    self._raise_invalid(new_path, file_id,
 
1719
                        "Unable to find block for this record."
 
1720
                        " Was the parent added?")
 
1721
                self._ensure_block(parent_block_idx, parent_entry_idx, dirname)
 
1722
 
 
1723
            block = self._dirblocks[block_index][1]
 
1724
            entry_index, present = self._find_entry_index(entry_key, block)
 
1725
            if real_add:
 
1726
                if old_path is not None:
 
1727
                    self._raise_invalid(new_path, file_id,
 
1728
                        'considered a real add but still had old_path at %s'
 
1729
                        % (old_path,))
 
1730
            if present:
 
1731
                entry = block[entry_index]
 
1732
                basis_kind = entry[1][1][0]
 
1733
                if basis_kind == b'a':
 
1734
                    entry[1][1] = new_details
 
1735
                elif basis_kind == b'r':
 
1736
                    raise NotImplementedError()
 
1737
                else:
 
1738
                    self._raise_invalid(new_path, file_id,
 
1739
                        "An entry was marked as a new add"
 
1740
                        " but the basis target already existed")
 
1741
            else:
 
1742
                # The exact key was not found in the block. However, we need to
 
1743
                # check if there is a key next to us that would have matched.
 
1744
                # We only need to check 2 locations, because there are only 2
 
1745
                # trees present.
 
1746
                for maybe_index in range(entry_index-1, entry_index+1):
 
1747
                    if maybe_index < 0 or maybe_index >= len(block):
 
1748
                        continue
 
1749
                    maybe_entry = block[maybe_index]
 
1750
                    if maybe_entry[0][:2] != (dirname, basename):
 
1751
                        # Just a random neighbor
 
1752
                        continue
 
1753
                    if maybe_entry[0][2] == file_id:
 
1754
                        raise AssertionError(
 
1755
                            '_find_entry_index didnt find a key match'
 
1756
                            ' but walking the data did, for %s'
 
1757
                            % (entry_key,))
 
1758
                    basis_kind = maybe_entry[1][1][0]
 
1759
                    if basis_kind not in (b'a', b'r'):
 
1760
                        self._raise_invalid(new_path, file_id,
 
1761
                            "we have an add record for path, but the path"
 
1762
                            " is already present with another file_id %s"
 
1763
                            % (maybe_entry[0][2],))
 
1764
 
 
1765
                entry = (entry_key, [DirState.NULL_PARENT_DETAILS,
 
1766
                                     new_details])
 
1767
                block.insert(entry_index, entry)
 
1768
 
 
1769
            active_kind = entry[1][0][0]
 
1770
            if active_kind == b'a':
 
1771
                # The active record shows up as absent, this could be genuine,
 
1772
                # or it could be present at some other location. We need to
 
1773
                # verify.
 
1774
                id_index = self._get_id_index()
 
1775
                # The id_index may not be perfectly accurate for tree1, because
 
1776
                # we haven't been keeping it updated. However, it should be
 
1777
                # fine for tree0, and that gives us enough info for what we
 
1778
                # need
 
1779
                keys = id_index.get(file_id, ())
 
1780
                for key in keys:
 
1781
                    block_i, entry_i, d_present, f_present = \
 
1782
                        self._get_block_entry_index(key[0], key[1], 0)
 
1783
                    if not f_present:
 
1784
                        continue
 
1785
                    active_entry = self._dirblocks[block_i][1][entry_i]
 
1786
                    if (active_entry[0][2] != file_id):
 
1787
                        # Some other file is at this path, we don't need to
 
1788
                        # link it.
 
1789
                        continue
 
1790
                    real_active_kind = active_entry[1][0][0]
 
1791
                    if real_active_kind in (b'a', b'r'):
 
1792
                        # We found a record, which was not *this* record,
 
1793
                        # which matches the file_id, but is not actually
 
1794
                        # present. Something seems *really* wrong.
 
1795
                        self._raise_invalid(new_path, file_id,
 
1796
                            "We found a tree0 entry that doesnt make sense")
 
1797
                    # Now, we've found a tree0 entry which matches the file_id
 
1798
                    # but is at a different location. So update them to be
 
1799
                    # rename records.
 
1800
                    active_dir, active_name = active_entry[0][:2]
 
1801
                    if active_dir:
 
1802
                        active_path = active_dir + b'/' + active_name
 
1803
                    else:
 
1804
                        active_path = active_name
 
1805
                    active_entry[1][1] = st(b'r', new_path, 0, False, b'')
 
1806
                    entry[1][0] = st(b'r', active_path, 0, False, b'')
 
1807
            elif active_kind == b'r':
 
1808
                raise NotImplementedError()
 
1809
 
 
1810
            new_kind = new_details[0]
 
1811
            if new_kind == b'd':
 
1812
                self._ensure_block(block_index, entry_index, new_path)
1619
1813
 
1620
1814
    def _update_basis_apply_changes(self, changes):
1621
1815
        """Apply a sequence of changes to tree 1 during update_basis_by_delta.
1623
1817
        :param adds: A sequence of changes. Each change is a tuple:
1624
1818
            (path_utf8, path_utf8, file_id, (entry_details))
1625
1819
        """
1626
 
        absent = 'ar'
1627
1820
        for old_path, new_path, file_id, new_details in changes:
1628
1821
            # 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')
 
1822
            entry = self._get_entry(1, file_id, new_path)
 
1823
            if entry[0] is None or entry[1][1][0] in (b'a', b'r'):
 
1824
                self._raise_invalid(new_path, file_id,
 
1825
                    'changed entry considered not present')
1639
1826
            entry[1][1] = new_details
1640
1827
 
1641
1828
    def _update_basis_apply_deletes(self, deletes):
1653
1840
        null = DirState.NULL_PARENT_DETAILS
1654
1841
        for old_path, new_path, file_id, _, real_delete in deletes:
1655
1842
            if real_delete != (new_path is None):
1656
 
                self._changes_aborted = True
1657
 
                raise AssertionError("bad delete delta")
 
1843
                self._raise_invalid(old_path, file_id, "bad delete delta")
1658
1844
            # the entry for this file_id must be in tree 1.
1659
1845
            dirname, basename = osutils.split(old_path)
1660
1846
            block_index, entry_index, dir_present, file_present = \
1661
1847
                self._get_block_entry_index(dirname, basename, 1)
1662
1848
            if not file_present:
1663
 
                self._changes_aborted = True
1664
 
                raise errors.InconsistentDelta(old_path, file_id,
 
1849
                self._raise_invalid(old_path, file_id,
1665
1850
                    'basis tree does not contain removed entry')
1666
1851
            entry = self._dirblocks[block_index][1][entry_index]
 
1852
            # The state of the entry in the 'active' WT
 
1853
            active_kind = entry[1][0][0]
1667
1854
            if entry[0][2] != file_id:
1668
 
                self._changes_aborted = True
1669
 
                raise errors.InconsistentDelta(old_path, file_id,
 
1855
                self._raise_invalid(old_path, file_id,
1670
1856
                    '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.')
 
1857
            dir_block = ()
 
1858
            old_kind = entry[1][1][0]
 
1859
            if active_kind in b'ar':
 
1860
                # The active tree doesn't have this file_id.
 
1861
                # The basis tree is changing this record. If this is a
 
1862
                # rename, then we don't want the record here at all
 
1863
                # anymore. If it is just an in-place change, we want the
 
1864
                # record here, but we'll add it if we need to. So we just
 
1865
                # delete it
 
1866
                if active_kind == b'r':
 
1867
                    active_path = entry[1][0][1]
 
1868
                    active_entry = self._get_entry(0, file_id, active_path)
 
1869
                    if active_entry[1][1][0] != b'r':
 
1870
                        self._raise_invalid(old_path, file_id,
 
1871
                            "Dirstate did not have matching rename entries")
 
1872
                    elif active_entry[1][0][0] in b'ar':
 
1873
                        self._raise_invalid(old_path, file_id,
 
1874
                            "Dirstate had a rename pointing at an inactive"
 
1875
                            " tree0")
 
1876
                    active_entry[1][1] = null
1677
1877
                del self._dirblocks[block_index][1][entry_index]
 
1878
                if old_kind == b'd':
 
1879
                    # This was a directory, and the active tree says it
 
1880
                    # doesn't exist, and now the basis tree says it doesn't
 
1881
                    # exist. Remove its dirblock if present
 
1882
                    (dir_block_index,
 
1883
                     present) = self._find_block_index_from_key(
 
1884
                        (old_path, b'', b''))
 
1885
                    if present:
 
1886
                        dir_block = self._dirblocks[dir_block_index][1]
 
1887
                        if not dir_block:
 
1888
                            # This entry is empty, go ahead and just remove it
 
1889
                            del self._dirblocks[dir_block_index]
1678
1890
            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
 
1891
                # There is still an active record, so just mark this
 
1892
                # removed.
 
1893
                entry[1][1] = null
 
1894
                block_i, entry_i, d_present, f_present = \
 
1895
                    self._get_block_entry_index(old_path, b'', 1)
 
1896
                if d_present:
 
1897
                    dir_block = self._dirblocks[block_i][1]
 
1898
            for child_entry in dir_block:
 
1899
                child_basis_kind = child_entry[1][1][0]
 
1900
                if child_basis_kind not in b'ar':
 
1901
                    self._raise_invalid(old_path, file_id,
 
1902
                        "The file id was deleted but its children were "
 
1903
                        "not deleted.")
1694
1904
 
1695
1905
    def _after_delta_check_parents(self, parents, index):
1696
1906
        """Check that parents required by the delta are all intact.
1705
1915
            # has the right file id.
1706
1916
            entry = self._get_entry(index, file_id, dirname_utf8)
1707
1917
            if entry[1] is None:
1708
 
                self._changes_aborted = True
1709
 
                raise errors.InconsistentDelta(dirname_utf8.decode('utf8'),
 
1918
                self._raise_invalid(dirname_utf8.decode('utf8'),
1710
1919
                    file_id, "This parent is not present.")
1711
1920
            # 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'),
 
1921
            if entry[1][index][0] != b'd':
 
1922
                self._raise_invalid(dirname_utf8.decode('utf8'),
1715
1923
                    file_id, "This parent is not a directory.")
1716
1924
 
1717
1925
    def _observed_sha1(self, entry, sha1, stat_value,
1718
 
        _stat_to_minikind=_stat_to_minikind, _pack_stat=pack_stat):
 
1926
        _stat_to_minikind=_stat_to_minikind):
1719
1927
        """Note the sha1 of a file.
1720
1928
 
1721
1929
        :param entry: The entry the sha1 is for.
1723
1931
        :param stat_value: The os.lstat for the file.
1724
1932
        """
1725
1933
        try:
1726
 
            minikind = _stat_to_minikind[stat_value.st_mode & 0170000]
 
1934
            minikind = _stat_to_minikind[stat_value.st_mode & 0o170000]
1727
1935
        except KeyError:
1728
1936
            # Unhandled kind
1729
1937
            return None
1730
 
        packed_stat = _pack_stat(stat_value)
1731
 
        if minikind == 'f':
 
1938
        if minikind == b'f':
1732
1939
            if self._cutoff_time is None:
1733
1940
                self._sha_cutoff_time()
1734
1941
            if (stat_value.st_mtime < self._cutoff_time
1735
1942
                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
 
1943
                entry[1][0] = (b'f', sha1, stat_value.st_size, entry[1][0][3],
 
1944
                               pack_stat(stat_value))
 
1945
                self._mark_modified([entry])
1739
1946
 
1740
1947
    def _sha_cutoff_time(self):
1741
1948
        """Return cutoff time.
1778
1985
        #       higher level, because there either won't be anything on disk,
1779
1986
        #       or the thing on disk will be a file.
1780
1987
        fs_encoding = osutils._fs_enc
1781
 
        if isinstance(abspath, unicode):
 
1988
        if isinstance(abspath, text_type):
1782
1989
            # abspath is defined as the path to pass to lstat. readlink is
1783
1990
            # buggy in python < 2.6 (it doesn't encode unicode path into FS
1784
1991
            # encoding), so we need to encode ourselves knowing that unicode
1785
1992
            # paths are produced by UnicodeDirReader on purpose.
1786
1993
            abspath = abspath.encode(fs_encoding)
1787
1994
        target = os.readlink(abspath)
1788
 
        if fs_encoding not in ('UTF-8', 'US-ASCII', 'ANSI_X3.4-1968'):
 
1995
        if fs_encoding not in ('utf-8', 'ascii'):
1789
1996
            # Change encoding if needed
1790
1997
            target = target.decode(fs_encoding).encode('UTF-8')
1791
1998
        return target
1799
2006
        """Serialise the entire dirstate to a sequence of lines."""
1800
2007
        if (self._header_state == DirState.IN_MEMORY_UNMODIFIED and
1801
2008
            self._dirblock_state == DirState.IN_MEMORY_UNMODIFIED):
1802
 
            # read whats on disk.
 
2009
            # read what's on disk.
1803
2010
            self._state_file.seek(0)
1804
2011
            return self._state_file.readlines()
1805
2012
        lines = []
1806
2013
        lines.append(self._get_parents_line(self.get_parent_ids()))
1807
2014
        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()))
 
2015
        lines.extend(self._iter_entry_lines())
1810
2016
        return self._get_output_lines(lines)
1811
2017
 
1812
2018
    def _get_ghosts_line(self, ghost_ids):
1813
2019
        """Create a line for the state file for ghost information."""
1814
 
        return '\0'.join([str(len(ghost_ids))] + ghost_ids)
 
2020
        return b'\0'.join([b'%d' % len(ghost_ids)] + ghost_ids)
1815
2021
 
1816
2022
    def _get_parents_line(self, parent_ids):
1817
2023
        """Create a line for the state file for parents information."""
1818
 
        return '\0'.join([str(len(parent_ids))] + parent_ids)
 
2024
        return b'\0'.join([b'%d' % len(parent_ids)] + parent_ids)
 
2025
 
 
2026
    def _iter_entry_lines(self):
 
2027
        """Create lines for entries."""
 
2028
        return map(self._entry_to_line, self._iter_entries())
1819
2029
 
1820
2030
    def _get_fields_to_entry(self):
1821
2031
        """Get a function which converts entry fields into a entry record.
1835
2045
                        fields[3],                # minikind
1836
2046
                        fields[4],                # fingerprint
1837
2047
                        _int(fields[5]),          # size
1838
 
                        fields[6] == 'y',         # executable
 
2048
                        fields[6] == b'y',         # executable
1839
2049
                        fields[7],                # packed_stat or revision_id
1840
2050
                    )])
1841
2051
            return fields_to_entry_0_parents
1847
2057
                        fields[3],                # minikind
1848
2058
                        fields[4],                # fingerprint
1849
2059
                        _int(fields[5]),          # size
1850
 
                        fields[6] == 'y',         # executable
 
2060
                        fields[6] == b'y',         # executable
1851
2061
                        fields[7],                # packed_stat or revision_id
1852
2062
                    ),
1853
2063
                    ( # Parent 1
1854
2064
                        fields[8],                # minikind
1855
2065
                        fields[9],                # fingerprint
1856
2066
                        _int(fields[10]),         # size
1857
 
                        fields[11] == 'y',        # executable
 
2067
                        fields[11] == b'y',        # executable
1858
2068
                        fields[12],               # packed_stat or revision_id
1859
2069
                    ),
1860
2070
                    ])
1867
2077
                        fields[3],                # minikind
1868
2078
                        fields[4],                # fingerprint
1869
2079
                        _int(fields[5]),          # size
1870
 
                        fields[6] == 'y',         # executable
 
2080
                        fields[6] == b'y',         # executable
1871
2081
                        fields[7],                # packed_stat or revision_id
1872
2082
                    ),
1873
2083
                    ( # Parent 1
1874
2084
                        fields[8],                # minikind
1875
2085
                        fields[9],                # fingerprint
1876
2086
                        _int(fields[10]),         # size
1877
 
                        fields[11] == 'y',        # executable
 
2087
                        fields[11] == b'y',        # executable
1878
2088
                        fields[12],               # packed_stat or revision_id
1879
2089
                    ),
1880
2090
                    ( # Parent 2
1881
2091
                        fields[13],               # minikind
1882
2092
                        fields[14],               # fingerprint
1883
2093
                        _int(fields[15]),         # size
1884
 
                        fields[16] == 'y',        # executable
 
2094
                        fields[16] == b'y',        # executable
1885
2095
                        fields[17],               # packed_stat or revision_id
1886
2096
                    ),
1887
2097
                    ])
1892
2102
                trees = [(fields[cur],                # minikind
1893
2103
                          fields[cur+1],              # fingerprint
1894
2104
                          _int(fields[cur+2]),        # size
1895
 
                          fields[cur+3] == 'y',       # executable
 
2105
                          fields[cur+3] == b'y',       # executable
1896
2106
                          fields[cur+4],              # stat or revision_id
1897
 
                         ) for cur in xrange(3, len(fields)-1, 5)]
 
2107
                         ) for cur in range(3, len(fields)-1, 5)]
1898
2108
                return path_name_file_id_key, trees
1899
2109
            return fields_to_entry_n_parents
1900
2110
 
1924
2134
            tree present there.
1925
2135
        """
1926
2136
        self._read_dirblocks_if_needed()
1927
 
        key = dirname, basename, ''
 
2137
        key = dirname, basename, b''
1928
2138
        block_index, present = self._find_block_index_from_key(key)
1929
2139
        if not present:
1930
2140
            # no such directory - return the dir index and 0 for the row.
1934
2144
        # linear search through entries at this path to find the one
1935
2145
        # requested.
1936
2146
        while entry_index < len(block) and block[entry_index][0][1] == basename:
1937
 
            if block[entry_index][1][tree_index][0] not in 'ar':
 
2147
            if block[entry_index][1][tree_index][0] not in (b'a', b'r'):
1938
2148
                # neither absent or relocated
1939
2149
                return block_index, entry_index, True, True
1940
2150
            entry_index += 1
1941
2151
        return block_index, entry_index, True, False
1942
2152
 
1943
 
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None, include_deleted=False):
 
2153
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None,
 
2154
                   include_deleted=False):
1944
2155
        """Get the dirstate entry for path in tree tree_index.
1945
2156
 
1946
2157
        If either file_id or path is supplied, it is used as the key to lookup.
1961
2172
        """
1962
2173
        self._read_dirblocks_if_needed()
1963
2174
        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'
 
2175
            if not isinstance(path_utf8, bytes):
 
2176
                raise errors.BzrError('path_utf8 is not bytes: %s %r'
1966
2177
                    % (type(path_utf8), path_utf8))
1967
2178
            # path lookups are faster
1968
2179
            dirname, basename = osutils.split(path_utf8)
1971
2182
            if not file_present:
1972
2183
                return None, None
1973
2184
            entry = self._dirblocks[block_index][1][entry_index]
1974
 
            if not (entry[0][2] and entry[1][tree_index][0] not in ('a', 'r')):
 
2185
            if not (entry[0][2] and entry[1][tree_index][0] not in (b'a', b'r')):
1975
2186
                raise AssertionError('unversioned entry?')
1976
2187
            if fileid_utf8:
1977
2188
                if entry[0][2] != fileid_utf8:
1980
2191
                                          ' tree_index, file_id and path')
1981
2192
            return entry
1982
2193
        else:
1983
 
            possible_keys = self._get_id_index().get(fileid_utf8, None)
 
2194
            possible_keys = self._get_id_index().get(fileid_utf8, ())
1984
2195
            if not possible_keys:
1985
2196
                return None, None
1986
2197
            for key in possible_keys:
1999
2210
                    entry = self._dirblocks[block_index][1][entry_index]
2000
2211
                    # TODO: We might want to assert that entry[0][2] ==
2001
2212
                    #       fileid_utf8.
2002
 
                    if entry[1][tree_index][0] in 'fdlt':
 
2213
                    # GZ 2017-06-09: Hoist set of minkinds somewhere
 
2214
                    if entry[1][tree_index][0] in {b'f', b'd', b'l', b't'}:
2003
2215
                        # this is the result we are looking for: the
2004
2216
                        # real home of this file_id in this tree.
2005
2217
                        return entry
2006
 
                    if entry[1][tree_index][0] == 'a':
 
2218
                    if entry[1][tree_index][0] == b'a':
2007
2219
                        # there is no home for this entry in this tree
2008
2220
                        if include_deleted:
2009
2221
                            return entry
2010
2222
                        return None, None
2011
 
                    if entry[1][tree_index][0] != 'r':
 
2223
                    if entry[1][tree_index][0] != b'r':
2012
2224
                        raise AssertionError(
2013
2225
                            "entry %r has invalid minikind %r for tree %r" \
2014
2226
                            % (entry,
2040
2252
            sha1_provider = DefaultSHA1Provider()
2041
2253
        result = cls(path, sha1_provider)
2042
2254
        # root dir and root dir contents with no children.
2043
 
        empty_tree_dirblocks = [('', []), ('', [])]
 
2255
        empty_tree_dirblocks = [(b'', []), (b'', [])]
2044
2256
        # a new root directory, with a NULLSTAT.
2045
2257
        empty_tree_dirblocks[0][1].append(
2046
 
            (('', '', inventory.ROOT_ID), [
2047
 
                ('d', '', 0, False, DirState.NULLSTAT),
 
2258
            ((b'', b'', inventory.ROOT_ID), [
 
2259
                (b'd', b'', 0, False, DirState.NULLSTAT),
2048
2260
            ]))
2049
2261
        result.lock_write()
2050
2262
        try:
2068
2280
        minikind = DirState._kind_to_minikind[kind]
2069
2281
        tree_data = inv_entry.revision
2070
2282
        if kind == 'directory':
2071
 
            fingerprint = ''
 
2283
            fingerprint = b''
2072
2284
            size = 0
2073
2285
            executable = False
2074
2286
        elif kind == 'symlink':
2075
2287
            if inv_entry.symlink_target is None:
2076
 
                fingerprint = ''
 
2288
                fingerprint = b''
2077
2289
            else:
2078
2290
                fingerprint = inv_entry.symlink_target.encode('utf8')
2079
2291
            size = 0
2080
2292
            executable = False
2081
2293
        elif kind == 'file':
2082
 
            fingerprint = inv_entry.text_sha1 or ''
 
2294
            fingerprint = inv_entry.text_sha1 or b''
2083
2295
            size = inv_entry.text_size or 0
2084
2296
            executable = inv_entry.executable
2085
2297
        elif kind == 'tree-reference':
2086
 
            fingerprint = inv_entry.reference_revision or ''
 
2298
            fingerprint = inv_entry.reference_revision or b''
2087
2299
            size = 0
2088
2300
            executable = False
2089
2301
        else:
2090
2302
            raise Exception("can't pack %s" % inv_entry)
2091
 
        return (minikind, fingerprint, size, executable, tree_data)
 
2303
        return static_tuple.StaticTuple(minikind, fingerprint, size,
 
2304
                                        executable, tree_data)
2092
2305
 
2093
2306
    def _iter_child_entries(self, tree_index, path_utf8):
2094
2307
        """Iterate over all the entries that are children of path_utf.
2095
2308
 
2096
 
        This only returns entries that are present (not in 'a', 'r') in
 
2309
        This only returns entries that are present (not in b'a', b'r') in
2097
2310
        tree_index. tree_index data is not refreshed, so if tree 0 is used,
2098
2311
        results may differ from that obtained if paths were statted to
2099
2312
        determine what ones were directories.
2103
2316
        """
2104
2317
        pending_dirs = []
2105
2318
        next_pending_dirs = [path_utf8]
2106
 
        absent = 'ar'
 
2319
        absent = (b'a', b'r')
2107
2320
        while next_pending_dirs:
2108
2321
            pending_dirs = next_pending_dirs
2109
2322
            next_pending_dirs = []
2110
2323
            for path in pending_dirs:
2111
2324
                block_index, present = self._find_block_index_from_key(
2112
 
                    (path, '', ''))
 
2325
                    (path, b'', b''))
2113
2326
                if block_index == 0:
2114
2327
                    block_index = 1
2115
2328
                    if len(self._dirblocks) == 1:
2124
2337
                    kind = entry[1][tree_index][0]
2125
2338
                    if kind not in absent:
2126
2339
                        yield entry
2127
 
                    if kind == 'd':
 
2340
                    if kind == b'd':
2128
2341
                        if entry[0][0]:
2129
 
                            path = entry[0][0] + '/' + entry[0][1]
 
2342
                            path = entry[0][0] + b'/' + entry[0][1]
2130
2343
                        else:
2131
2344
                            path = entry[0][1]
2132
2345
                        next_pending_dirs.append(path)
2135
2348
        """Iterate over all the entries in the dirstate.
2136
2349
 
2137
2350
        Each yelt item is an entry in the standard format described in the
2138
 
        docstring of bzrlib.dirstate.
 
2351
        docstring of breezy.dirstate.
2139
2352
        """
2140
2353
        self._read_dirblocks_if_needed()
2141
2354
        for directory in self._dirblocks:
2143
2356
                yield entry
2144
2357
 
2145
2358
    def _get_id_index(self):
2146
 
        """Get an id index of self._dirblocks."""
 
2359
        """Get an id index of self._dirblocks.
 
2360
 
 
2361
        This maps from file_id => [(directory, name, file_id)] entries where
 
2362
        that file_id appears in one of the trees.
 
2363
        """
2147
2364
        if self._id_index is None:
2148
2365
            id_index = {}
2149
2366
            for key, tree_details in self._iter_entries():
2150
 
                id_index.setdefault(key[2], set()).add(key)
 
2367
                self._add_to_id_index(id_index, key)
2151
2368
            self._id_index = id_index
2152
2369
        return self._id_index
2153
2370
 
 
2371
    def _add_to_id_index(self, id_index, entry_key):
 
2372
        """Add this entry to the _id_index mapping."""
 
2373
        # This code used to use a set for every entry in the id_index. However,
 
2374
        # it is *rare* to have more than one entry. So a set is a large
 
2375
        # overkill. And even when we do, we won't ever have more than the
 
2376
        # number of parent trees. Which is still a small number (rarely >2). As
 
2377
        # such, we use a simple tuple, and do our own uniqueness checks. While
 
2378
        # the 'in' check is O(N) since N is nicely bounded it shouldn't ever
 
2379
        # cause quadratic failure.
 
2380
        file_id = entry_key[2]
 
2381
        entry_key = static_tuple.StaticTuple.from_sequence(entry_key)
 
2382
        if file_id not in id_index:
 
2383
            id_index[file_id] = static_tuple.StaticTuple(entry_key,)
 
2384
        else:
 
2385
            entry_keys = id_index[file_id]
 
2386
            if entry_key not in entry_keys:
 
2387
                id_index[file_id] = entry_keys + (entry_key,)
 
2388
 
 
2389
    def _remove_from_id_index(self, id_index, entry_key):
 
2390
        """Remove this entry from the _id_index mapping.
 
2391
 
 
2392
        It is an programming error to call this when the entry_key is not
 
2393
        already present.
 
2394
        """
 
2395
        file_id = entry_key[2]
 
2396
        entry_keys = list(id_index[file_id])
 
2397
        entry_keys.remove(entry_key)
 
2398
        id_index[file_id] = static_tuple.StaticTuple.from_sequence(entry_keys)
 
2399
 
2154
2400
    def _get_output_lines(self, lines):
2155
2401
        """Format lines for final output.
2156
2402
 
2158
2404
            path lines.
2159
2405
        """
2160
2406
        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),))
 
2407
        lines.append(b'') # a final newline
 
2408
        inventory_text = b'\0\n\0'.join(lines)
 
2409
        output_lines.append(b'crc32: %d\n' % (zlib.crc32(inventory_text),))
2164
2410
        # -3, 1 for num parents, 1 for ghosts, 1 for final newline
2165
2411
        num_entries = len(lines)-3
2166
 
        output_lines.append('num_entries: %s\n' % (num_entries,))
 
2412
        output_lines.append(b'num_entries: %d\n' % (num_entries,))
2167
2413
        output_lines.append(inventory_text)
2168
2414
        return output_lines
2169
2415
 
2170
2416
    def _make_deleted_row(self, fileid_utf8, parents):
2171
2417
        """Return a deleted row for fileid_utf8."""
2172
 
        return ('/', 'RECYCLED.BIN', 'file', fileid_utf8, 0, DirState.NULLSTAT,
2173
 
            ''), parents
 
2418
        return (b'/', b'RECYCLED.BIN', b'file', fileid_utf8, 0, DirState.NULLSTAT,
 
2419
            b''), parents
2174
2420
 
2175
2421
    def _num_present_parents(self):
2176
2422
        """The number of parent entries in each record row."""
2177
2423
        return len(self._parents) - len(self._ghosts)
2178
2424
 
2179
 
    @staticmethod
2180
 
    def on_file(path, sha1_provider=None):
 
2425
    @classmethod
 
2426
    def on_file(cls, path, sha1_provider=None, worth_saving_limit=0):
2181
2427
        """Construct a DirState on the file at path "path".
2182
2428
 
2183
2429
        :param path: The path at which the dirstate file on disk should live.
2184
2430
        :param sha1_provider: an object meeting the SHA1Provider interface.
2185
2431
            If None, a DefaultSHA1Provider is used.
 
2432
        :param worth_saving_limit: when the exact number of hash changed
 
2433
            entries is known, only bother saving the dirstate if more than
 
2434
            this count of entries have changed. -1 means never save.
2186
2435
        :return: An unlocked DirState object, associated with the given path.
2187
2436
        """
2188
2437
        if sha1_provider is None:
2189
2438
            sha1_provider = DefaultSHA1Provider()
2190
 
        result = DirState(path, sha1_provider)
 
2439
        result = cls(path, sha1_provider,
 
2440
                     worth_saving_limit=worth_saving_limit)
2191
2441
        return result
2192
2442
 
2193
2443
    def _read_dirblocks_if_needed(self):
2211
2461
        """
2212
2462
        self._read_prelude()
2213
2463
        parent_line = self._state_file.readline()
2214
 
        info = parent_line.split('\0')
 
2464
        info = parent_line.split(b'\0')
2215
2465
        num_parents = int(info[0])
2216
2466
        self._parents = info[1:-1]
2217
2467
        ghost_line = self._state_file.readline()
2218
 
        info = ghost_line.split('\0')
 
2468
        info = ghost_line.split(b'\0')
2219
2469
        num_ghosts = int(info[1])
2220
2470
        self._ghosts = info[2:-1]
2221
2471
        self._header_state = DirState.IN_MEMORY_UNMODIFIED
2243
2493
            raise errors.BzrError(
2244
2494
                'invalid header line: %r' % (header,))
2245
2495
        crc_line = self._state_file.readline()
2246
 
        if not crc_line.startswith('crc32: '):
 
2496
        if not crc_line.startswith(b'crc32: '):
2247
2497
            raise errors.BzrError('missing crc32 checksum: %r' % crc_line)
2248
 
        self.crc_expected = int(crc_line[len('crc32: '):-1])
 
2498
        self.crc_expected = int(crc_line[len(b'crc32: '):-1])
2249
2499
        num_entries_line = self._state_file.readline()
2250
 
        if not num_entries_line.startswith('num_entries: '):
 
2500
        if not num_entries_line.startswith(b'num_entries: '):
2251
2501
            raise errors.BzrError('missing num_entries line')
2252
 
        self._num_entries = int(num_entries_line[len('num_entries: '):-1])
 
2502
        self._num_entries = int(num_entries_line[len(b'num_entries: '):-1])
2253
2503
 
2254
 
    def sha1_from_stat(self, path, stat_result, _pack_stat=pack_stat):
 
2504
    def sha1_from_stat(self, path, stat_result):
2255
2505
        """Find a sha1 given a stat lookup."""
2256
 
        return self._get_packed_stat_index().get(_pack_stat(stat_result), None)
 
2506
        return self._get_packed_stat_index().get(pack_stat(stat_result), None)
2257
2507
 
2258
2508
    def _get_packed_stat_index(self):
2259
2509
        """Get a packed_stat index of self._dirblocks."""
2260
2510
        if self._packed_stat_index is None:
2261
2511
            index = {}
2262
2512
            for key, tree_details in self._iter_entries():
2263
 
                if tree_details[0][0] == 'f':
 
2513
                if tree_details[0][0] == b'f':
2264
2514
                    index[tree_details[0][4]] = tree_details[0][1]
2265
2515
            self._packed_stat_index = index
2266
2516
        return self._packed_stat_index
2285
2535
            trace.mutter('Not saving DirState because '
2286
2536
                    '_changes_aborted is set.')
2287
2537
            return
2288
 
        if (self._header_state == DirState.IN_MEMORY_MODIFIED or
2289
 
            self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
 
2538
        # TODO: Since we now distinguish IN_MEMORY_MODIFIED from
 
2539
        #       IN_MEMORY_HASH_MODIFIED, we should only fail quietly if we fail
 
2540
        #       to save an IN_MEMORY_HASH_MODIFIED, and fail *noisily* if we
 
2541
        #       fail to save IN_MEMORY_MODIFIED
 
2542
        if not self._worth_saving():
 
2543
            return
2290
2544
 
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.
 
2545
        grabbed_write_lock = False
 
2546
        if self._lock_state != 'w':
 
2547
            grabbed_write_lock, new_lock = self._lock_token.temporary_write_lock()
 
2548
            # Switch over to the new lock, as the old one may be closed.
 
2549
            # TODO: jam 20070315 We should validate the disk file has
 
2550
            #       not changed contents, since temporary_write_lock may
 
2551
            #       not be an atomic operation.
 
2552
            self._lock_token = new_lock
 
2553
            self._state_file = new_lock.f
 
2554
            if not grabbed_write_lock:
 
2555
                # We couldn't grab a write lock, so we switch back to a read one
 
2556
                return
 
2557
        try:
 
2558
            lines = self.get_lines()
 
2559
            self._state_file.seek(0)
 
2560
            self._state_file.writelines(lines)
 
2561
            self._state_file.truncate()
 
2562
            self._state_file.flush()
 
2563
            self._maybe_fdatasync()
 
2564
            self._mark_unmodified()
 
2565
        finally:
 
2566
            if grabbed_write_lock:
 
2567
                self._lock_token = self._lock_token.restore_read_lock()
 
2568
                self._state_file = self._lock_token.f
2295
2569
                # TODO: jam 20070315 We should validate the disk file has
2296
 
                #       not changed contents. Since temporary_write_lock may
 
2570
                #       not changed contents. Since restore_read_lock may
2297
2571
                #       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.
 
2572
 
 
2573
    def _maybe_fdatasync(self):
 
2574
        """Flush to disk if possible and if not configured off."""
 
2575
        if self._config_stack.get('dirstate.fdatasync'):
 
2576
            osutils.fdatasync(self._state_file.fileno())
 
2577
 
 
2578
    def _worth_saving(self):
 
2579
        """Is it worth saving the dirstate or not?"""
 
2580
        if (self._header_state == DirState.IN_MEMORY_MODIFIED
 
2581
            or self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
 
2582
            return True
 
2583
        if self._dirblock_state == DirState.IN_MEMORY_HASH_MODIFIED:
 
2584
            if self._worth_saving_limit == -1:
 
2585
                # We never save hash changes when the limit is -1
 
2586
                return False
 
2587
            # If we're using smart saving and only a small number of
 
2588
            # entries have changed their hash, don't bother saving. John has
 
2589
            # suggested using a heuristic here based on the size of the
 
2590
            # changed files and/or tree. For now, we go with a configurable
 
2591
            # number of changes, keeping the calculation time
 
2592
            # as low overhead as possible. (This also keeps all existing
 
2593
            # tests passing as the default is 0, i.e. always save.)
 
2594
            if len(self._known_hash_changes) >= self._worth_saving_limit:
 
2595
                return True
 
2596
        return False
2317
2597
 
2318
2598
    def _set_data(self, parent_ids, dirblocks):
2319
2599
        """Set the full dirstate data in memory.
2328
2608
        """
2329
2609
        # our memory copy is now authoritative.
2330
2610
        self._dirblocks = dirblocks
2331
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
2332
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2611
        self._mark_modified(header_modified=True)
2333
2612
        self._parents = list(parent_ids)
2334
2613
        self._id_index = None
2335
2614
        self._packed_stat_index = None
2337
2616
    def set_path_id(self, path, new_id):
2338
2617
        """Change the id of path to new_id in the current working tree.
2339
2618
 
2340
 
        :param path: The path inside the tree to set - '' is the root, 'foo'
 
2619
        :param path: The path inside the tree to set - b'' is the root, 'foo'
2341
2620
            is the path foo in the root.
2342
2621
        :param new_id: The new id to assign to the path. This must be a utf8
2343
2622
            file id (not unicode, and not None).
2351
2630
        if entry[0][2] == new_id:
2352
2631
            # Nothing to change.
2353
2632
            return
 
2633
        if new_id.__class__ != bytes:
 
2634
            raise AssertionError(
 
2635
                "must be a utf8 file_id not %s" % (type(new_id), ))
2354
2636
        # mark the old path absent, and insert a new root path
2355
2637
        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
 
2638
        self.update_minimal((b'', b'', new_id), b'd',
 
2639
            path_utf8=b'', packed_stat=entry[1][0][4])
 
2640
        self._mark_modified()
2359
2641
 
2360
2642
    def set_parent_trees(self, trees, ghosts):
2361
2643
        """Set the parent trees for the dirstate.
2406
2688
        parent_trees = [tree for rev_id, tree in trees if rev_id not in ghosts]
2407
2689
        # how many trees do we end up with
2408
2690
        parent_count = len(parent_trees)
 
2691
        st = static_tuple.StaticTuple
2409
2692
 
2410
2693
        # one: the current tree
2411
2694
        for entry in self._iter_entries():
2412
2695
            # skip entries not in the current tree
2413
 
            if entry[1][0][0] in 'ar': # absent, relocated
 
2696
            if entry[1][0][0] in (b'a', b'r'): # absent, relocated
2414
2697
                continue
2415
2698
            by_path[entry[0]] = [entry[1][0]] + \
2416
2699
                [DirState.NULL_PARENT_DETAILS] * parent_count
2417
 
            id_index[entry[0][2]] = set([entry[0]])
 
2700
            # TODO: Possibly inline this, since we know it isn't present yet
 
2701
            #       id_index[entry[0][2]] = (entry[0],)
 
2702
            self._add_to_id_index(id_index, entry[0])
2418
2703
 
2419
2704
        # now the parent trees:
2420
2705
        for tree_index, tree in enumerate(parent_trees):
2426
2711
            # the suffix is from tree_index+1:parent_count+1.
2427
2712
            new_location_suffix = [DirState.NULL_PARENT_DETAILS] * (parent_count - tree_index)
2428
2713
            # now stitch in all the entries from this tree
2429
 
            for path, entry in tree.inventory.iter_entries_by_dir():
 
2714
            last_dirname = None
 
2715
            for path, entry in tree.iter_entries_by_dir():
2430
2716
                # here we process each trees details for each item in the tree.
2431
2717
                # we first update any existing entries for the id at other paths,
2432
2718
                # then we either create or update the entry for the id at the
2439
2725
                file_id = entry.file_id
2440
2726
                path_utf8 = path.encode('utf8')
2441
2727
                dirname, basename = osutils.split(path_utf8)
2442
 
                new_entry_key = (dirname, basename, file_id)
 
2728
                if dirname == last_dirname:
 
2729
                    # Try to re-use objects as much as possible
 
2730
                    dirname = last_dirname
 
2731
                else:
 
2732
                    last_dirname = dirname
 
2733
                new_entry_key = st(dirname, basename, file_id)
2443
2734
                # tree index consistency: All other paths for this id in this tree
2444
2735
                # index must point to the correct path.
2445
 
                for entry_key in id_index.setdefault(file_id, set()):
 
2736
                entry_keys = id_index.get(file_id, ())
 
2737
                for entry_key in entry_keys:
2446
2738
                    # TODO:PROFILING: It might be faster to just update
2447
2739
                    # rather than checking if we need to, and then overwrite
2448
2740
                    # the one we are located at.
2451
2743
                        # other trees, so put absent pointers there
2452
2744
                        # This is the vertical axis in the matrix, all pointing
2453
2745
                        # 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.
 
2746
                        by_path[entry_key][tree_index] = st(b'r', path_utf8, 0,
 
2747
                                                            False, b'')
 
2748
                # by path consistency: Insert into an existing path record
 
2749
                # (trivial), or add a new one with relocation pointers for the
 
2750
                # other tree indexes.
 
2751
                if new_entry_key in entry_keys:
 
2752
                    # there is already an entry where this data belongs, just
 
2753
                    # insert it.
2459
2754
                    by_path[new_entry_key][tree_index] = \
2460
2755
                        self._inv_entry_to_details(entry)
2461
2756
                else:
2463
2758
                    # mapping from path,id. We need to look up the correct path
2464
2759
                    # for the indexes from 0 to tree_index -1
2465
2760
                    new_details = []
2466
 
                    for lookup_index in xrange(tree_index):
 
2761
                    for lookup_index in range(tree_index):
2467
2762
                        # boundary case: this is the first occurence of file_id
2468
 
                        # so there are no id_indexs, possibly take this out of
 
2763
                        # so there are no id_indexes, possibly take this out of
2469
2764
                        # the loop?
2470
 
                        if not len(id_index[file_id]):
 
2765
                        if not len(entry_keys):
2471
2766
                            new_details.append(DirState.NULL_PARENT_DETAILS)
2472
2767
                        else:
2473
2768
                            # 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.
 
2769
                            a_key = next(iter(entry_keys))
 
2770
                            if by_path[a_key][lookup_index][0] in (b'r', b'a'):
 
2771
                                # its a pointer or missing statement, use it as
 
2772
                                # is.
2480
2773
                                new_details.append(by_path[a_key][lookup_index])
2481
2774
                            else:
2482
2775
                                # 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, ''))
 
2776
                                real_path = (b'/'.join(a_key[0:2])).strip(b'/')
 
2777
                                new_details.append(st(b'r', real_path, 0, False,
 
2778
                                                      b''))
2485
2779
                    new_details.append(self._inv_entry_to_details(entry))
2486
2780
                    new_details.extend(new_location_suffix)
2487
2781
                    by_path[new_entry_key] = new_details
2488
 
                    id_index[file_id].add(new_entry_key)
 
2782
                    self._add_to_id_index(id_index, new_entry_key)
2489
2783
        # --- end generation of full tree mappings
2490
2784
 
2491
2785
        # sort and output all the entries
2492
 
        new_entries = self._sort_entries(by_path.items())
 
2786
        new_entries = self._sort_entries(viewitems(by_path))
2493
2787
        self._entries_to_current_state(new_entries)
2494
2788
        self._parents = [rev_id for rev_id, tree in trees]
2495
2789
        self._ghosts = list(ghosts)
2496
 
        self._header_state = DirState.IN_MEMORY_MODIFIED
2497
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2790
        self._mark_modified(header_modified=True)
2498
2791
        self._id_index = id_index
2499
2792
 
2500
2793
    def _sort_entries(self, entry_list):
2504
2797
        try to keep everything in sorted blocks all the time, but sometimes
2505
2798
        it's easier to sort after the fact.
2506
2799
        """
2507
 
        def _key(entry):
 
2800
        # When sorting, we usually have 10x more entries than directories. (69k
 
2801
        # total entries, 4k directories). So cache the results of splitting.
 
2802
        # Saving time and objects. Also, use StaticTuple to avoid putting all
 
2803
        # of these object into python's garbage collector.
 
2804
        split_dirs = {}
 
2805
        def _key(entry, _split_dirs=split_dirs, _st=static_tuple.StaticTuple):
2508
2806
            # sort by: directory parts, file name, file id
2509
 
            return entry[0][0].split('/'), entry[0][1], entry[0][2]
 
2807
            dirpath, fname, file_id = entry[0]
 
2808
            try:
 
2809
                split = _split_dirs[dirpath]
 
2810
            except KeyError:
 
2811
                split = _st.from_sequence(dirpath.split(b'/'))
 
2812
                _split_dirs[dirpath] = split
 
2813
            return _st(split, fname, file_id)
2510
2814
        return sorted(entry_list, key=_key)
2511
2815
 
2512
2816
    def set_state_from_inventory(self, new_inv):
2542
2846
        # underlying dirstate.
2543
2847
        old_iterator = iter(list(self._iter_entries()))
2544
2848
        # both must have roots so this is safe:
2545
 
        current_new = new_iterator.next()
2546
 
        current_old = old_iterator.next()
 
2849
        current_new = next(new_iterator)
 
2850
        current_old = next(old_iterator)
2547
2851
        def advance(iterator):
2548
2852
            try:
2549
 
                return iterator.next()
 
2853
                return next(iterator)
2550
2854
            except StopIteration:
2551
2855
                return None
2552
2856
        while current_new or current_old:
2553
2857
            # skip entries in old that are not really there
2554
 
            if current_old and current_old[1][0][0] in 'ar':
 
2858
            if current_old and current_old[1][0][0] in (b'a', b'r'):
2555
2859
                # relocated or absent
2556
2860
                current_old = advance(old_iterator)
2557
2861
                continue
2563
2867
                new_entry_key = (new_dirname, new_basename, new_id)
2564
2868
                current_new_minikind = \
2565
2869
                    DirState._kind_to_minikind[current_new[1].kind]
2566
 
                if current_new_minikind == 't':
2567
 
                    fingerprint = current_new[1].reference_revision or ''
 
2870
                if current_new_minikind == b't':
 
2871
                    fingerprint = current_new[1].reference_revision or b''
2568
2872
                else:
2569
2873
                    # We normally only insert or remove records, or update
2570
2874
                    # them when it has significantly changed.  Then we want to
2571
2875
                    # erase its fingerprint.  Unaffected records should
2572
2876
                    # normally not be updated at all.
2573
 
                    fingerprint = ''
 
2877
                    fingerprint = b''
2574
2878
            else:
2575
2879
                # for safety disable variables
2576
2880
                new_path_utf8 = new_dirname = new_basename = new_id = \
2615
2919
                # both sides are dealt with, move on
2616
2920
                current_old = advance(old_iterator)
2617
2921
                current_new = advance(new_iterator)
2618
 
            elif (cmp_by_dirs(new_dirname, current_old[0][0]) < 0
 
2922
            elif (lt_by_dirs(new_dirname, current_old[0][0])
2619
2923
                  or (new_dirname == current_old[0][0]
2620
2924
                      and new_entry_key[1:] < current_old[0][1:])):
2621
2925
                # new comes before:
2637
2941
                        current_old[0][1].decode('utf8'))
2638
2942
                self._make_absent(current_old)
2639
2943
                current_old = advance(old_iterator)
2640
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
2944
        self._mark_modified()
2641
2945
        self._id_index = None
2642
2946
        self._packed_stat_index = None
2643
2947
        if tracing:
2644
2948
            trace.mutter("set_state_from_inventory complete.")
2645
2949
 
 
2950
    def set_state_from_scratch(self, working_inv, parent_trees, parent_ghosts):
 
2951
        """Wipe the currently stored state and set it to something new.
 
2952
 
 
2953
        This is a hard-reset for the data we are working with.
 
2954
        """
 
2955
        # Technically, we really want a write lock, but until we write, we
 
2956
        # don't really need it.
 
2957
        self._requires_lock()
 
2958
        # root dir and root dir contents with no children. We have to have a
 
2959
        # root for set_state_from_inventory to work correctly.
 
2960
        empty_root = ((b'', b'', inventory.ROOT_ID),
 
2961
                      [(b'd', b'', 0, False, DirState.NULLSTAT)])
 
2962
        empty_tree_dirblocks = [(b'', [empty_root]), (b'', [])]
 
2963
        self._set_data([], empty_tree_dirblocks)
 
2964
        self.set_state_from_inventory(working_inv)
 
2965
        self.set_parent_trees(parent_trees, parent_ghosts)
 
2966
 
2646
2967
    def _make_absent(self, current_old):
2647
2968
        """Mark current_old - an entry - as absent for tree 0.
2648
2969
 
2655
2976
        all_remaining_keys = set()
2656
2977
        # Dont check the working tree, because it's going.
2657
2978
        for details in current_old[1][1:]:
2658
 
            if details[0] not in 'ar': # absent, relocated
 
2979
            if details[0] not in (b'a', b'r'): # absent, relocated
2659
2980
                all_remaining_keys.add(current_old[0])
2660
 
            elif details[0] == 'r': # relocated
 
2981
            elif details[0] == b'r': # relocated
2661
2982
                # record the key for the real path.
2662
2983
                all_remaining_keys.add(tuple(osutils.split(details[1])) + (current_old[0][2],))
2663
2984
            # absent rows are not present at any path.
2673
2994
            block[1].pop(entry_index)
2674
2995
            # if we have an id_index in use, remove this key from it for this id.
2675
2996
            if self._id_index is not None:
2676
 
                self._id_index[current_old[0][2]].remove(current_old[0])
 
2997
                self._remove_from_id_index(self._id_index, current_old[0])
2677
2998
        # update all remaining keys for this id to record it as absent. The
2678
2999
        # existing details may either be the record we are marking as deleted
2679
3000
        # (if there were other trees with the id present at this path), or may
2689
3010
                raise AssertionError('could not find entry for %s' % (update_key,))
2690
3011
            update_tree_details = self._dirblocks[update_block_index][1][update_entry_index][1]
2691
3012
            # it must not be absent at the moment
2692
 
            if update_tree_details[0][0] == 'a': # absent
 
3013
            if update_tree_details[0][0] == b'a': # absent
2693
3014
                raise AssertionError('bad row %r' % (update_tree_details,))
2694
3015
            update_tree_details[0] = DirState.NULL_PARENT_DETAILS
2695
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
3016
        self._mark_modified()
2696
3017
        return last_reference
2697
3018
 
2698
 
    def update_minimal(self, key, minikind, executable=False, fingerprint='',
 
3019
    def update_minimal(self, key, minikind, executable=False, fingerprint=b'',
2699
3020
        packed_stat=None, size=0, path_utf8=None, fullscan=False):
2700
3021
        """Update an entry to the state in tree 0.
2701
3022
 
2704
3025
        updated as well.
2705
3026
 
2706
3027
        :param key: (dir, name, file_id) for the new entry
2707
 
        :param minikind: The type for the entry ('f' == 'file', 'd' ==
 
3028
        :param minikind: The type for the entry (b'f' == 'file', b'd' ==
2708
3029
                'directory'), etc.
2709
3030
        :param executable: Should the executable bit be set?
2710
3031
        :param fingerprint: Simple fingerprint for new entry: canonical-form
2723
3044
        block = self._find_block(key)[1]
2724
3045
        if packed_stat is None:
2725
3046
            packed_stat = DirState.NULLSTAT
2726
 
        # XXX: Some callers pass '' as the packed_stat, and it seems to be
 
3047
        # XXX: Some callers pass b'' as the packed_stat, and it seems to be
2727
3048
        # sometimes present in the dirstate - this seems oddly inconsistent.
2728
3049
        # mbp 20071008
2729
3050
        entry_index, present = self._find_entry_index(key, block)
2732
3053
        if not present:
2733
3054
            # New record. Check there isn't a entry at this path already.
2734
3055
            if not fullscan:
2735
 
                low_index, _ = self._find_entry_index(key[0:2] + ('',), block)
 
3056
                low_index, _ = self._find_entry_index(key[0:2] + (b'',), block)
2736
3057
                while low_index < len(block):
2737
3058
                    entry = block[low_index]
2738
3059
                    if entry[0][0:2] == key[0:2]:
2739
 
                        if entry[1][0][0] not in 'ar':
 
3060
                        if entry[1][0][0] not in (b'a', b'r'):
2740
3061
                            # This entry has the same path (but a different id) as
2741
3062
                            # the new entry we're adding, and is present in ths
2742
3063
                            # tree.
2743
 
                            raise errors.InconsistentDelta(
2744
 
                                ("%s/%s" % key[0:2]).decode('utf8'), key[2],
 
3064
                            self._raise_invalid(
 
3065
                                (b"%s/%s" % key[0:2]).decode('utf8'), key[2],
2745
3066
                                "Attempt to add item at path already occupied by "
2746
3067
                                "id %r" % entry[0][2])
2747
3068
                        low_index += 1
2748
3069
                    else:
2749
3070
                        break
2750
3071
            # new entry, synthesis cross reference here,
2751
 
            existing_keys = id_index.setdefault(key[2], set())
 
3072
            existing_keys = id_index.get(key[2], ())
2752
3073
            if not existing_keys:
2753
3074
                # not currently in the state, simplest case
2754
3075
                new_entry = key, [new_details] + self._empty_parent_info()
2784
3105
                    # entry, if not already examined, is skipped over by that
2785
3106
                    # loop.
2786
3107
                    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)
 
3108
                    other_entry[1][0] = (b'r', path_utf8, 0, False, b'')
 
3109
                    if self._maybe_remove_row(other_block, other_entry_index,
 
3110
                                              id_index):
 
3111
                        # If the row holding this was removed, we need to
 
3112
                        # recompute where this entry goes
 
3113
                        entry_index, _ = self._find_entry_index(key, block)
2790
3114
 
2791
3115
                # This loop:
2792
3116
                # adds a tuple to the new details for each column
2794
3118
                #  - or by creating a new pointer to the right row inside that column
2795
3119
                num_present_parents = self._num_present_parents()
2796
3120
                if num_present_parents:
 
3121
                    # TODO: This re-evaluates the existing_keys set, do we need
 
3122
                    #       to do that ourselves?
2797
3123
                    other_key = list(existing_keys)[0]
2798
 
                for lookup_index in xrange(1, num_present_parents + 1):
 
3124
                for lookup_index in range(1, num_present_parents + 1):
2799
3125
                    # grab any one entry, use it to find the right path.
2800
3126
                    # TODO: optimise this to reduce memory use in highly
2801
3127
                    # fragmented situations by reusing the relocation
2809
3135
                    if not present:
2810
3136
                        raise AssertionError('update_minimal: could not find entry for %s' % (other_key,))
2811
3137
                    update_details = self._dirblocks[update_block_index][1][update_entry_index][1][lookup_index]
2812
 
                    if update_details[0] in 'ar': # relocated, absent
 
3138
                    if update_details[0] in (b'a', b'r'): # relocated, absent
2813
3139
                        # its a pointer or absent in lookup_index's tree, use
2814
3140
                        # it as is.
2815
3141
                        new_entry[1].append(update_details)
2816
3142
                    else:
2817
3143
                        # we have the right key, make a pointer to it.
2818
3144
                        pointer_path = osutils.pathjoin(*other_key[0:2])
2819
 
                        new_entry[1].append(('r', pointer_path, 0, False, ''))
 
3145
                        new_entry[1].append((b'r', pointer_path, 0, False, b''))
2820
3146
            block.insert(entry_index, new_entry)
2821
 
            existing_keys.add(key)
 
3147
            self._add_to_id_index(id_index, key)
2822
3148
        else:
2823
3149
            # Does the new state matter?
2824
3150
            block[entry_index][1][0] = new_details
2833
3159
            # converted to relocated.
2834
3160
            if path_utf8 is None:
2835
3161
                raise AssertionError('no path')
2836
 
            for entry_key in id_index.setdefault(key[2], set()):
 
3162
            existing_keys = id_index.get(key[2], ())
 
3163
            if key not in existing_keys:
 
3164
                raise AssertionError('We found the entry in the blocks, but'
 
3165
                    ' the key is not in the id_index.'
 
3166
                    ' key: %s, existing_keys: %s' % (key, existing_keys))
 
3167
            for entry_key in existing_keys:
2837
3168
                # TODO:PROFILING: It might be faster to just update
2838
3169
                # rather than checking if we need to, and then overwrite
2839
3170
                # the one we are located at.
2849
3180
                    if not present:
2850
3181
                        raise AssertionError('not present: %r', entry_key)
2851
3182
                    self._dirblocks[block_index][1][entry_index][1][0] = \
2852
 
                        ('r', path_utf8, 0, False, '')
 
3183
                        (b'r', path_utf8, 0, False, b'')
2853
3184
        # add a containing dirblock if needed.
2854
 
        if new_details[0] == 'd':
2855
 
            subdir_key = (osutils.pathjoin(*key[0:2]), '', '')
 
3185
        if new_details[0] == b'd':
 
3186
            # GZ 2017-06-09: Using pathjoin why?
 
3187
            subdir_key = (osutils.pathjoin(*key[0:2]), b'', b'')
2856
3188
            block_index, present = self._find_block_index_from_key(subdir_key)
2857
3189
            if not present:
2858
3190
                self._dirblocks.insert(block_index, (subdir_key[0], []))
2859
3191
 
2860
 
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
3192
        self._mark_modified()
2861
3193
 
2862
3194
    def _maybe_remove_row(self, block, index, id_index):
2863
3195
        """Remove index if it is absent or relocated across the row.
2864
3196
        
2865
3197
        id_index is updated accordingly.
 
3198
        :return: True if we removed the row, False otherwise
2866
3199
        """
2867
3200
        present_in_row = False
2868
3201
        entry = block[index]
2869
3202
        for column in entry[1]:
2870
 
            if column[0] not in 'ar':
 
3203
            if column[0] not in (b'a', b'r'):
2871
3204
                present_in_row = True
2872
3205
                break
2873
3206
        if not present_in_row:
2874
3207
            block.pop(index)
2875
 
            id_index[entry[0][2]].remove(entry[0])
 
3208
            self._remove_from_id_index(id_index, entry[0])
 
3209
            return True
 
3210
        return False
2876
3211
 
2877
3212
    def _validate(self):
2878
3213
        """Check that invariants on the dirblock are correct.
2885
3220
        # NOTE: This must always raise AssertionError not just assert,
2886
3221
        # otherwise it may not behave properly under python -O
2887
3222
        #
2888
 
        # TODO: All entries must have some content that's not 'a' or 'r',
 
3223
        # TODO: All entries must have some content that's not b'a' or b'r',
2889
3224
        # otherwise it could just be removed.
2890
3225
        #
2891
3226
        # TODO: All relocations must point directly to a real entry.
2896
3231
        from pprint import pformat
2897
3232
        self._read_dirblocks_if_needed()
2898
3233
        if len(self._dirblocks) > 0:
2899
 
            if not self._dirblocks[0][0] == '':
 
3234
            if not self._dirblocks[0][0] == b'':
2900
3235
                raise AssertionError(
2901
3236
                    "dirblocks don't start with root block:\n" + \
2902
3237
                    pformat(self._dirblocks))
2903
3238
        if len(self._dirblocks) > 1:
2904
 
            if not self._dirblocks[1][0] == '':
 
3239
            if not self._dirblocks[1][0] == b'':
2905
3240
                raise AssertionError(
2906
3241
                    "dirblocks missing root directory:\n" + \
2907
3242
                    pformat(self._dirblocks))
2908
3243
        # the dirblocks are sorted by their path components, name, and dir id
2909
 
        dir_names = [d[0].split('/')
 
3244
        dir_names = [d[0].split(b'/')
2910
3245
                for d in self._dirblocks[1:]]
2911
3246
        if dir_names != sorted(dir_names):
2912
3247
            raise AssertionError(
2936
3271
            current tree. (It is invalid to have a non-absent file in an absent
2937
3272
            directory.)
2938
3273
            """
2939
 
            if entry[0][0:2] == ('', ''):
 
3274
            if entry[0][0:2] == (b'', b''):
2940
3275
                # There should be no parent for the root row
2941
3276
                return
2942
3277
            parent_entry = self._get_entry(tree_index, path_utf8=entry[0][0])
2944
3279
                raise AssertionError(
2945
3280
                    "no parent entry for: %s in tree %s"
2946
3281
                    % (this_path, tree_index))
2947
 
            if parent_entry[1][tree_index][0] != 'd':
 
3282
            if parent_entry[1][tree_index][0] != b'd':
2948
3283
                raise AssertionError(
2949
3284
                    "Parent entry for %s is not marked as a valid"
2950
3285
                    " directory. %s" % (this_path, parent_entry,))
2958
3293
        # We check this with a dict per tree pointing either to the present
2959
3294
        # name, or None if absent.
2960
3295
        tree_count = self._num_present_parents() + 1
2961
 
        id_path_maps = [dict() for i in range(tree_count)]
 
3296
        id_path_maps = [{} for _ in range(tree_count)]
2962
3297
        # Make sure that all renamed entries point to the correct location.
2963
3298
        for entry in self._iter_entries():
2964
3299
            file_id = entry[0][2]
2972
3307
            for tree_index, tree_state in enumerate(entry[1]):
2973
3308
                this_tree_map = id_path_maps[tree_index]
2974
3309
                minikind = tree_state[0]
2975
 
                if minikind in 'ar':
 
3310
                if minikind in (b'a', b'r'):
2976
3311
                    absent_positions += 1
2977
3312
                # have we seen this id before in this column?
2978
3313
                if file_id in this_tree_map:
2979
3314
                    previous_path, previous_loc = this_tree_map[file_id]
2980
3315
                    # any later mention of this file must be consistent with
2981
3316
                    # what was said before
2982
 
                    if minikind == 'a':
 
3317
                    if minikind == b'a':
2983
3318
                        if previous_path is not None:
2984
3319
                            raise AssertionError(
2985
3320
                            "file %s is absent in row %r but also present " \
2986
3321
                            "at %r"% \
2987
 
                            (file_id, entry, previous_path))
2988
 
                    elif minikind == 'r':
 
3322
                            (file_id.decode('utf-8'), entry, previous_path))
 
3323
                    elif minikind == b'r':
2989
3324
                        target_location = tree_state[1]
2990
3325
                        if previous_path != target_location:
2991
3326
                            raise AssertionError(
3001
3336
                                (entry, previous_path, previous_loc))
3002
3337
                        check_valid_parent()
3003
3338
                else:
3004
 
                    if minikind == 'a':
 
3339
                    if minikind == b'a':
3005
3340
                        # absent; should not occur anywhere else
3006
3341
                        this_tree_map[file_id] = None, this_path
3007
 
                    elif minikind == 'r':
 
3342
                    elif minikind == b'r':
3008
3343
                        # relocation, must occur at expected location
3009
3344
                        this_tree_map[file_id] = tree_state[1], this_path
3010
3345
                    else:
3014
3349
                raise AssertionError(
3015
3350
                    "entry %r has no data for any tree." % (entry,))
3016
3351
        if self._id_index is not None:
3017
 
            for file_id, entry_keys in self._id_index.iteritems():
 
3352
            for file_id, entry_keys in viewitems(self._id_index):
3018
3353
                for entry_key in entry_keys:
 
3354
                    # Check that the entry in the map is pointing to the same
 
3355
                    # file_id
3019
3356
                    if entry_key[2] != file_id:
3020
3357
                        raise AssertionError(
3021
3358
                            'file_id %r did not match entry key %s'
3022
3359
                            % (file_id, entry_key))
 
3360
                    # And that from this entry key, we can look up the original
 
3361
                    # record
 
3362
                    block_index, present = self._find_block_index_from_key(entry_key)
 
3363
                    if not present:
 
3364
                        raise AssertionError('missing block for entry key: %r', entry_key)
 
3365
                    entry_index, present = self._find_entry_index(entry_key, self._dirblocks[block_index][1])
 
3366
                    if not present:
 
3367
                        raise AssertionError('missing entry for key: %r', entry_key)
 
3368
                if len(entry_keys) != len(set(entry_keys)):
 
3369
                    raise AssertionError(
 
3370
                        'id_index contained non-unique data for %s'
 
3371
                        % (entry_keys,))
3023
3372
 
3024
3373
    def _wipe_state(self):
3025
3374
        """Forget all state information about the dirstate."""
3047
3396
        self._lock_state = 'r'
3048
3397
        self._state_file = self._lock_token.f
3049
3398
        self._wipe_state()
 
3399
        return lock.LogicalLockResult(self.unlock)
3050
3400
 
3051
3401
    def lock_write(self):
3052
3402
        """Acquire a write lock on the dirstate."""
3060
3410
        self._lock_state = 'w'
3061
3411
        self._state_file = self._lock_token.f
3062
3412
        self._wipe_state()
 
3413
        return lock.LogicalLockResult(self.unlock, self._lock_token)
3063
3414
 
3064
3415
    def unlock(self):
3065
3416
        """Drop any locks held on the dirstate."""
3082
3433
 
3083
3434
 
3084
3435
def py_update_entry(state, entry, abspath, stat_value,
3085
 
                 _stat_to_minikind=DirState._stat_to_minikind,
3086
 
                 _pack_stat=pack_stat):
 
3436
                    _stat_to_minikind=DirState._stat_to_minikind):
3087
3437
    """Update the entry based on what is actually on disk.
3088
3438
 
3089
3439
    This function only calculates the sha if it needs to - if the entry is
3098
3448
        target of a symlink.
3099
3449
    """
3100
3450
    try:
3101
 
        minikind = _stat_to_minikind[stat_value.st_mode & 0170000]
 
3451
        minikind = _stat_to_minikind[stat_value.st_mode & 0o170000]
3102
3452
    except KeyError:
3103
3453
        # Unhandled kind
3104
3454
        return None
3105
 
    packed_stat = _pack_stat(stat_value)
 
3455
    packed_stat = pack_stat(stat_value)
3106
3456
    (saved_minikind, saved_link_or_sha1, saved_file_size,
3107
3457
     saved_executable, saved_packed_stat) = entry[1][0]
 
3458
    if not isinstance(saved_minikind, bytes):
 
3459
        raise TypeError(saved_minikind)
3108
3460
 
3109
 
    if minikind == 'd' and saved_minikind == 't':
3110
 
        minikind = 't'
 
3461
    if minikind == b'd' and saved_minikind == b't':
 
3462
        minikind = b't'
3111
3463
    if (minikind == saved_minikind
3112
3464
        and packed_stat == saved_packed_stat):
3113
3465
        # The stat hasn't changed since we saved, so we can re-use the
3114
3466
        # saved sha hash.
3115
 
        if minikind == 'd':
 
3467
        if minikind == b'd':
3116
3468
            return None
3117
3469
 
3118
3470
        # size should also be in packed_stat
3122
3474
    # If we have gotten this far, that means that we need to actually
3123
3475
    # process this entry.
3124
3476
    link_or_sha1 = None
3125
 
    if minikind == 'f':
 
3477
    worth_saving = True
 
3478
    if minikind == b'f':
3126
3479
        executable = state._is_executable(stat_value.st_mode,
3127
3480
                                         saved_executable)
3128
3481
        if state._cutoff_time is None:
3130
3483
        if (stat_value.st_mtime < state._cutoff_time
3131
3484
            and stat_value.st_ctime < state._cutoff_time
3132
3485
            and len(entry[1]) > 1
3133
 
            and entry[1][1][0] != 'a'):
 
3486
            and entry[1][1][0] != b'a'):
3134
3487
            # Could check for size changes for further optimised
3135
3488
            # avoidance of sha1's. However the most prominent case of
3136
3489
            # over-shaing is during initial add, which this catches.
3138
3491
            # are calculated at the same time, so checking just the size
3139
3492
            # gains nothing w.r.t. performance.
3140
3493
            link_or_sha1 = state._sha1_file(abspath)
3141
 
            entry[1][0] = ('f', link_or_sha1, stat_value.st_size,
 
3494
            entry[1][0] = (b'f', link_or_sha1, stat_value.st_size,
3142
3495
                           executable, packed_stat)
3143
3496
        else:
3144
 
            entry[1][0] = ('f', '', stat_value.st_size,
 
3497
            entry[1][0] = (b'f', b'', stat_value.st_size,
3145
3498
                           executable, DirState.NULLSTAT)
3146
 
    elif minikind == 'd':
 
3499
            worth_saving = False
 
3500
    elif minikind == b'd':
3147
3501
        link_or_sha1 = None
3148
 
        entry[1][0] = ('d', '', 0, False, packed_stat)
3149
 
        if saved_minikind != 'd':
 
3502
        entry[1][0] = (b'd', b'', 0, False, packed_stat)
 
3503
        if saved_minikind != b'd':
3150
3504
            # This changed from something into a directory. Make sure we
3151
3505
            # have a directory block for it. This doesn't happen very
3152
3506
            # often, so this doesn't have to be super fast.
3154
3508
                state._get_block_entry_index(entry[0][0], entry[0][1], 0)
3155
3509
            state._ensure_block(block_index, entry_index,
3156
3510
                               osutils.pathjoin(entry[0][0], entry[0][1]))
3157
 
    elif minikind == 'l':
 
3511
        else:
 
3512
            worth_saving = False
 
3513
    elif minikind == b'l':
 
3514
        if saved_minikind == b'l':
 
3515
            worth_saving = False
3158
3516
        link_or_sha1 = state._read_link(abspath, saved_link_or_sha1)
3159
3517
        if state._cutoff_time is None:
3160
3518
            state._sha_cutoff_time()
3161
3519
        if (stat_value.st_mtime < state._cutoff_time
3162
3520
            and stat_value.st_ctime < state._cutoff_time):
3163
 
            entry[1][0] = ('l', link_or_sha1, stat_value.st_size,
 
3521
            entry[1][0] = (b'l', link_or_sha1, stat_value.st_size,
3164
3522
                           False, packed_stat)
3165
3523
        else:
3166
 
            entry[1][0] = ('l', '', stat_value.st_size,
 
3524
            entry[1][0] = (b'l', b'', stat_value.st_size,
3167
3525
                           False, DirState.NULLSTAT)
3168
 
    state._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
3526
    if worth_saving:
 
3527
        state._mark_modified([entry])
3169
3528
    return link_or_sha1
3170
3529
 
3171
3530
 
3184
3543
        self.old_dirname_to_file_id = {}
3185
3544
        self.new_dirname_to_file_id = {}
3186
3545
        # Are we doing a partial iter_changes?
3187
 
        self.partial = search_specific_files != set([''])
 
3546
        self.partial = search_specific_files != {''}
3188
3547
        # Using a list so that we can access the values and change them in
3189
3548
        # nested scope. Each one is [path, file_id, entry]
3190
3549
        self.last_source_parent = [None, None]
3236
3595
            source_details = DirState.NULL_PARENT_DETAILS
3237
3596
        else:
3238
3597
            source_details = entry[1][self.source_index]
 
3598
        # GZ 2017-06-09: Eck, more sets.
 
3599
        _fdltr = {b'f', b'd', b'l', b't', b'r'}
 
3600
        _fdlt = {b'f', b'd', b'l', b't'}
 
3601
        _ra = (b'r', b'a')
3239
3602
        target_details = entry[1][self.target_index]
3240
3603
        target_minikind = target_details[0]
3241
 
        if path_info is not None and target_minikind in 'fdlt':
 
3604
        if path_info is not None and target_minikind in _fdlt:
3242
3605
            if not (self.target_index == 0):
3243
3606
                raise AssertionError()
3244
3607
            link_or_sha1 = update_entry(self.state, entry,
3250
3613
            link_or_sha1 = None
3251
3614
        file_id = entry[0][2]
3252
3615
        source_minikind = source_details[0]
3253
 
        if source_minikind in 'fdltr' and target_minikind in 'fdlt':
 
3616
        if source_minikind in _fdltr and target_minikind in _fdlt:
3254
3617
            # claimed content in both: diff
3255
3618
            #   r    | fdlt   |      | add source to search, add id path move and perform
3256
3619
            #        |        |      | diff check on source-target
3257
3620
            #   r    | fdlt   |  a   | dangling file that was present in the basis.
3258
3621
            #        |        |      | ???
3259
 
            if source_minikind in 'r':
 
3622
            if source_minikind == b'r':
3260
3623
                # add the source to the search path to find any children it
3261
3624
                # has.  TODO ? : only add if it is a container ?
3262
 
                if not osutils.is_inside_any(self.searched_specific_files,
 
3625
                if not is_inside_any(self.searched_specific_files,
3263
3626
                                             source_details[1]):
3264
3627
                    self.search_specific_files.add(source_details[1])
3265
3628
                # generate the old path; this is needed for stating later
3272
3635
                # update the source details variable to be the real
3273
3636
                # location.
3274
3637
                if old_entry == (None, None):
3275
 
                    raise errors.CorruptDirstate(self.state._filename,
 
3638
                    raise DirstateCorrupt(self.state._filename,
3276
3639
                        "entry '%s/%s' is considered renamed from %r"
3277
3640
                        " but source does not exist\n"
3278
3641
                        "entry: %s" % (entry[0][0], entry[0][1], old_path, entry))
3294
3657
                    if path is None:
3295
3658
                        old_path = path = pathjoin(old_dirname, old_basename)
3296
3659
                    self.new_dirname_to_file_id[path] = file_id
3297
 
                    if source_minikind != 'd':
 
3660
                    if source_minikind != b'd':
3298
3661
                        content_change = True
3299
3662
                    else:
3300
3663
                        # directories have no fingerprint
3301
3664
                        content_change = False
3302
3665
                    target_exec = False
3303
3666
                elif target_kind == 'file':
3304
 
                    if source_minikind != 'f':
 
3667
                    if source_minikind != b'f':
3305
3668
                        content_change = True
3306
3669
                    else:
3307
3670
                        # Check the sha. We can't just rely on the size as
3323
3686
                    else:
3324
3687
                        target_exec = target_details[3]
3325
3688
                elif target_kind == 'symlink':
3326
 
                    if source_minikind != 'l':
 
3689
                    if source_minikind != b'l':
3327
3690
                        content_change = True
3328
3691
                    else:
3329
3692
                        content_change = (link_or_sha1 != source_details[1])
3330
3693
                    target_exec = False
3331
3694
                elif target_kind == 'tree-reference':
3332
 
                    if source_minikind != 't':
 
3695
                    if source_minikind != b't':
3333
3696
                        content_change = True
3334
3697
                    else:
3335
3698
                        content_change = False
3338
3701
                    if path is None:
3339
3702
                        path = pathjoin(old_dirname, old_basename)
3340
3703
                    raise errors.BadFileKindError(path, path_info[2])
3341
 
            if source_minikind == 'd':
 
3704
            if source_minikind == b'd':
3342
3705
                if path is None:
3343
3706
                    old_path = path = pathjoin(old_dirname, old_basename)
3344
3707
                self.old_dirname_to_file_id[old_path] = file_id
3409
3772
                       (self.utf8_decode(old_basename)[0], self.utf8_decode(entry[0][1])[0]),
3410
3773
                       (source_kind, target_kind),
3411
3774
                       (source_exec, target_exec)), changed
3412
 
        elif source_minikind in 'a' and target_minikind in 'fdlt':
 
3775
        elif source_minikind in b'a' and target_minikind in _fdlt:
3413
3776
            # looks like a new file
3414
3777
            path = pathjoin(entry[0][0], entry[0][1])
3415
3778
            # parent id is the entry for the path in the target tree
3446
3809
                       (None, self.utf8_decode(entry[0][1])[0]),
3447
3810
                       (None, None),
3448
3811
                       (None, False)), True
3449
 
        elif source_minikind in 'fdlt' and target_minikind in 'a':
 
3812
        elif source_minikind in _fdlt and target_minikind in b'a':
3450
3813
            # unversioned, possibly, or possibly not deleted: we dont care.
3451
3814
            # if its still on disk, *and* theres no other entry at this
3452
3815
            # path [we dont know this in this routine at the moment -
3464
3827
                   (self.utf8_decode(entry[0][1])[0], None),
3465
3828
                   (DirState._minikind_to_kind[source_minikind], None),
3466
3829
                   (source_details[3], None)), True
3467
 
        elif source_minikind in 'fdlt' and target_minikind in 'r':
 
3830
        elif source_minikind in _fdlt and target_minikind in b'r':
3468
3831
            # a rename; could be a true rename, or a rename inherited from
3469
3832
            # a renamed parent. TODO: handle this efficiently. Its not
3470
3833
            # common case to rename dirs though, so a correct but slow
3471
3834
            # implementation will do.
3472
 
            if not osutils.is_inside_any(self.searched_specific_files, target_details[1]):
 
3835
            if not is_inside_any(self.searched_specific_files, target_details[1]):
3473
3836
                self.search_specific_files.add(target_details[1])
3474
 
        elif source_minikind in 'ra' and target_minikind in 'ra':
 
3837
        elif source_minikind in _ra and target_minikind in _ra:
3475
3838
            # neither of the selected trees contain this file,
3476
3839
            # so skip over it. This is not currently directly tested, but
3477
3840
            # is indirectly via test_too_much.TestCommands.test_conflicts.
3480
3843
            raise AssertionError("don't know how to compare "
3481
3844
                "source_minikind=%r, target_minikind=%r"
3482
3845
                % (source_minikind, target_minikind))
3483
 
            ## import pdb;pdb.set_trace()
3484
3846
        return None, None
3485
3847
 
3486
3848
    def __iter__(self):
3500
3862
        if new_path:
3501
3863
            # Not the root and not a delete: queue up the parents of the path.
3502
3864
            self.search_specific_file_parents.update(
3503
 
                osutils.parent_directories(new_path.encode('utf8')))
 
3865
                p.encode('utf8') for p in osutils.parent_directories(new_path))
3504
3866
            # Add the root directory which parent_directories does not
3505
3867
            # provide.
3506
 
            self.search_specific_file_parents.add('')
 
3868
            self.search_specific_file_parents.add(b'')
3507
3869
 
3508
3870
    def iter_changes(self):
3509
3871
        """Iterate over the changes."""
3510
3872
        utf8_decode = cache_utf8._utf8_decode
3511
 
        _cmp_by_dirs = cmp_by_dirs
 
3873
        _lt_by_dirs = lt_by_dirs
3512
3874
        _process_entry = self._process_entry
3513
3875
        search_specific_files = self.search_specific_files
3514
3876
        searched_specific_files = self.searched_specific_files
3563
3925
            root_abspath = self.tree.abspath(current_root_unicode)
3564
3926
            try:
3565
3927
                root_stat = os.lstat(root_abspath)
3566
 
            except OSError, e:
 
3928
            except OSError as e:
3567
3929
                if e.errno == errno.ENOENT:
3568
3930
                    # the path does not exist: let _process_entry know that.
3569
3931
                    root_dir_info = None
3571
3933
                    # some other random error: hand it up.
3572
3934
                    raise
3573
3935
            else:
3574
 
                root_dir_info = ('', current_root,
 
3936
                root_dir_info = (b'', current_root,
3575
3937
                    osutils.file_kind_from_stat_mode(root_stat.st_mode), root_stat,
3576
3938
                    root_abspath)
3577
3939
                if root_dir_info[2] == 'directory':
3605
3967
                       (None, root_dir_info[2]),
3606
3968
                       (None, new_executable)
3607
3969
                      )
3608
 
            initial_key = (current_root, '', '')
 
3970
            initial_key = (current_root, b'', b'')
3609
3971
            block_index, _ = self.state._find_block_index_from_key(initial_key)
3610
3972
            if block_index == 0:
3611
3973
                # we have processed the total root already, but because the
3616
3978
            else:
3617
3979
                dir_iterator = osutils._walkdirs_utf8(root_abspath, prefix=current_root)
3618
3980
                try:
3619
 
                    current_dir_info = dir_iterator.next()
3620
 
                except OSError, e:
 
3981
                    current_dir_info = next(dir_iterator)
 
3982
                except OSError as e:
3621
3983
                    # on win32, python2.4 has e.errno == ERROR_DIRECTORY, but
3622
3984
                    # python 2.5 has e.errno == EINVAL,
3623
3985
                    #            and e.winerror == ERROR_DIRECTORY
3635
3997
                    else:
3636
3998
                        raise
3637
3999
                else:
3638
 
                    if current_dir_info[0][0] == '':
 
4000
                    if current_dir_info[0][0] == b'':
3639
4001
                        # 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':
 
4002
                        bzr_index = bisect.bisect_left(current_dir_info[1], (b'.bzr',))
 
4003
                        if current_dir_info[1][bzr_index][0] != b'.bzr':
3642
4004
                            raise AssertionError()
3643
4005
                        del current_dir_info[1][bzr_index]
3644
4006
            # walk until both the directory listing and the versioned metadata
3645
4007
            # are exhausted.
3646
4008
            if (block_index < len(self.state._dirblocks) and
3647
 
                osutils.is_inside(current_root, self.state._dirblocks[block_index][0])):
 
4009
                is_inside(current_root, self.state._dirblocks[block_index][0])):
3648
4010
                current_block = self.state._dirblocks[block_index]
3649
4011
            else:
3650
4012
                current_block = None
3652
4014
                   current_block is not None):
3653
4015
                if (current_dir_info and current_block
3654
4016
                    and current_dir_info[0][0] != current_block[0]):
3655
 
                    if _cmp_by_dirs(current_dir_info[0][0], current_block[0]) < 0:
 
4017
                    if _lt_by_dirs(current_dir_info[0][0], current_block[0]):
3656
4018
                        # filesystem data refers to paths not covered by the dirblock.
3657
4019
                        # this has two possibilities:
3658
4020
                        # A) it is versioned but empty, so there is no block for it
3692
4054
 
3693
4055
                        # This dir info has been handled, go to the next
3694
4056
                        try:
3695
 
                            current_dir_info = dir_iterator.next()
 
4057
                            current_dir_info = next(dir_iterator)
3696
4058
                        except StopIteration:
3697
4059
                            current_dir_info = None
3698
4060
                    else:
3714
4076
                                    yield result
3715
4077
                        block_index +=1
3716
4078
                        if (block_index < len(self.state._dirblocks) and
3717
 
                            osutils.is_inside(current_root,
 
4079
                            is_inside(current_root,
3718
4080
                                              self.state._dirblocks[block_index][0])):
3719
4081
                            current_block = self.state._dirblocks[block_index]
3720
4082
                        else:
3753
4115
                            if changed or self.include_unchanged:
3754
4116
                                yield result
3755
4117
                    elif (current_entry[0][1] != current_path_info[1]
3756
 
                          or current_entry[1][self.target_index][0] in 'ar'):
 
4118
                          or current_entry[1][self.target_index][0] in (b'a', b'r')):
3757
4119
                        # The current path on disk doesn't match the dirblock
3758
4120
                        # record. Either the dirblock is marked as absent, or
3759
4121
                        # the file on disk is not present at all in the
3838
4200
                if current_block is not None:
3839
4201
                    block_index += 1
3840
4202
                    if (block_index < len(self.state._dirblocks) and
3841
 
                        osutils.is_inside(current_root, self.state._dirblocks[block_index][0])):
 
4203
                        is_inside(current_root, self.state._dirblocks[block_index][0])):
3842
4204
                        current_block = self.state._dirblocks[block_index]
3843
4205
                    else:
3844
4206
                        current_block = None
3845
4207
                if current_dir_info is not None:
3846
4208
                    try:
3847
 
                        current_dir_info = dir_iterator.next()
 
4209
                        current_dir_info = next(dir_iterator)
3848
4210
                    except StopIteration:
3849
4211
                        current_dir_info = None
3850
4212
        for result in self._iter_specific_file_parents():
3857
4219
            # Even in extremely large trees this should be modest, so currently
3858
4220
            # no attempt is made to optimise.
3859
4221
            path_utf8 = self.search_specific_file_parents.pop()
3860
 
            if osutils.is_inside_any(self.searched_specific_files, path_utf8):
 
4222
            if is_inside_any(self.searched_specific_files, path_utf8):
3861
4223
                # We've examined this path.
3862
4224
                continue
3863
4225
            if path_utf8 in self.searched_exact_paths:
3872
4234
            found_item = False
3873
4235
            for candidate_entry in path_entries:
3874
4236
                # Find entries present in target at this path:
3875
 
                if candidate_entry[1][self.target_index][0] not in 'ar':
 
4237
                if candidate_entry[1][self.target_index][0] not in (b'a', b'r'):
3876
4238
                    found_item = True
3877
4239
                    selected_entries.append(candidate_entry)
3878
4240
                # Find entries present in source at this path:
3879
4241
                elif (self.source_index is not None and
3880
 
                    candidate_entry[1][self.source_index][0] not in 'ar'):
 
4242
                    candidate_entry[1][self.source_index][0] not in (b'a', b'r')):
3881
4243
                    found_item = True
3882
 
                    if candidate_entry[1][self.target_index][0] == 'a':
 
4244
                    if candidate_entry[1][self.target_index][0] == b'a':
3883
4245
                        # Deleted, emit it here.
3884
4246
                        selected_entries.append(candidate_entry)
3885
4247
                    else:
3909
4271
                        result[6][1] != 'directory'):
3910
4272
                        # This stopped being a directory, the old children have
3911
4273
                        # to be included.
3912
 
                        if entry[1][self.source_index][0] == 'r':
 
4274
                        if entry[1][self.source_index][0] == b'r':
3913
4275
                            # renamed, take the source path
3914
4276
                            entry_path_utf8 = entry[1][self.source_index][1]
3915
4277
                        else:
3916
4278
                            entry_path_utf8 = path_utf8
3917
 
                        initial_key = (entry_path_utf8, '', '')
 
4279
                        initial_key = (entry_path_utf8, b'', b'')
3918
4280
                        block_index, _ = self.state._find_block_index_from_key(
3919
4281
                            initial_key)
3920
4282
                        if block_index == 0:
3923
4285
                        current_block = None
3924
4286
                        if block_index < len(self.state._dirblocks):
3925
4287
                            current_block = self.state._dirblocks[block_index]
3926
 
                            if not osutils.is_inside(
 
4288
                            if not is_inside(
3927
4289
                                entry_path_utf8, current_block[0]):
3928
4290
                                # No entries for this directory at all.
3929
4291
                                current_block = None
3930
4292
                        if current_block is not None:
3931
4293
                            for entry in current_block[1]:
3932
 
                                if entry[1][self.source_index][0] in 'ar':
 
4294
                                if entry[1][self.source_index][0] in (b'a', b'r'):
3933
4295
                                    # Not in the source tree, so doesn't have to be
3934
4296
                                    # included.
3935
4297
                                    continue
3949
4311
        abspath = self.tree.abspath(unicode_path)
3950
4312
        try:
3951
4313
            stat = os.lstat(abspath)
3952
 
        except OSError, e:
 
4314
        except OSError as e:
3953
4315
            if e.errno == errno.ENOENT:
3954
4316
                # the path does not exist.
3955
4317
                return None
3956
4318
            else:
3957
4319
                raise
3958
 
        utf8_basename = utf8_path.rsplit('/', 1)[-1]
 
4320
        utf8_basename = utf8_path.rsplit(b'/', 1)[-1]
3959
4321
        dir_info = (utf8_path, utf8_basename,
3960
4322
            osutils.file_kind_from_stat_mode(stat.st_mode), stat,
3961
4323
            abspath)
3969
4331
 
3970
4332
# Try to load the compiled form if possible
3971
4333
try:
3972
 
    from bzrlib._dirstate_helpers_pyx import (
 
4334
    from ._dirstate_helpers_pyx import (
3973
4335
        _read_dirblocks,
3974
4336
        bisect_dirblock,
3975
4337
        _bisect_path_left,
3976
4338
        _bisect_path_right,
3977
 
        cmp_by_dirs,
 
4339
        lt_by_dirs,
 
4340
        pack_stat,
3978
4341
        ProcessEntryC as _process_entry,
3979
4342
        update_entry as update_entry,
3980
4343
        )
3981
 
except ImportError, e:
 
4344
except ImportError as e:
3982
4345
    osutils.failed_to_load_extension(e)
3983
 
    from bzrlib._dirstate_helpers_py import (
 
4346
    from ._dirstate_helpers_py import (
3984
4347
        _read_dirblocks,
3985
4348
        bisect_dirblock,
3986
4349
        _bisect_path_left,
3987
4350
        _bisect_path_right,
3988
 
        cmp_by_dirs,
 
4351
        lt_by_dirs,
 
4352
        pack_stat,
3989
4353
        )
3990
4354
    # FIXME: It would be nice to be able to track moved lines so that the
3991
4355
    # corresponding python code can be moved to the _dirstate_helpers_py