/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 bzrlib/mutabletree.py

  • Committer: Martin Pool
  • Date: 2008-05-08 04:12:06 UTC
  • mto: This revision was merged to the branch mainline in revision 3415.
  • Revision ID: mbp@sourcefrog.net-20080508041206-tkrr8ucmcyrlzkum
Some review cleanups for assertion removal

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2006, 2007 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""MutableTree object.
 
18
 
 
19
See MutableTree for more details.
 
20
"""
 
21
 
 
22
 
 
23
from bzrlib.lazy_import import lazy_import
 
24
lazy_import(globals(), """
 
25
import os
 
26
 
 
27
from bzrlib import (
 
28
    add,
 
29
    bzrdir,
 
30
    hooks,
 
31
    )
 
32
from bzrlib.osutils import dirname
 
33
from bzrlib.revisiontree import RevisionTree
 
34
from bzrlib.trace import mutter, warning
 
35
""")
 
36
 
 
37
from bzrlib import (
 
38
    errors,
 
39
    osutils,
 
40
    tree,
 
41
    )
 
42
from bzrlib.decorators import needs_read_lock, needs_write_lock
 
43
from bzrlib.osutils import splitpath
 
44
from bzrlib.symbol_versioning import DEPRECATED_PARAMETER
 
45
 
 
46
 
 
47
def needs_tree_write_lock(unbound):
 
48
    """Decorate unbound to take out and release a tree_write lock."""
 
49
    def tree_write_locked(self, *args, **kwargs):
 
50
        self.lock_tree_write()
 
51
        try:
 
52
            return unbound(self, *args, **kwargs)
 
53
        finally:
 
54
            self.unlock()
 
55
    tree_write_locked.__doc__ = unbound.__doc__
 
56
    tree_write_locked.__name__ = unbound.__name__
 
57
    return tree_write_locked
 
58
 
 
59
 
 
60
class MutableTree(tree.Tree):
 
61
    """A MutableTree is a specialisation of Tree which is able to be mutated.
 
62
 
 
63
    Generally speaking these mutations are only possible within a lock_write
 
64
    context, and will revert if the lock is broken abnormally - but this cannot
 
65
    be guaranteed - depending on the exact implementation of the mutable state.
 
66
 
 
67
    The most common form of Mutable Tree is WorkingTree, see bzrlib.workingtree.
 
68
    For tests we also have MemoryTree which is a MutableTree whose contents are
 
69
    entirely in memory.
 
70
 
 
71
    For now, we are not treating MutableTree as an interface to provide
 
72
    conformance tests for - rather we are testing MemoryTree specifically, and 
 
73
    interface testing implementations of WorkingTree.
 
74
 
 
75
    A mutable tree always has an associated Branch and BzrDir object - the
 
76
    branch and bzrdir attributes.
 
77
    """
 
78
 
 
79
    @needs_tree_write_lock
 
80
    def add(self, files, ids=None, kinds=None):
 
81
        """Add paths to the set of versioned paths.
 
82
 
 
83
        Note that the command line normally calls smart_add instead,
 
84
        which can automatically recurse.
 
85
 
 
86
        This adds the files to the inventory, so that they will be
 
87
        recorded by the next commit.
 
88
 
 
89
        :param files: List of paths to add, relative to the base of the tree.
 
90
        :param ids: If set, use these instead of automatically generated ids.
 
91
            Must be the same length as the list of files, but may
 
92
            contain None for ids that are to be autogenerated.
 
93
        :param kinds: Optional parameter to specify the kinds to be used for
 
94
            each file.
 
95
 
 
96
        TODO: Perhaps callback with the ids and paths as they're added.
 
