~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Martin Pool
  • Date: 2005-05-09 04:38:31 UTC
  • Revision ID: mbp@sourcefrog.net-20050509043831-d45f7832b7d4d5b0
- better message when refusing to add symlinks (from mpe)

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
 
        possible_master_transports=[]
185
 
        if not 'branch-nick' in revprops:
186
 
            revprops['branch-nick'] = self.branch._get_nick(
187
 
                possible_master_transports)
188
 
        author = kwargs.pop('author', None)
189
 
        if author is not None:
190
 
            if 'author' in revprops:
191
 
                # XXX: maybe we should just accept one of them?
192
 
                raise AssertionError('author property given twice')
193
 
            revprops['author'] = author
194
 
        # args for wt.commit start at message from the Commit.commit method,
195
 
        args = (message, ) + args
196
 
        for hook in MutableTree.hooks['start_commit']:
197
 
            hook(self)
198
 
        committed_id = commit.Commit().commit(working_tree=self,
199
 
            revprops=revprops,
200
 
            possible_master_transports=possible_master_transports,
201
 
            *args, **kwargs)
202
 
        return committed_id
203
 
 
204
 
    def _gather_kinds(self, files, kinds):
205
 
        """Helper function for add - sets the entries of kinds."""
206
 
        raise NotImplementedError(self._gather_kinds)
207
 
 
208
 
    def get_file_with_stat(self, file_id, path=None):
209
 
        """Get a file handle and stat object for file_id.
210
 
 
211
 
        The default implementation returns (self.get_file, None) for backwards
212
 
        compatibility.
213
 
 
214
 
        :param file_id: The file id to read.
215
 
        :param path: The path of the file, if it is known.
216
 
        :return: A tuple (file_handle, stat_value_or_None). If the tree has
217
 
            no stat facility, or need for a stat cache feedback during commit,
218
 
            it may return None for the second element of the tuple.
219
 
        """
220
 
        return (self.get_file(file_id, path), None)
221
 
 
222
 
    @needs_read_lock
223
 
    def last_revision(self):
224
 
        """Return the revision id of the last commit performed in this tree.
225
 
 
226
 
        In early tree formats the result of last_revision is the same as the
227
 
        branch last_revision, but that is no longer the case for modern tree
228
 
        formats.
229
 
        
230
 
        last_revision returns the left most parent id, or None if there are no
231
 
        parents.
232
 
 
233
 
        last_revision was deprecated as of 0.11. Please use get_parent_ids
234
 
        instead.
235
 
        """
236
 
        raise NotImplementedError(self.last_revision)
237
 
 
238
 
    def lock_tree_write(self):
239
 
        """Lock the working tree for write, and the branch for read.
240
 
 
241
 
        This is useful for operations which only need to mutate the working
242
 
        tree. Taking out branch write locks is a relatively expensive process
243
 
        and may fail if the branch is on read only media. So branch write locks
244
 
        should only be taken out when we are modifying branch data - such as in
245
 
        operations like commit, pull, uncommit and update.
246
 
        """
247
 
        raise NotImplementedError(self.lock_tree_write)
248
 
 
249
 
    def lock_write(self):
250
 
        """Lock the tree and its branch. This allows mutating calls to be made.
251
 
 
252
 
        Some mutating methods will take out implicit write locks, but in 
253
 
        general you should always obtain a write lock before calling mutating
254
 
        methods on a tree.
255
 
        """
256
 
        raise NotImplementedError(self.lock_write)
257
 
 
258
 
    @needs_write_lock
259
 
    def mkdir(self, path, file_id=None):
260
 
        """Create a directory in the tree. if file_id is None, one is assigned.
261
 
 
262
 
        :param path: A unicode file path.
263
 
        :param file_id: An optional file-id.
264
 
        :return: the file id of the new directory.
265
 
        """
266
 
        raise NotImplementedError(self.mkdir)
267
 
 
268
 
    def _observed_sha1(self, file_id, path, (sha1, stat_value)):
269
 
        """Tell the tree we have observed a paths sha1.
270
 
 
271
 
        The intent of this function is to allow trees that have a hashcache to
272
 
        update the hashcache during commit. If the observed file is too new
273
 
        (based on the stat_value) to be safely hash-cached the tree will ignore
274
 
        it. 
275
 
 
276
 
        The default implementation does nothing.
277
 
 
278
 
        :param file_id: The file id
279
 
        :param path: The file path
280
 
        :param sha1: The sha 1 that was observed.
281
 
        :param stat_value: A stat result for the file the sha1 was read from.
282
 
        :return: None
283
 
        """
284
 
 
285
 
    @needs_write_lock
286
 
    def put_file_bytes_non_atomic(self, file_id, bytes):
