~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-20 16:15:26 UTC
  • mfrom: (5430.4.1 work)
  • Revision ID: pqm@pqm.ubuntu.com-20100920161526-3r87u084xg7d3pd6
(vila) Tweak tools/check-newsbug.py and do some light editing in
 doc/developers/ppa.txt. (Vincent Ladeuil)

Show diffs side-by-side

added added

removed removed

Lines of Context:
32
32
    hooks,
33
33
    osutils,
34
34
    revisiontree,
 
35
    inventory,
35
36
    symbol_versioning,
36
37
    trace,
37
38
    tree,
182
183
               **kwargs):
183
184
        # avoid circular imports
184
185
        from bzrlib import commit
185
 
        if revprops is None:
186
 
            revprops = {}
187
186
        possible_master_transports=[]
188
 
        if not 'branch-nick' in revprops:
189
 
            revprops['branch-nick'] = self.branch._get_nick(
 
187
        revprops = commit.Commit.update_revprops(
 
188
                revprops,
 
189
                self.branch,
 
190
                kwargs.pop('authors', None),
 
191
                kwargs.pop('author', None),
190
192
                kwargs.get('local', False),
191
193
                possible_master_transports)
192
 
        authors = kwargs.pop('authors', None)
193
 
        author = kwargs.pop('author', None)
194
 
        if authors is not None:
195
 
            if author is not None:
196
 
                raise AssertionError('Specifying both author and authors '
197
 
                        'is not allowed. Specify just authors instead')
198
 
            if 'author' in revprops or 'authors' in revprops:
199
 
                # XXX: maybe we should just accept one of them?
200
 
                raise AssertionError('author property given twice')
201
 
            if authors:
202
 
                for individual in authors:
203
 
                    if '\n' in individual:
204
 
                        raise AssertionError('\\n is not a valid character '
205
 
                                'in an author identity')
206
 
                revprops['authors'] = '\n'.join(authors)
207
 
        if author is not None:
208
 
            symbol_versioning.warn('The parameter author was deprecated'
209
 
                   ' in version 1.13. Use authors instead',
210
 
                   DeprecationWarning)
211
 
            if 'author' in revprops or 'authors' in revprops:
212
 
                # XXX: maybe we should just accept one of them?
213
 
                raise AssertionError('author property given twice')
214
 
            if '\n' in author:
215
 
                raise AssertionError('\\n is not a valid character '
216
 
                        'in an author identity')
217
 
            revprops['authors'] = author
218
194
        # args for wt.commit start at message from the Commit.commit method,
219
195
        args = (message, ) + args
220
196
        for hook in MutableTree.hooks['start_commit']:
400
376
        This is designed more towards DWIM for humans than API clarity.
401
377
        For the specific behaviour see the help for cmd_add().
402
378
 
 
379
        :param file_list: List of zero or more paths.  *NB: these are 
 
380
            interpreted relative to the process cwd, not relative to the 
 
381
            tree.*  (Add and most other tree methods use tree-relative
 
382
            paths.)
403
383
        :param action: A reporter to be called with the inventory, parent_ie,
404
384
            path and kind of the path being added. It may return a file_id if
405
385
            a specific one should be used.
436
416
            for c in self.conflicts():
437
417
                conflicts_related.update(c.associated_filenames())
438
418
 
 
419
        # expand any symlinks in the directory part, while leaving the
 
420
        # filename alone
 
421
        file_list = map(osutils.normalizepath, file_list)
 
422
 
439
423
        # validate user file paths and convert all paths to tree
440
424
        # relative : it's cheaper to make a tree relative path an abspath
441
425
        # than to convert an abspath to tree relative, and it's cheaper to
740
724
        file_id or None to generate a new file id
741
725
    :returns: None
742
726
    """
 
727
    # if the parent exists, but isn't a directory, we have to do the
 
728
    # kind change now -- really the inventory shouldn't pretend to know
 
729
    # the kind of wt files, but it does.
 
730
    if parent_ie.kind != 'directory':
 
731
        # nb: this relies on someone else checking that the path we're using
 
732
        # doesn't contain symlinks.
 
733
        new_parent_ie = inventory.make_entry('directory', parent_ie.name,
 
734
            parent_ie.parent_id, parent_ie.file_id)
 
735
        del inv[parent_ie.file_id]
 
736
        inv.add(new_parent_ie)
 
737
        parent_ie = new_parent_ie
743
738
    file_id = file_id_callback(inv, parent_ie, path, kind)
744
739
    entry = inv.make_entry(kind, path.base_path, parent_ie.file_id,
745
740
        file_id=file_id)