97
        """
 
98
        if isinstance(files, basestring):
 
99
            # XXX: Passing a single string is inconsistent and should be
 
100
            # deprecated.
 
101
            if not (ids is None or isinstance(ids, basestring)):
 
102
                raise AssertionError()
 
103
            if not (kinds is None or isinstance(kinds, basestring)):
 
104
                raise AssertionError()
 
105
            files = [files]
 
106
            if ids is not None:
 
107
                ids = [ids]
 
108
            if kinds is not None:
 
109
                kinds = [kinds]
 
110
 
 
111
        files = [path.strip('/') for path in files]
 
112
 
 
113
        if ids is None:
 
114
            ids = [None] * len(files)
 
115
        else:
 
116
            if not (len(ids) == len(files)):
 
117
                raise AssertionError()
 
118
        if kinds is None:
 
119
            kinds = [None] * len(files)
 
120
        elif not len(kinds) == len(files):
 
121
            raise AssertionError()
 
122
        for f in files:
 
123
            # generic constraint checks:
 
124
            if self.is_control_filename(f):
 
125
                raise errors.ForbiddenControlFileError(filename=f)
 
126
            fp = splitpath(f)
 
127
        # fill out file kinds for all files [not needed when we stop 
 
128
        # caring about the instantaneous file kind within a uncommmitted tree
 
129
        #
 
130
        self._gather_kinds(files, kinds)
 
131
        self._add(files, ids, kinds)
 
132
 
 
133
    def add_reference(self, sub_tree):
 
134
        """Add a TreeReference to the tree, pointing at sub_tree"""
 
135
        raise errors.UnsupportedOperation(self.add_reference, self)
 
136
 
 
137
    def _add_reference(self, sub_tree):
 
138
        """Standard add_reference implementation, for use by subclasses"""
 
139
        try:
 
140
            sub_tree_path = self.relpath(sub_tree.basedir)
 
141
        except errors.PathNotChild:
 
142
            raise errors.BadReferenceTarget(self, sub_tree,
 
143
                                            'Target not inside tree.')
 
144
        sub_tree_id = sub_tree.get_root_id()
 
145
        if sub_tree_id == self.get_root_id():
 
146
            raise errors.BadReferenceTarget(self, sub_tree,
 
147
                                     'Trees have the same root id.')
 
148
        if sub_tree_id in self.inventory:
 
149
            raise errors.BadReferenceTarget(self, sub_tree,
 
150
                                            'Root id already present in tree')
 
151
        self._add([sub_tree_path], [sub_tree_id], ['tree-reference'])
 
152
 
 
153
    def _add(self, files, ids, kinds):
 
154
        """Helper function for add - updates the inventory.
 
155
 
 
156
        :param files: sequence of pathnames, relative to the tree root
 
157
        :param ids: sequence of suggested ids for the files (may be None)
 
158
        :param kinds: sequence of  inventory kinds of the files (i.e. may
 
159
            contain "tree-reference")
 
160
        """
 
161
        raise NotImplementedError(self._add)
 
162
 
 
163
    @needs_tree_write_lock
 
164
    def apply_inventory_delta(self, changes):
 
165
        """Apply changes to the inventory as an atomic operation.
 
166
 
 
167
        :param changes: An inventory delta to apply to the working tree's
 
168
            inventory.
 
169
        :return None:
 
170
        :seealso Inventory.apply_delta: For details on the changes parameter.
 
171
        """
 
172
        self.flush()
 
173
        inv = self.inventory
 
174
        inv.apply_delta(changes)
 
175
        self._write_inventory(inv)
 
176
 
 
177
    @needs_write_lock
 
178
    def commit(self, message=None, revprops=None, *args,
 
179
               **kwargs):
 
180
        # avoid circular imports
 
181
        from bzrlib import commit
 
182
        if revprops is None:
 
183
            revprops = {}
 
184
        if not 'branch-nick' in revprops:
 
185
            revprops['branch-nick'] = self.branch.nick
 
186
        author = kwargs.pop('author', None)
 
187
        if author is not None:
 
188
            if 'author' in revprops:
 
189
                # XXX: maybe we should just accept one of them?
 
190
                raise AssertionError('author property given twice')
 
191
            revprops['author'] = author
 
192
        # args for wt.commit start at message from the Commit.commit method,
 
193
        args = (message, ) + args
 
194
        for hook in MutableTree.hooks['start_commit']:
 
195
            hook(self)
 
196
        committed_id = commit.Commit().commit(working_tree=self,
 
197
            revprops=revprops, *args, **kwargs)
 
198
        return committed_id
 
199
 
 
200
    def _gather_kinds(self, files, kinds):
 
201
        """Helper function for add - sets the entries of kinds."""
 
202
        raise NotImplementedError(self._gather_kinds)
 
203
 
 
204
    @needs_read_lock
 
205
    def last_revision(self):
 
206
        """Return the revision id of the last commit performed in this tree.
 