287
 
        """Update the content of a file in the tree.
288
 
        
289
 
        Note that the file is written in-place rather than being
290
 
        written to a temporary location and renamed. As a consequence,
291
 
        readers can potentially see the file half-written.
292
 
 
293
 
        :param file_id: file-id of the file
294
 
        :param bytes: the new file contents
295
 
        """
296
 
        raise NotImplementedError(self.put_file_bytes_non_atomic)
297
 
 
298
 
    def set_parent_ids(self, revision_ids, allow_leftmost_as_ghost=False):
299
 
        """Set the parents ids of the working tree.
300
 
 
301
 
        :param revision_ids: A list of revision_ids.
302
 
        """
303
 
        raise NotImplementedError(self.set_parent_ids)
304
 
 
305
 
    def set_parent_trees(self, parents_list, allow_leftmost_as_ghost=False):
306
 
        """Set the parents of the working tree.
307
 
 
308
 
        :param parents_list: A list of (revision_id, tree) tuples. 
309
 
            If tree is None, then that element is treated as an unreachable
310
 
            parent tree - i.e. a ghost.
311
 
        """
312
 
        raise NotImplementedError(self.set_parent_trees)
313
 
 
314
 
    @needs_tree_write_lock
315
 
    def smart_add(self, file_list, recurse=True, action=None, save=True):
316
 
        """Version file_list, optionally recursing into directories.
317
 
 
318
 
        This is designed more towards DWIM for humans than API clarity.
319
 
        For the specific behaviour see the help for cmd_add().
320
 
 
321
 
        :param action: A reporter to be called with the inventory, parent_ie,
322
 
            path and kind of the path being added. It may return a file_id if 
323
 
            a specific one should be used.
324
 
        :param save: Save the inventory after completing the adds. If False
325
 
            this provides dry-run functionality by doing the add and not saving
326
 
            the inventory.
327
 
        :return: A tuple - files_added, ignored_files. files_added is the count
328
 
            of added files, and ignored_files is a dict mapping files that were
329
 
            ignored to the rule that caused them to be ignored.
330
 
        """
331
 
        # not in an inner loop; and we want to remove direct use of this,
332
 
        # so here as a reminder for now. RBC 20070703
333
 
        from bzrlib.inventory import InventoryEntry
334
 
        if action is None:
335
 
            action = add.AddAction()
336
 
        
337
 
        if not file_list:
338
 
            # no paths supplied: add the entire tree.
339
 
            file_list = [u'.']
340
 
        # mutter("smart add of %r")
341
 
        inv = self.inventory
342
 
        added = []
343
 
        ignored = {}
344
 
        dirs_to_add = []
345
 
        user_dirs = set()
346
 
 
347
 
        # validate user file paths and convert all paths to tree 
348
 
        # relative : it's cheaper to make a tree relative path an abspath
349
 
        # than to convert an abspath to tree relative.
350
 
        for filepath in file_list:
351
 
            rf = _FastPath(self.relpath(filepath))
352
 
            # validate user parameters. Our recursive code avoids adding new files
353
 
            # that need such validation 
354
 
            if self.is_control_filename(rf.raw_path):
355
 
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
356
 
            
357
 
            abspath = self.abspath(rf.raw_path)
358
 
            kind = osutils.file_kind(abspath)
359
 
            if kind == 'directory':
360
 
                # schedule the dir for scanning
361
 
                user_dirs.add(rf)
362
 
            else:
363
 
                if not InventoryEntry.versionable_kind(kind):
364
 
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
365
 
            # ensure the named path is added, so that ignore rules in the later directory
366
 
            # walk dont skip it.
367
 
            # we dont have a parent ie known yet.: use the relatively slower inventory 
368
 
            # probing method
369
 
            versioned = inv.has_filename(rf.raw_path)
370
 
            if versioned:
371
 
                continue
372
 
            added.extend(_add_one_and_parent(self, inv, None, rf, kind, action))
373
 
 
374
 
        if not recurse:
375
 
            # no need to walk any directories at all.
376
 
            if len(added) > 0 and save:
377
 
                self._write_inventory(inv)
378
 
            return added, ignored
379
 
 
380
 
        # only walk the minimal parents needed: we have user_dirs to override
381
 
        # ignores.
382
 
        prev_dir = None
383
 
 
384
 
        is_inside = osutils.is_inside_or_parent_of_any
385
 
        for path in sorted(user_dirs):
386
 
            if (prev_dir is None or not is_inside([prev_dir], path.raw_path)):
387
 
                dirs_to_add.append((path, None))
388
 
            prev_dir = path.raw_path
389
 
 
390
 
        # dirs_to_add is initialised to a list of directories, but as we scan
391
 
        # directories we append files to it.
392
 
        # XXX: We should determine kind of files when we scan them rather than
393
 
        # adding to this list. RBC 20070703
394
 
        for directory, parent_ie in dirs_to_add:
