/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: v.ladeuil+lp at free
  • Date: 2006-10-04 09:40:03 UTC
  • mto: (2145.1.1 keepalive)
  • mto: This revision was merged to the branch mainline in revision 2146.
  • Revision ID: v.ladeuil+lp@free.fr-20061004094003-4a058914ddf1401a
Better design for bogus servers. Both urllib and pycurl pass tests.

* bzrlib/transport/http/_urllib2_wrappers.py:
(ConnectionHandler.create_connection): Raise InvalidURL instead of
urllib2.URLError.
(AbstractHTTPHandler.http_request): New method to override default
urllib2 headers.

* bzrlib/tests/test_http.py:
(TestHttpConnections): Renamed from HttpMixins. Note that this class
should be inherited by a class that also inherit from
TestCaseWithWebserver, this was already the case, but that's now
mentioned in the doc string.
(TestHttpConnections.test_has_on_bogus_host): Moved from
TestHttpConnections_urllib to TestHttpConnections so that pycurl get
tested too.
(TestWithTransport_pycurl): New class to share the pycurl dependency
testing when creating test cases for pycurl.
(TestHttpConnections_urllib, TestHttpConnections_pycurl): Simplified
accordingly.
(TestWallServer, TestWallServer_urllib, TestWallServer_pycurl): Use the
same pattern as TestHttpConnections to share tests between urllib and
pycurl.

* bzrlib/errors.py: 
(ConnectionError): Yes, the 'Connection error' part seems redundant
with the leading ConnectionError in the message, but so far, at least
one test (SSHVendorBadConnection.test_bad_connection_ssh) relies on it
and I don't know if suppressing this redundancy is the best way to
solve the problem.

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
 
 
20
import os
 
21
 
 
22
from bzrlib.errors import BoundBranchOutOfDate
 
23
 
 
24
 
 
25
def uncommit(branch, dry_run=False, verbose=False, revno=None, tree=None):
 
26
    """Remove the last revision from the supplied branch.
 
27
 
 
28
    :param dry_run: Don't actually change anything
 
29
    :param verbose: Print each step as you do it
 
30
    :param revno: Remove back to this revision
 
31
    """
 
32
    unlockable = []
 
33
    try:
 
34
        if tree is not None:
 
35
            tree.lock_write()
 
36
            unlockable.append(tree)
 
37
        
 
38
        branch.lock_write()
 
39
        unlockable.append(branch)
 
40
 
 
41
        pending_merges = []
 
42
        if tree is not None:
 
43
            pending_merges = tree.get_parent_ids()[1:]
 
44
 
 
45
        master = branch.get_master_branch()
 
46
        if master is not None:
 
47
            master.lock_write()
 
48
            unlockable.append(master)
 
49
        rh = branch.revision_history()
 
50
        if master is not None and rh[-1] != master.last_revision():
 
51
            raise BoundBranchOutOfDate(branch, master)
 
52
        if revno is None:
 
53
            revno = len(rh)
 
54
 
 
55
        files_to_remove = []
 
56
        for r in range(revno-1, len(rh)):
 
57
            rev_id = rh.pop()
 
58
            rev = branch.repository.get_revision(rev_id)
 
59
            # When we finish popping off the pending merges, we want
 
60
            # them to stay in the order that they used to be.
 
61
            # but we pop from the end, so reverse the order, and
 
62
            # then get the order right at the end
 
63
            pending_merges.extend(reversed(rev.parent_ids[1:]))
 
64
            if verbose:
 
65
                print 'Removing revno %d: %s' % (len(rh)+1, rev_id)
 
66
 
 
67
        # Committing before we start removing files, because
 
68
        # once we have removed at least one, all the rest are invalid.
 
69
        if not dry_run:
 
70
            if master is not None:
 
71
                master.set_revision_history(rh)
 
72
            branch.set_revision_history(rh)
 
73
            if tree is not None:
 
74
                branch_tip = branch.last_revision()
 
75
                if branch_tip is not None:
 
76
                    parents = [branch.last_revision()]
 
77
                else:
 
78
                    parents = []
 
79
                parents.extend(reversed(pending_merges))
 
80
                tree.set_parent_ids(parents)
 
81
    finally:
 
82
        for item in reversed(unlockable):
 
83
            item.unlock()