/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/uncommit.py

  • Committer: Vincent Ladeuil
  • Date: 2007-06-24 15:16:40 UTC
  • mto: (2547.2.1 jam-integration)
  • mto: This revision was merged to the branch mainline in revision 2550.
  • Revision ID: v.ladeuil+lp@free.fr-20070624151640-5tg62jb5g9tqvybd
Fix 121889 by working around urllib2 bug.

* tests/HTTPTestUtil.py:
(DigestAuthRequestHandler.send_header_auth_reqed): python-2.4.1
fail to decode the header without the quotes. The RFC do not
require them, python >= 2.4.4 handles them gracefully. Not a big
deal.

* tests/test_http.py:
(TestAuth.setUp): Add a comment in hope I will not running around
shouting: "Who takes my traces ? Gimme my traces !" when running
the only tests who capture their own traces without showing them.

* transport/http/_urllib2_wrappers.py (BasicAuthHandler,
DigestAuthHandler): Not directly related to the bug, bug good to
fix anyway, the digest auth should be preferred to the basic
one. To do so, the digest handler should be tried before the basic
one.

* builtins.py:
(cmd_selftest.run): Fix typo. Note to reviewers: No, it's not
related to the bug. No there are no tests for that. No I don't
intend to write some :) But I'll understand if you veto that
because you want to take care of it :D

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2006 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
"""Remove the last revision from the history of the current branch."""
 
18
 
 
19
# TODO: make the guts of this methods on tree, branch.
 
20
 
 
21
import os
 
22
 
 
23
from bzrlib.branch import Branch
 
24
from bzrlib.errors import BoundBranchOutOfDate
 
25
 
 
26
 
 
27
def uncommit(branch, dry_run=False, verbose=False, revno=None, tree=None):
 
28
    """Remove the last revision from the supplied branch.
 
29
 
 
30
    :param dry_run: Don't actually change anything
 
31
    :param verbose: Print each step as you do it
 
32
    :param revno: Remove back to this revision
 
33
    """
 
34
    unlockable = []
 
35
    try:
 
36
        if tree is not None:
 
37
            tree.lock_write()
 
38
            unlockable.append(tree)
 
39
        
 
40
        branch.lock_write()
 
41
        unlockable.append(branch)
 
42
 
 
43
        pending_merges = []
 
44
        if tree is not None:
 
45
            pending_merges = tree.get_parent_ids()[1:]
 
46
 
 
47
        master = branch.get_master_branch()
 
48
        if master is not None:
 
49
            master.lock_write()
 
50
            unlockable.append(master)
 
51
        rh = branch.revision_history()
 
52
        if master is not None and rh[-1] != master.last_revision():
 
53
            raise BoundBranchOutOfDate(branch, master)
 
54
        if revno is None:
 
55
            revno = len(rh)
 
56
        old_revno, old_tip = branch.last_revision_info()
 
57
        new_revno = revno -1
 
58
 
 
59
        files_to_remove = []
 
60
        for r in range(revno-1, len(rh)):
 
61
            rev_id = rh.pop()
 
62
            # NB: performance would be better using the revision graph rather
 
63
            # than the whole revision.
 
64
            rev = branch.repository.get_revision(rev_id)
 
65
            # When we finish popping off the pending merges, we want
 
66
            # them to stay in the order that they used to be.
 
67
            # but we pop from the end, so reverse the order, and
 
68
            # then get the order right at the end
 
69
            pending_merges.extend(reversed(rev.parent_ids[1:]))
 
70
            if verbose:
 
71
                print 'Removing revno %d: %s' % (len(rh)+1, rev_id)
 
72
 
 
73
        # Committing before we start removing files, because
 
74
        # once we have removed at least one, all the rest are invalid.
 
75
        if not dry_run:
 
76
            if master is not None:
 
77
                master.set_revision_history(rh)
 
78
            branch.set_revision_history(rh)
 
79
            new_tip = branch.last_revision()
 
80
            if master is None:
 
81
                hook_local = None
 
82
                hook_master = branch
 
83
            else:
 
84
                hook_local = branch
 
85
                hook_master = master
 
86
            for hook in Branch.hooks['post_uncommit']:
 
87
                hook(hook_local, hook_master, old_revno, old_tip, new_revno,
 
88
                    new_tip)
 
89
            if tree is not None:
 
90
                if new_tip is not None:
 
91
                    parents = [new_tip]
 
92
                else:
 
93
                    parents = []
 
94
                parents.extend(reversed(pending_merges))
 
95
                tree.set_parent_ids(parents)
 
96
    finally:
 
97
        for item in reversed(unlockable):
 
98
            item.unlock()