395
 
            # directory is tree-relative
396
 
            abspath = self.abspath(directory.raw_path)
397
 
 
398
 
            # get the contents of this directory.
399
 
 
400
 
            # find the kind of the path being added.
401
 
            kind = osutils.file_kind(abspath)
402
 
 
403
 
            if not InventoryEntry.versionable_kind(kind):
404
 
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
405
 
                continue
406
 
 
407
 
            if parent_ie is not None:
408
 
                versioned = directory.base_path in parent_ie.children
409
 
            else:
410
 
                # without the parent ie, use the relatively slower inventory 
411
 
                # probing method
412
 
                versioned = inv.has_filename(directory.raw_path)
413
 
 
414
 
            if kind == 'directory':
415
 
                try:
416
 
                    sub_branch = bzrdir.BzrDir.open(abspath)
417
 
                    sub_tree = True
418
 
                except errors.NotBranchError:
419
 
                    sub_tree = False
420
 
                except errors.UnsupportedFormatError:
421
 
                    sub_tree = True
422
 
            else:
423
 
                sub_tree = False
424
 
 
425
 
            if directory.raw_path == '':
426
 
                # mutter("tree root doesn't need to be added")
427
 
                sub_tree = False
428
 
            elif versioned:
429
 
                pass
430
 
                # mutter("%r is already versioned", abspath)
431
 
            elif sub_tree:
432
 
                # XXX: This is wrong; people *might* reasonably be trying to add
433
 
                # subtrees as subtrees.  This should probably only be done in formats 
434
 
                # which can represent subtrees, and even then perhaps only when
435
 
                # the user asked to add subtrees.  At the moment you can add them
436
 
                # specially through 'join --reference', which is perhaps
437
 
                # reasonable: adding a new reference is a special operation and
438
 
                # can have a special behaviour.  mbp 20070306
439
 
                mutter("%r is a nested bzr tree", abspath)
440
 
            else:
441
 
                _add_one(self, inv, parent_ie, directory, kind, action)
442
 
                added.append(directory.raw_path)
443
 
 
444
 
            if kind == 'directory' and not sub_tree:
445
 
                if parent_ie is not None:
446
 
                    # must be present:
447
 
                    this_ie = parent_ie.children[directory.base_path]
448
 
                else:
449
 
                    # without the parent ie, use the relatively slower inventory 
450
 
                    # probing method
451
 
                    this_id = inv.path2id(directory.raw_path)
452
 
                    if this_id is None:
453
 
                        this_ie = None
454
 
                    else:
455
 
                        this_ie = inv[this_id]
456
 
 
457
 
                for subf in sorted(os.listdir(abspath)):
458
 
                    # here we could use TreeDirectory rather than 
459
 
                    # string concatenation.
460
 
                    subp = osutils.pathjoin(directory.raw_path, subf)
461
 
                    # TODO: is_control_filename is very slow. Make it faster. 
462
 
                    # TreeDirectory.is_control_filename could also make this 
463
 
                    # faster - its impossible for a non root dir to have a 
464
 
                    # control file.
465
 
                    if self.is_control_filename(subp):
466
 
                        mutter("skip control directory %r", subp)
467
 
                    elif subf in this_ie.children:
468
 
                        # recurse into this already versioned subdir.
469
 
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
470
 
                    else:
471
 
                        # user selection overrides ignoes
472
 
                        # ignore while selecting files - if we globbed in the
473
 
                        # outer loop we would ignore user files.
474
 
                        ignore_glob = self.is_ignored(subp)
475
 
                        if ignore_glob is not None:
476
 
                            # mutter("skip ignored sub-file %r", subp)
477
 
                            ignored.setdefault(ignore_glob, []).append(subp)
478
 
                        else:
479
 
                            #mutter("queue to add sub-file %r", subp)
480
 
                            dirs_to_add.append((_FastPath(subp, subf), this_ie))
481
 
 
482
 
        if len(added) > 0:
483
 
            if save:
484
 
                self._write_inventory(inv)
485
 
            else:
486
 
                self.read_working_inventory()
487
 
        return added, ignored
488
 
 
489
 
    def update_basis_by_delta(self, new_revid, delta):
490
 
        """Update the parents of this tree after a commit.
491
 
 
492
 
        This gives the tree one parent, with revision id new_revid. The
493
 
        inventory delta is applied to the current basis tree to generate the
494
 
        inventory for the parent new_revid, and all other parent trees are
495
 
        discarded.
496
 
 
497
 
        All the changes in the delta should be changes synchronising the basis
498
 
        tree with some or all of the working tree, with a change to a directory
499
 
        requiring that its contents have been recursively included. That is,
500
 
        this is not a general purpose tree modification routine, but a helper
501
 
        for commit which is not required to handle situations that do not arise
502
 
        outside of commit.
503
 
 
504
 
        :param new_revid: The new revision id for the trees parent.
505
 
        :param delta: An inventory delta (see apply_inventory_delta) describing
506
 
            the changes from the current left most parent revision to new_revid.
507
 
        """
