/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-12-01 15:06:29 UTC
  • mto: (2172.3.1 bzr.73948)
  • mto: This revision was merged to the branch mainline in revision 2181.
  • Revision ID: v.ladeuil+lp@free.fr-20061201150629-zjd2an87u0r7nhhw
The tests that would have help avoid bug #73948 and all that mess :)

* bzrlib/transport/http/response.py:
(handle_response): Translate a 416 http error code into a bzr
exception.

* bzrlib/transport/http/_urllib2_wrappers.py:
(HTTPDefaultErrorHandler.http_error_default): Translate a 416 http
error code into a bzr exception.

* bzrlib/transport/http/_pycurl.py:
(PyCurlTransport._curl_perform): It could happen that pycrul
itself detect a short read.

* bzrlib/transport/http/__init__.py:
(HttpTransportBase._retry_get): New method, factorizing the retry
logic.
(HttpTransportBase.readv): We can have exception during the
initial GET worth degrading the range requirements (i.e. retrying
the GET request with either single or not ranges).

* bzrlib/tests/test_transport_implementations.py:
(TransportTests.test_readv_short_read): InvalidRange can also be
raised.

* bzrlib/tests/test_http.py:
(TestRangeRequestServer.test_readv_invalid_ranges): Was named
test_readv_short_read, the new name make the intent
clearer. Depending of the code path used (urllib or pycurl), both
exceptions can be raised.

* bzrlib/tests/HttpServer.py:
(TestingHTTPRequestHandler.do_GET): If invalid ranges are
specified, returns a 416 instead of the whole file (both are valid
according to the RFC).

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/uncommit.py
 
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()