207
 
 
208
        In early tree formats the result of last_revision is the same as the
 
209
        branch last_revision, but that is no longer the case for modern tree
 
210
        formats.
 
211
        
 
212
        last_revision returns the left most parent id, or None if there are no
 
213
        parents.
 
214
 
 
215
        last_revision was deprecated as of 0.11. Please use get_parent_ids
 
216
        instead.
 
217
        """
 
218
        raise NotImplementedError(self.last_revision)
 
219
 
 
220
    def lock_tree_write(self):
 
221
        """Lock the working tree for write, and the branch for read.
 
222
 
 
223
        This is useful for operations which only need to mutate the working
 
224
        tree. Taking out branch write locks is a relatively expensive process
 
225
        and may fail if the branch is on read only media. So branch write locks
 
226
        should only be taken out when we are modifying branch data - such as in
 
227
        operations like commit, pull, uncommit and update.
 
228
        """
 
229
        raise NotImplementedError(self.lock_tree_write)
 
230
 
 
231
    def lock_write(self):
 
232
        """Lock the tree and its branch. This allows mutating calls to be made.
 
233
 
 
234
        Some mutating methods will take out implicit write locks, but in 
 
235
        general you should always obtain a write lock before calling mutating
 
236
        methods on a tree.
 
237
        """
 
238
        raise NotImplementedError(self.lock_write)
 
239
 
 
240
    @needs_write_lock
 
241
    def mkdir(self, path, file_id=None):
 
242
        """Create a directory in the tree. if file_id is None, one is assigned.
 
243
 
 
244
        :param path: A unicode file path.
 
245
        :param file_id: An optional file-id.
 
246
        :return: the file id of the new directory.
 
247
        """
 
248
        raise NotImplementedError(self.mkdir)
 
249
 
 
250
    def set_parent_ids(self, revision_ids, allow_leftmost_as_ghost=False):
 
251
        """Set the parents ids of the working tree.
 
252
 
 
253
        :param revision_ids: A list of revision_ids.
 
254
        """
 
255
        raise NotImplementedError(self.set_parent_ids)
 
256
 
 
257
    def set_parent_trees(self, parents_list, allow_leftmost_as_ghost=False):
 
258
        """Set the parents of the working tree.
 
259
 
 
260
        :param parents_list: A list of (revision_id, tree) tuples. 
 
261
            If tree is None, then that element is treated as an unreachable
 
262
            parent tree - i.e. a ghost.
 
263
        """
 
264
        raise NotImplementedError(self.set_parent_trees)
 
265
 
 
266
    @needs_tree_write_lock
 
267
    def smart_add(self, file_list, recurse=True, action=None, save=True):
 
268
        """Version file_list, optionally recursing into directories.
 
269
 
 
270
        This is designed more towards DWIM for humans than API clarity.
 
271
        For the specific behaviour see the help for cmd_add().
 
272
 
 
273
        :param action: A reporter to be called with the inventory, parent_ie,
 
274
            path and kind of the path being added. It may return a file_id if 
 
275
            a specific one should be used.
 
276
        :param save: Save the inventory after completing the adds. If False
 
277
            this provides dry-run functionality by doing the add and not saving
 
278
            the inventory.
 
279
        :return: A tuple - files_added, ignored_files. files_added is the count
 
280
            of added files, and ignored_files is a dict mapping files that were
 
281
            ignored to the rule that caused them to be ignored.
 
