/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/bundle/__init__.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/bundle/__init__.py
 
1
# Copyright (C) 2005, 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
from bzrlib.lazy_import import lazy_import
 
18
lazy_import(globals(), """
 
19
from bzrlib import (
 
20
    errors,
 
21
    urlutils,
 
22
    )
 
23
from bzrlib.bundle import serializer as _serializer
 
24
from bzrlib.transport import get_transport as _get_transport
 
25
""")
 
26
 
 
27
 
 
28
def read_bundle_from_url(url):
 
29
    """Read a bundle from a given URL.
 
30
 
 
31
    :return: A BundleReader, may raise NotABundle if the target 
 
32
            is not a proper bundle.
 
33
    """
 
34
    url, filename = urlutils.split(url, exclude_trailing_slash=False)
 
35
    if not filename:
 
36
        # A path to a directory was passed in
 
37
        # definitely not a bundle
 
38
        raise errors.NotABundle('A directory cannot be a bundle')
 
39
 
 
40
    # All of this must be in the try/except
 
41
    # Some transports cannot detect that we are trying to read a
 
42
    # directory until we actually issue read() on the handle.
 
43
    try:
 
44
        t = _get_transport(url)
 
45
        f = t.get(filename)
 
46
        return _serializer.read_bundle(f)
 
47
    except (errors.TransportError, errors.PathError), e:
 
48
        raise errors.NotABundle(str(e))
 
49
    except (IOError,), e:
 
50
        # jam 20060707
 
51
        # Abstraction leakage, SFTPTransport.get('directory')
 
52
        # doesn't always fail at get() time. Sometimes it fails
 
53
        # during read. And that raises a generic IOError with
 
54
        # just the string 'Failure'
 
55
        # StubSFTPServer does fail during get() (because of prefetch) 
 
56
        # so it has an opportunity to translate the error.
 
57
        raise errors.NotABundle(str(e))
 
58