508
 
        # if the tree is updated by a pull to the branch, as happens in
509
 
        # WorkingTree2, when there was no separation between branch and tree,
510
 
        # then just clear merges, efficiency is not a concern for now as this
511
 
        # is legacy environments only, and they are slow regardless.
512
 
        if self.last_revision() == new_revid:
513
 
            self.set_parent_ids([new_revid])
514
 
            return
515
 
        # generic implementation based on Inventory manipulation. See
516
 
        # WorkingTree classes for optimised versions for specific format trees.
517
 
        basis = self.basis_tree()
518
 
        basis.lock_read()
519
 
        inventory = basis.inventory
520
 
        basis.unlock()
521
 
        inventory.apply_delta(delta)
522
 
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
523
 
        self.set_parent_trees([(new_revid, rev_tree)])
524
 
 
525
 
 
526
 
class MutableTreeHooks(hooks.Hooks):
527
 
    """A dictionary mapping a hook name to a list of callables for mutabletree 
528
 
    hooks.
529
 
    """
530
 
 
531
 
    def __init__(self):
532
 
        """Create the default hooks.
533
 
 
534
 
        """
535
 
        hooks.Hooks.__init__(self)
536
 
        # Invoked before a commit is done in a tree. New in 1.4
537
 
        self['start_commit'] = []
538
 
 
539
 
 
540
 
# install the default hooks into the MutableTree class.
541
 
MutableTree.hooks = MutableTreeHooks()
542
 
 
543
 
 
544
 
class _FastPath(object):
545
 
    """A path object with fast accessors for things like basename."""
546
 
 
547
 
    __slots__ = ['raw_path', 'base_path']
548
 
 
549
 
    def __init__(self, path, base_path=None):
550
 
        """Construct a FastPath from path."""
551
 
        if base_path is None:
552
 
            self.base_path = osutils.basename(path)
553
 
        else:
554
 
            self.base_path = base_path
555
 
        self.raw_path = path
556
 
 
557
 
    def __cmp__(self, other):
558
 
        return cmp(self.raw_path, other.raw_path)
559
 
 
560
 
    def __hash__(self):
561
 
        return hash(self.raw_path)
562
 
 
563
 
 
564
 
def _add_one_and_parent(tree, inv, parent_ie, path, kind, action):
565
 
    """Add a new entry to the inventory and automatically add unversioned parents.
566
 
 
567
 
    :param inv: Inventory which will receive the new entry.
568
 
    :param parent_ie: Parent inventory entry if known, or None.  If
569
 
        None, the parent is looked up by name and used if present, otherwise it
570
 
        is recursively added.
571
 
    :param kind: Kind of new entry (file, directory, etc)
572
 
    :param action: callback(inv, parent_ie, path, kind); return ignored.
573
 
    :return: A list of paths which have been added.
574
 
    """
575
 
    # Nothing to do if path is already versioned.
576
 
    # This is safe from infinite recursion because the tree root is
577
 
    # always versioned.
578
 
    if parent_ie is not None:
579
 
        # we have a parent ie already
580
 
        added = []
581
 
    else:
582
 
        # slower but does not need parent_ie
583
 
        if inv.has_filename(path.raw_path):
584
 
            return []
585
 
        # its really not there : add the parent
586
 
        # note that the dirname use leads to some extra str copying etc but as
587
 
        # there are a limited number of dirs we can be nested under, it should
588
 
        # generally find it very fast and not recurse after that.
589
 
        added = _add_one_and_parent(tree, inv, None,
590
 
            _FastPath(dirname(path.raw_path)), 'directory', action)
591
 
        parent_id = inv.path2id(dirname(path.raw_path))
592
 
        parent_ie = inv[parent_id]
593
 
    _add_one(tree, inv, parent_ie, path, kind, action)
594
 
    return added + [path.raw_path]
595
 
 
596
 
 
597
 
def _add_one(tree, inv, parent_ie, path, kind, file_id_callback):
598
 
    """Add a new entry to the inventory.
599
 
 
600
 
    :param inv: Inventory which will receive the new entry.
601
 
    :param parent_ie: Parent inventory entry.
602
 
    :param kind: Kind of new entry (file, directory, etc)
603
 
    :param file_id_callback: callback(inv, parent_ie, path, kind); return a
604
 
        file_id or None to generate a new file id
605
 
    :returns: None
606
 
    """
607
 
    file_id = file_id_callback(inv, parent_ie, path, kind)
608
 
    entry = inv.make_entry(kind, path.base_path, parent_ie.file_id,
609
 
        file_id=file_id)
610
 
    inv.add(entry)