/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/annotate.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/annotate.py
1
 
# Copyright (C) 2004, 2005 by Canonical Ltd
2
 
 
 
1
# Copyright (C) 2004, 2005 Canonical Ltd
 
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
 
 
7
#
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
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
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
26
26
# e.g. "3:12 Tue", "13 Oct", "Oct 2005", etc.  
27
27
 
28
28
import sys
29
 
import os
30
29
import time
31
30
 
32
 
import bzrlib.weave
33
31
from bzrlib.config import extract_email_address
34
 
from bzrlib.errors import BzrError
 
32
from bzrlib.errors import NoEmailInUsername
35
33
 
36
34
 
37
35
def annotate_file(branch, rev_id, file_id, verbose=False, full=False,
41
39
 
42
40
    prevanno=''
43
41
    annotation = list(_annotate_file(branch, rev_id, file_id))
44
 
    max_origin_len = max(len(origin) for origin in set(x[1] for x in annotation))
 
42
    if len(annotation) == 0:
 
43
        max_origin_len = 0
 
44
    else:
 
45
        max_origin_len = max(len(origin) for origin in set(x[1] for x in annotation))
45
46
    for (revno_str, author, date_str, line_rev_id, text ) in annotation:
46
47
        if verbose:
47
48
            anno = '%5s %-*s %8s ' % (revno_str, max_origin_len, author, date_str)
58
59
    w = branch.repository.weave_store.get_weave(file_id, 
59
60
        branch.repository.get_transaction())
60
61
    last_origin = None
61
 
    for origin, text in w.annotate_iter(rev_id):
 
62
    annotations = list(w.annotate_iter(rev_id))
 
63
    revision_ids = set(o for o, t in annotations)
 
64
    revision_ids = [o for o in revision_ids if 
 
65
                    branch.repository.has_revision(o)]
 
66
    revisions = dict((r.revision_id, r) for r in 
 
67
                     branch.repository.get_revisions(revision_ids))
 
68
    for origin, text in annotations:
62
69
        text = text.rstrip('\r\n')
63
70
        if origin == last_origin:
64
71
            (revno_str, author, date_str) = ('','','')
65
72
        else:
66
73
            last_origin = origin
67
 
            if not branch.repository.has_revision(origin):
 
74
            if origin not in revisions:
68
75
                (revno_str, author, date_str) = ('?','?','?')
69
76
            else:
70
77
                if origin in rh:
71
78
                    revno_str = str(rh.index(origin) + 1)
72
79
                else:
73
80
                    revno_str = 'merge'
74
 
            rev = branch.repository.get_revision(origin)
 
81
            rev = revisions[origin]
75
82
            tz = rev.timezone or 0
76
83
            date_str = time.strftime('%Y%m%d', 
77
84
                                     time.gmtime(rev.timestamp + tz))
80
87
            author = rev.committer
81
88
            try:
82
89
                author = extract_email_address(author)
83
 
            except BzrError:
 
90
            except NoEmailInUsername:
84
91
                pass        # use the whole name
85
92
        yield (revno_str, author, date_str, origin, text)