/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/store/text.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/store/text.py
 
1
# Copyright (C) 2005 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
"""A store that keeps the full text of every version.
 
18
 
 
19
This store keeps uncompressed versions of the full text. It does not
 
20
do any sort of delta compression.
 
21
"""
 
22
 
 
23
import os
 
24
import bzrlib.store
 
25
from bzrlib.trace import mutter
 
26
from bzrlib.errors import BzrError, NoSuchFile, FileExists
 
27
 
 
28
import gzip
 
29
from cStringIO import StringIO
 
30
 
 
31
 
 
32
class TextStore(bzrlib.store.TransportStore):
 
33
    """Store that holds files indexed by unique names.
 
34
 
 
35
    Files can be added, but not modified once they are in.  Typically
 
36
    the hash is used as the name, or something else known to be unique,
 
37
    such as a UUID.
 
38
 
 
39
    Files are stored uncompressed, with no delta compression.
 
40
    """
 
41
 
 
42
    def _add_compressed(self, fn, f):
 
43
        from cStringIO import StringIO
 
44
        from bzrlib.osutils import pumpfile
 
45
        
 
46
        if isinstance(f, basestring):
 
47
            f = StringIO(f)
 
48
            
 
49
        sio = StringIO()
 
50
        gf = gzip.GzipFile(mode='wb', fileobj=sio)
 
51
        # if pumpfile handles files that don't fit in ram,
 
52
        # so will this function
 
53
        pumpfile(f, gf)
 
54
        gf.close()
 
55
        sio.seek(0)
 
56
        self._try_put(fn, sio)
 
57
 
 
58
    def _add(self, fn, f):
 
59
        if self._compressed:
 
60
            self._add_compressed(fn, f)
 
61
        else:
 
62
            self._try_put(fn, f)
 
63
 
 
64
    def _try_put(self, fn, f):
 
65
        try:
 
66
            self._transport.put_file(fn, f, mode=self._file_mode)
 
67
        except NoSuchFile:
 
68
            if not self._prefixed:
 
69
                raise
 
70
            try:
 
71
                self._transport.mkdir(os.path.dirname(fn), mode=self._dir_mode)
 
72
            except FileExists:
 
73
                pass
 
74
            self._transport.put_file(fn, f, mode=self._file_mode)
 
75
 
 
76
    def _get(self, fn):
 
77
        if fn.endswith('.gz'):
 
78
            return self._get_compressed(fn)
 
79
        else:
 
80
            return self._transport.get(fn)
 
81
 
 
82
    def _copy_one(self, fileid, suffix, other, pb):
 
83
        # TODO: Once the copy_to interface is improved to allow a source
 
84
        #       and destination targets, then we can always do the copy
 
85
        #       as long as other is a TextStore
 
86
        if not (isinstance(other, TextStore)
 
87
            and other._prefixed == self._prefixed):
 
88
            return super(TextStore, self)._copy_one(fileid, suffix, other, pb)
 
89
 
 
90
        mutter('_copy_one: %r, %r', fileid, suffix)
 
91
        path = other._get_name(fileid, suffix)
 
92
        if path is None:
 
93
            raise KeyError(fileid + '-' + str(suffix))
 
94
 
 
95
        try:
 
96
            result = other._transport.copy_to([path], self._transport,
 
97
                                              mode=self._file_mode)
 
98
        except NoSuchFile:
 
99
            if not self._prefixed:
 
100
                raise
 
101
            try:
 
102
                self._transport.mkdir(self.hash_prefix(fileid)[:-1], mode=self._dir_mode)
 
103
            except FileExists:
 
104
                pass
 
105
            result = other._transport.copy_to([path], self._transport,
 
106
                                              mode=self._file_mode)
 
107
 
 
108
        if result != 1:
 
109
            raise BzrError('Unable to copy file: %r' % (path,))
 
110
 
 
111
    def _get_compressed(self, filename):
 
112
        """Returns a file reading from a particular entry."""
 
113
        f = self._transport.get(filename)
 
114
        # gzip.GzipFile.read() requires a tell() function
 
115
        # but some transports return objects that cannot seek
 
116
        # so buffer them in a StringIO instead
 
117
        if getattr(f, 'tell', None) is not None:
 
118
            return gzip.GzipFile(mode='rb', fileobj=f)
 
119
        else:
 
120
            from cStringIO import StringIO
 
121
            sio = StringIO(f.read())
 
122
            return gzip.GzipFile(mode='rb', fileobj=sio)