282
        """
 
283
        # not in an inner loop; and we want to remove direct use of this,
 
284
        # so here as a reminder for now. RBC 20070703
 
285
        from bzrlib.inventory import InventoryEntry
 
286
        if action is None:
 
287
            action = add.AddAction()
 
288
        
 
289
        if not file_list:
 
290
            # no paths supplied: add the entire tree.
 
291
            file_list = [u'.']
 
292
        # mutter("smart add of %r")
 
293
        inv = self.inventory
 
294
        added = []
 
295
        ignored = {}
 
296
        dirs_to_add = []
 
297
        user_dirs = set()
 
298
 
 
299
        # validate user file paths and convert all paths to tree 
 
300
        # relative : it's cheaper to make a tree relative path an abspath
 
301
        # than to convert an abspath to tree relative.
 
302
        for filepath in file_list:
 
303
            rf = _FastPath(self.relpath(filepath))
 
304
            # validate user parameters. Our recursive code avoids adding new files
 
305
            # that need such validation 
 
306
            if self.is_control_filename(rf.raw_path):
 
307
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
 
308
            
 
309
            abspath = self.abspath(rf.raw_path)
 
310
            kind = osutils.file_kind(abspath)
 
311
            if kind == 'directory':
 
312
                # schedule the dir for scanning
 
313
                user_dirs.add(rf)
 
314
            else:
 
315
                if not InventoryEntry.versionable_kind(kind):
 
316
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
 
317
            # ensure the named path is added, so that ignore rules in the later directory
 
318
            # walk dont skip it.
 
319
            # we dont have a parent ie known yet.: use the relatively slower inventory 
 
320
            # probing method
 
321
            versioned = inv.has_filename(rf.raw_path)
 
322
            if versioned:
 
323
                continue
 
324
            added.extend(_add_one_and_parent(self, inv, None, rf, kind, action))
 
325
 
 
326
        if not recurse:
 
327
            # no need to walk any directories at all.
 
328
            if len(added) > 0 and save:
 
329
                self._write_inventory(inv)
 
330
            return added, ignored
 
331
 
 
332
        # only walk the minimal parents needed: we have user_dirs to override
 
333
        # ignores.
 
334
        prev_dir = None
 
335
 
 
336
        is_inside = osutils.is_inside_or_parent_of_any
 
337
        for path in sorted(user_dirs):
 
338
            if (prev_dir is None or not is_inside([prev_dir], path.raw_path)):
 
339
                dirs_to_add.append((path, None))
 
340
            prev_dir = path.raw_path
 
341
 
 
342
        # dirs_to_add is initialised to a list of directories, but as we scan
 
343
        # directories we append files to it.
 
344
        # XXX: We should determine kind of files when we scan them rather than
 
345
        # adding to this list. RBC 20070703
 
346
        for directory, parent_ie in dirs_to_add:
 
347
            # directory is tree-relative
 
348
            abspath = self.abspath(directory.raw_path)
 
349
 
 
350
            # get the contents of this directory.
 
351
 
 
352
            # find the kind of the path being added.
 
353
            kind = osutils.file_kind(abspath)
 
354
 
 
355
            if not InventoryEntry.versionable_kind(kind):
 
356
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
 
357
                continue
 
358
 
 
359
            if parent_ie is not None:
 
360
                versioned = directory.base_path in parent_ie.children
 
361
            else:
 
362
                # without the parent ie, use the relatively slower inventory 
 
363
                # probing method
 
364
                versioned = inv.has_filename(directory.raw_path)
 
365
 
 
366
            if kind == 'directory':
 
367
                try:
 
368
                    sub_branch = bzrdir.BzrDir.open(abspath)
 
369
                    sub_tree = True
 
370
                except errors.NotBranchError:
 
371
                    sub_tree = False
 
372
                except errors.UnsupportedFormatError:
 
373
                    sub_tree = True
 
374
            else:
 
375
                sub_tree = False
 
376
 
 
377
            if directory.raw_path == '':
 
378
                # mutter("tree root doesn't need to be added")
 
379
                sub_tree = False
 
380
            elif versioned:
 
381
                pass
 
382
                # mutter("%r is already versioned", abspath)
 
383
            elif sub_tree:
 
384
                # XXX: This is wrong; people *might* reasonably be trying to add
 
385
                # subtrees as subtrees.  This should probably only be done in formats 
 
386
                # which can represent subtrees, and even then perhaps only when
 
387
                # the user asked to add subtrees.  At the moment you can add them
 
388
                # specially through 'join --reference', which is perhaps
 
389
                # reasonable: adding a new reference is a special operation and
 
390
                # can have a special behaviour.  mbp 20070306
 
391
                mutter("%r is a nested bzr tree", abspath)
 
392
            else:
 
393
                _add_one(self, inv, parent_ie, directory, kind, action)
 
394
                added.append(directory.raw_path)
 
395
 
 
396
            if kind == 'directory' and not sub_tree:
 
397
                if parent_ie is not None:
 
398
                    # must be present:
 
399
                    this_ie = parent_ie.children[directory.base_path]
 
400
                else:
 
401
                    # without the parent ie, use the relatively slower inventory 
 
402
                    # probing method
 
403
                    this_id = inv.path2id(directory.raw_path)
 
404
                    if this_id is None:
 
405
                        this_ie = None
 
406
                    else:
 
407
                        this_ie = inv[this_id]
 
408
 
 
409
                for subf in sorted(os.listdir(abspath)):
 
410
                    # here we could use TreeDirectory rather than 
 
411
                    # string concatenation.
 
412
                    subp = osutils.pathjoin(directory.raw_path, subf)
 
413
                    # TODO: is_control_filename is very slow. Make it faster. 
 
414
                    # TreeDirectory.is_control_filename could also make this 
 
415
                    # faster - its impossible for a non root dir to have a 
 
416
                    # control file.
 
417
                    if self.is_control_filename(subp):
 
418
                        mutter("skip control directory %r", subp)
 
419
                    elif subf in this_ie.children:
 
420
                        # recurse into this already versioned subdir.
 
421
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
422
                    else:
 
423
                        # user selection overrides ignoes
 
424
                        # ignore while selecting files - if we globbed in the
 
425
                        # outer loop we would ignore user files.
 
426
                        ignore_glob = self.is_ignored(subp)
 
427
                        if ignore_glob is not None:
 
428
                            # mutter("skip ignored sub-file %r", subp)
 
429
                            ignored.setdefault(ignore_glob, []).append(subp)
 
430
                        else:
 
431
                            #mutter("queue to add sub-file %r", subp)
 
432
                            dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
433
 
 
434
        if len(added) > 0:
 
435
            if save:
 
436
                self._write_inventory(inv)
 
437
            else:
 
438
                self.read_working_inventory()
 
439
        return added, ignored
 
440
 
 
441
    def update_basis_by_delta(self, new_revid, delta):
 
442
        """Update the parents of this tree after a commit.
 
