/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/generate_ids.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/generate_ids.py
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""Common code for generating file or revision ids."""
18
18
 
32
32
    lazy_regex,
33
33
    )
34
34
 
35
 
# the regex removes any weird characters; we don't escape them
 
35
# the regex removes any weird characters; we don't escape them 
36
36
# but rather just pull them out
37
37
_file_id_chars_re = lazy_regex.lazy_compile(r'[^\w.]')
38
38
_rev_id_chars_re = lazy_regex.lazy_compile(r'[^-\w.+@]')
42
42
 
43
43
def _next_id_suffix():
44
44
    """Create a new file id suffix that is reasonably unique.
45
 
 
 
45
    
46
46
    On the first call we combine the current time with 64 bits of randomness to
47
47
    give a highly probably globally unique number. Then each call in the same
48
48
    process adds 1 to a serial number we append to that unique value.
49
49
    """
50
 
    # XXX TODO: change bzrlib.add.smart_add_tree to call workingtree.add() rather
 
50
    # XXX TODO: change bzrlib.add.smart_add to call workingtree.add() rather 
51
51
    # than having to move the id randomness out of the inner loop like this.
52
52
    # XXX TODO: for the global randomness this uses we should add the thread-id
53
53
    # before the serial #.
80
80
    #    filesystems
81
81
    # 4) Removing starting '.' characters to prevent the file ids from
82
82
    #    being considered hidden.
83
 
    ascii_word_only = str(_file_id_chars_re.sub('', name.lower()))
 
83
    ascii_word_only = _file_id_chars_re.sub('', name.lower())
84
84
    short_no_dots = ascii_word_only.lstrip('.')[:20]
85
85
    return short_no_dots + _next_id_suffix()
86
86
 
114
114
    if timestamp is None:
115
115
        timestamp = time.time()
116
116
 
117
 
    rev_id = u'-'.join((user_or_email,
118
 
                        osutils.compact_date(timestamp),
119
 
                        unique_chunk))
120
 
    return rev_id.encode('utf8')
 
117
    return '-'.join((user_or_email,
 
118
                    osutils.compact_date(timestamp),
 
119
                    unique_chunk))