/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/revision.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
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
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from __future__ import absolute_import
18
 
 
19
17
# TODO: Some kind of command-line display of revision properties:
20
18
# perhaps show them in log -v and allow them as options to the commit command.
21
19
 
22
20
 
23
 
from .lazy_import import lazy_import
 
21
from bzrlib.lazy_import import lazy_import
24
22
lazy_import(globals(), """
25
 
from breezy import bugtracker
 
23
from bzrlib import deprecated_graph
 
24
from bzrlib import bugtracker
26
25
""")
27
 
from . import (
 
26
from bzrlib import (
28
27
    errors,
29
 
    osutils,
30
 
    )
31
 
from .sixish import (
32
 
    text_type,
33
 
    )
 
28
    symbol_versioning,
 
29
    )
 
30
from bzrlib.osutils import contains_whitespace
34
31
 
35
 
NULL_REVISION = b"null:"
36
 
CURRENT_REVISION = b"current:"
 
32
NULL_REVISION="null:"
 
33
CURRENT_REVISION="current:"
37
34
 
38
35
 
39
36
class Revision(object):
88
85
 
89
86
    def _check_properties(self):
90
87
        """Verify that all revision properties are OK."""
91
 
        for name, value in self.properties.items():
92
 
            # GZ 2017-06-10: What sort of string are properties exactly?
93
 
            not_text = not isinstance(name, (text_type, str))
94
 
            if not_text or osutils.contains_whitespace(name):
 
88
        for name, value in self.properties.iteritems():
 
89
            if not isinstance(name, basestring) or contains_whitespace(name):
95
90
                raise ValueError("invalid property name %r" % name)
96
 
            if not isinstance(value, (text_type, bytes)):
 
91
            if not isinstance(value, basestring):
97
92
                raise ValueError("invalid property value %r for %r" %
98
93
                                 (value, name))
99
94
 
126
121
        else:
127
122
            return ''
128
123
 
 
124
    @symbol_versioning.deprecated_method(symbol_versioning.deprecated_in((1, 13, 0)))
 
125
    def get_apparent_author(self):
 
126
        """Return the apparent author of this revision.
 
127
 
 
128
        This method is deprecated in favour of get_apparent_authors.
 
129
 
 
130
        If the revision properties contain any author names,
 
131
        return the first. Otherwise return the committer name.
 
132
        """
 
133
        authors = self.get_apparent_authors()
 
134
        if authors:
 
135
            return authors[0]
 
136
        else:
 
137
            return None
 
138
 
129
139
    def get_apparent_authors(self):
130
140
        """Return the apparent authors of this revision.
131
141
 
152
162
            try:
153
163
                url, status = line.split(None, 2)
154
164
            except ValueError:
155
 
                raise bugtracker.InvalidLineInBugsProperty(line)
 
165
                raise errors.InvalidLineInBugsProperty(line)
156
166
            if status not in bugtracker.ALLOWED_BUG_STATUSES:
157
 
                raise bugtracker.InvalidBugStatus(status)
 
167
                raise errors.InvalidBugStatus(status)
158
168
            yield url, status
159
169
 
160
170
 
168
178
                yield ancestor, distance
169
179
            try:
170
180
                revision = revision_source.get_revision(ancestor)
171
 
            except errors.NoSuchRevision as e:
 
181
            except errors.NoSuchRevision, e:
172
182
                if e.revision == revision_id:
173
183
                    raise
174
184
                else:
210
220
 
211
221
    :return: True if the revision is reserved, False otherwise
212
222
    """
213
 
    return isinstance(revision_id, bytes) and revision_id.endswith(b':')
 
223
    return isinstance(revision_id, basestring) and revision_id.endswith(':')
214
224
 
215
225
 
216
226
def check_not_reserved_id(revision_id):
222
232
def ensure_null(revision_id):
223
233
    """Ensure only NULL_REVISION is used to represent the null revision"""
224
234
    if revision_id is None:
225
 
        raise ValueError(
226
 
            'NULL_REVISION should be used for the null'
227
 
            ' revision instead of None.')
228
 
    return revision_id
 
235
        symbol_versioning.warn('NULL_REVISION should be used for the null'
 
236
            ' revision instead of None, as of bzr 0.91.',
 
237
            DeprecationWarning, stacklevel=2)
 
238
        return NULL_REVISION
 
239
    else:
 
240
        return revision_id
229
241
 
230
242
 
231
243
def is_null(revision_id):
232
244
    if revision_id is None:
233
 
        raise ValueError('NULL_REVISION should be used for the null'
234
 
                         ' revision instead of None.')
235
 
    return (revision_id == NULL_REVISION)
 
245
        symbol_versioning.warn('NULL_REVISION should be used for the null'
 
246
            ' revision instead of None, as of bzr 0.90.',
 
247
            DeprecationWarning, stacklevel=2)
 
248
    return revision_id in (None, NULL_REVISION)