443
 
 
444
        This gives the tree one parent, with revision id new_revid. The
 
445
        inventory delta is applied to the current basis tree to generate the
 
446
        inventory for the parent new_revid, and all other parent trees are
 
447
        discarded.
 
448
 
 
449
        All the changes in the delta should be changes synchronising the basis
 
450
        tree with some or all of the working tree, with a change to a directory
 
451
        requiring that its contents have been recursively included. That is,
 
452
        this is not a general purpose tree modification routine, but a helper
 
453
        for commit which is not required to handle situations that do not arise
 
454
        outside of commit.
 
455
 
 
456
        :param new_revid: The new revision id for the trees parent.
 
457
        :param delta: An inventory delta (see apply_inventory_delta) describing
 
458
            the changes from the current left most parent revision to new_revid.
 
459
        """
 
460
        # if the tree is updated by a pull to the branch, as happens in
 
461
        # WorkingTree2, when there was no separation between branch and tree,
 
462
        # then just clear merges, efficiency is not a concern for now as this
 
463
        # is legacy environments only, and they are slow regardless.
 
464
        if self.last_revision() == new_revid:
 
465
            self.set_parent_ids([new_revid])
 
466
            return
 
467
        # generic implementation based on Inventory manipulation. See
 
468
        # WorkingTree classes for optimised versions for specific format trees.
 
469
        basis = self.basis_tree()
 
470
        basis.lock_read()
 
471
        inventory = basis.inventory
 
472
        basis.unlock()
 
473
        inventory.apply_delta(delta)
 
474
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
 
475
        self.set_parent_trees([(new_revid, rev_tree)])
 
476
 
 
477
 
 
478
class MutableTreeHooks(hooks.Hooks):
 
479
    """A dictionary mapping a hook name to a list of callables for mutabletree 
 
480
    hooks.
 
481
    """
 
482
 
 
483
    def __init__(self):
 
484
        """Create the default hooks.
 
485
 
 
486
        """
 
487
        hooks.Hooks.__init__(self)
 
488
        # Invoked before a commit is done in a tree. New in 1.4
 
489
        self['start_commit'] = []
 
490
 
 
491
 
 
492
# install the default hooks into the MutableTree class.
 
493
MutableTree.hooks = MutableTreeHooks()
 
494
 
 
495
 
 
496
class _FastPath(object):
 
497
    """A path object with fast accessors for things like basename."""
 
498
 
 
499
    __slots__ = ['raw_path', 'base_path']
 
500
 
 
501
    def __init__(self, path, base_path=None):
 
502
        """Construct a FastPath from path."""
 
503
        if base_path is None:
 
504
            self.base_path = osutils.basename(path)
 
505
        else:
 
506
            self.base_path = base_path
 
507
        self.raw_path = path
 
508
 
 
509
    def __cmp__(self, other):
 
510
        return cmp(self.raw_path, other.raw_path)
 
511
 
 
512
    def __hash__(self):
 
513
        return hash(self.raw_path)
 
514
 
 
515
 
 
516
def _add_one_and_parent(tree, inv, parent_ie, path, kind, action):
 
517
    """Add a new entry to the inventory and automatically add unversioned parents.
 
518
 
 
519
    :param inv: Inventory which will receive the new entry.
 
520
    :param parent_ie: Parent inventory entry if known, or None.  If
 
521
        None, the parent is looked up by name and used if present, otherwise it
 
522
        is recursively added.
 
523
    :param kind: Kind of new entry (file, directory, etc)
 
524
    :param action: callback(inv, parent_ie, path, kind); return ignored.
 
525
    :return: A list of paths which have been added.
 
526
    """
 
527
    # Nothing to do if path is already versioned.
 
528
    # This is safe from infinite recursion because the tree root is
 
529
    # always versioned.
 
530
    if parent_ie is not None:
 
531
        # we have a parent ie already
 
532
        added = []
 
533
    else:
 
534
        # slower but does not need parent_ie
 
535
        if inv.has_filename(path.raw_path):
 
536
            return []
 
537
        # its really not there : add the parent
 
538
        # note that the dirname use leads to some extra str copying etc but as
 
539
        # there are a limited number of dirs we can be nested under, it should
 
540
        # generally find it very fast and not recurse after that.
 
541
        added = _add_one_and_parent(tree, inv, None,
 
542
            _FastPath(dirname(path.raw_path)), 'directory', action)
 
543
        parent_id = inv.path2id(dirname(path.raw_path))
 
544
        parent_ie = inv[parent_id]
 
545
    _add_one(tree, inv, parent_ie, path, kind, action)
 
546
    return added + [path.raw_path]
 
547
 
 
548
 
 
549
def _add_one(tree, inv, parent_ie, path, kind, file_id_callback):
 
550
    """Add a new entry to the inventory.
 
551
 
 
552
    :param inv: Inventory which will receive the new entry.
 
553
    :param parent_ie: Parent inventory entry.
 
554
    :param kind: Kind of new entry (file, directory, etc)
 
555
    :param file_id_callback: callback(inv, parent_ie, path, kind); return a
 
556
        file_id or None to generate a new file id
 
557
    :returns: None
 
558
    """
 
559
    file_id = file_id_callback(inv, parent_ie, path, kind)
 
560
    entry = inv.make_entry(kind, path.base_path, parent_ie.file_id,
 
561
        file_id=file_id)
 
562
    inv.add(entry)