~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/revision.py

  • Committer: Tim Penhey
  • Date: 2008-04-25 11:23:00 UTC
  • mto: (3473.1.1 ianc-integration)
  • mto: This revision was merged to the branch mainline in revision 3474.
  • Revision ID: tim@penhey.net-20080425112300-sf5soa5dg2d37kvc
Added tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# (C) 2005 Canonical
2
 
 
 
1
# Copyright (C) 2005, 2006 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
16
16
 
17
 
 
18
 
import bzrlib.errors
19
 
from bzrlib.graph import farthest_nodes, node_distances, all_descendants
20
 
 
21
 
class RevisionReference(object):
22
 
    """
23
 
    Reference to a stored revision.
24
 
 
25
 
    Includes the revision_id and revision_sha1.
26
 
    """
27
 
    revision_id = None
28
 
    revision_sha1 = None
29
 
    def __init__(self, revision_id, revision_sha1=None):
30
 
        if revision_id == None \
31
 
           or isinstance(revision_id, basestring):
32
 
            self.revision_id = revision_id
33
 
        else:
34
 
            raise ValueError('bad revision_id %r' % revision_id)
35
 
 
36
 
        if revision_sha1 != None:
37
 
            if isinstance(revision_sha1, basestring) \
38
 
               and len(revision_sha1) == 40:
39
 
                self.revision_sha1 = revision_sha1
40
 
            else:
41
 
                raise ValueError('bad revision_sha1 %r' % revision_sha1)
42
 
                
 
17
# TODO: Some kind of command-line display of revision properties: 
 
18
# perhaps show them in log -v and allow them as options to the commit command.
 
19
 
 
20
 
 
21
from bzrlib import (
 
22
    errors,
 
23
    symbol_versioning,
 
24
    )
 
25
from bzrlib.deprecated_graph import (
 
26
    all_descendants,
 
27
    Graph,
 
28
    node_distances,
 
29
    select_farthest,
 
30
    )
 
31
from bzrlib.osutils import contains_whitespace
 
32
from bzrlib.progress import DummyProgress
 
33
from bzrlib.symbol_versioning import (deprecated_function,
 
34
        )
 
35
 
 
36
NULL_REVISION="null:"
 
37
CURRENT_REVISION="current:"
43
38
 
44
39
 
45
40
class Revision(object):
51
46
 
52
47
    After bzr 0.0.5 revisions are allowed to have multiple parents.
53
48
 
54
 
    parents
55
 
        List of parent revisions, each is a RevisionReference.
 
49
    parent_ids
 
50
        List of parent revision_ids
 
51
 
 
52
    properties
 
53
        Dictionary of revision properties.  These are attached to the
 
54
        revision as extra metadata.  The name must be a single 
 
55
        word; the value can be an arbitrary string.
56
56
    """
57
 
    inventory_id = None
58
 
    inventory_sha1 = None
59
 
    revision_id = None
60
 
    timestamp = None
61
 
    message = None
62
 
    timezone = None
63
 
    committer = None
64
57
    
65
 
    def __init__(self, **args):
 
58
    def __init__(self, revision_id, properties=None, **args):
 
59
        self.revision_id = revision_id
 
60
        self.properties = properties or {}
 
61
        self._check_properties()
 
62
        self.parent_ids = []
 
63
        self.parent_sha1s = []
 
64
        """Not used anymore - legacy from for 4."""
66
65
        self.__dict__.update(args)
67
 
        self.parents = []
68
 
 
69
66
 
70
67
    def __repr__(self):
71
68
        return "<Revision id %s>" % self.revision_id
73
70
    def __eq__(self, other):
74
71
        if not isinstance(other, Revision):
75
72
            return False
76
 
        return (self.inventory_id == other.inventory_id
77
 
                and self.inventory_sha1 == other.inventory_sha1
 
73
        # FIXME: rbc 20050930 parent_ids are not being compared
 
74
        return (
 
75
                self.inventory_sha1 == other.inventory_sha1
78
76
                and self.revision_id == other.revision_id
79
77
                and self.timestamp == other.timestamp
80
78
                and self.message == other.message
81
79
                and self.timezone == other.timezone
82
 
                and self.committer == other.committer)
 
80
                and self.committer == other.committer
 
81
                and self.properties == other.properties)
83
82
 
84
83
    def __ne__(self, other):
85
84
        return not self.__eq__(other)
86
85
 
87
 
        
88
 
 
89
 
REVISION_ID_RE = None
90
 
 
91
 
def validate_revision_id(rid):
92
 
    """Check rid is syntactically valid for a revision id."""
93
 
    global REVISION_ID_RE
94
 
    if not REVISION_ID_RE:
95
 
        import re
96
 
        REVISION_ID_RE = re.compile('[\w.-]+@[\w.-]+--?\d+--?[0-9a-f]+\Z')
97
 
 
98
 
    if not REVISION_ID_RE.match(rid):
99
 
        raise ValueError("malformed revision-id %r" % rid)
100
 
 
101
 
def is_ancestor(revision_id, candidate_id, revision_source):
 
86
    def _check_properties(self):
 
87
        """Verify that all revision properties are OK."""
 
88
        for name, value in self.properties.iteritems():
 
89
            if not isinstance(name, basestring) or contains_whitespace(name):
 
90
                raise ValueError("invalid property name %r" % name)
 
91
            if not isinstance(value, basestring):
 
92
                raise ValueError("invalid property value %r for %r" % 
 
93
                                 (name, value))
 
94
 
 
95
    def get_history(self, repository):
 
96
        """Return the canonical line-of-history for this revision.
 
97
 
 
98
        If ghosts are present this may differ in result from a ghost-free
 
99
        repository.
 
100
        """
 
101
        current_revision = self
 
102
        reversed_result = []
 
103
        while current_revision is not None:
 
104
            reversed_result.append(current_revision.revision_id)
 
105
            if not len (current_revision.parent_ids):
 
106
                reversed_result.append(None)
 
107
                current_revision = None
 
108
            else:
 
109
                next_revision_id = current_revision.parent_ids[0]
 
110
                current_revision = repository.get_revision(next_revision_id)
 
111
        reversed_result.reverse()
 
112
        return reversed_result
 
113
 
 
114
    def get_summary(self):
 
115
        """Get the first line of the log message for this revision.
 
116
        """
 
117
        return self.message.lstrip().split('\n', 1)[0]
 
118
 
 
119
    def get_apparent_author(self):
 
120
        """Return the apparent author of this revision.
 
121
 
 
122
        If the revision properties contain the author name,
 
123
        return it. Otherwise return the committer name.
 
124
        """
 
125
        return self.properties.get('author', self.committer)
 
126
 
 
127
 
 
128
@deprecated_function(symbol_versioning.one_zero)
 
129
def is_ancestor(revision_id, candidate_id, branch):
102
130
    """Return true if candidate_id is an ancestor of revision_id.
 
131
 
103
132
    A false negative will be returned if any intermediate descendent of
104
133
    candidate_id is not present in any of the revision_sources.
105
134
    
106
135
    revisions_source is an object supporting a get_revision operation that
107
136
    behaves like Branch's.
 
137
 
 
138
    This function is deprecated, it is better for callers to directly use
 
139
    Graph.is_ancestor() (just watch out that the parameter order is switched)
108
140
    """
 
141
    return branch.repository.get_graph().is_ancestor(candidate_id, revision_id)
109
142
 
110
 
    for ancestor_id, distance in iter_ancestors(revision_id, revision_source):
111
 
        if ancestor_id == candidate_id:
112
 
            return True
113
 
    return False
114
143
 
115
144
def iter_ancestors(revision_id, revision_source, only_present=False):
116
145
    ancestors = (revision_id,)
122
151
                yield ancestor, distance
123
152
            try:
124
153
                revision = revision_source.get_revision(ancestor)
125
 
            except bzrlib.errors.NoSuchRevision, e:
 
154
            except errors.NoSuchRevision, e:
126
155
                if e.revision == revision_id:
127
156
                    raise 
128
157
                else:
129
158
                    continue
130
159
            if only_present:
131
160
                yield ancestor, distance
132
 
            new_ancestors.extend([p.revision_id for p in revision.parents])
 
161
            new_ancestors.extend(revision.parent_ids)
133
162
        ancestors = new_ancestors
134
163
        distance += 1
135
164
 
145
174
    anc_iter = enumerate(iter_ancestors(revision_id, revision_source,
146
175
                         only_present=True))
147
176
    for anc_order, (anc_id, anc_distance) in anc_iter:
148
 
        if not found_ancestors.has_key(anc_id):
 
177
        if anc_id not in found_ancestors:
149
178
            found_ancestors[anc_id] = (anc_order, anc_distance)
150
179
    return found_ancestors
151
180
    
159
188
    return matches
160
189
 
161
190
 
162
 
def old_common_ancestor(revision_a, revision_b, revision_source):
163
 
    """Find the ancestor common to both revisions that is closest to both.
164
 
    """
165
 
    from bzrlib.trace import mutter
166
 
    a_ancestors = find_present_ancestors(revision_a, revision_source)
167
 
    b_ancestors = find_present_ancestors(revision_b, revision_source)
168
 
    a_intersection = []
169
 
    b_intersection = []
170
 
    # a_order is used as a tie-breaker when two equally-good bases are found
171
 
    for revision, (a_order, a_distance) in a_ancestors.iteritems():
172
 
        if b_ancestors.has_key(revision):
173
 
            a_intersection.append((a_distance, a_order, revision))
174
 
            b_intersection.append((b_ancestors[revision][1], a_order, revision))
175
 
    mutter("a intersection: %r" % a_intersection)
176
 
    mutter("b intersection: %r" % b_intersection)
177
 
 
178
 
    a_closest = __get_closest(a_intersection)
179
 
    if len(a_closest) == 0:
180
 
        return None
181
 
    b_closest = __get_closest(b_intersection)
182
 
    assert len(b_closest) != 0
183
 
    mutter ("a_closest %r" % a_closest)
184
 
    mutter ("b_closest %r" % b_closest)
185
 
    if a_closest[0] in b_closest:
186
 
        return a_closest[0]
187
 
    elif b_closest[0] in a_closest:
188
 
        return b_closest[0]
189
 
    else:
190
 
        raise bzrlib.errors.AmbiguousBase((a_closest[0], b_closest[0]))
191
 
    return a_closest[0]
192
 
 
 
191
@deprecated_function(symbol_versioning.one_four)
193
192
def revision_graph(revision, revision_source):
194
193
    """Produce a graph of the ancestry of the specified revision.
195
 
    Return root, ancestors map, descendants map
196
 
 
197
 
    TODO: Produce graphs with the NULL revision as root, so that we can find
198
 
    a common even when trees are not branches don't represent a single line
199
 
    of descent.
 
194
    
 
195
    :return: root, ancestors map, descendants map
200
196
    """
 
197
    revision_source.lock_read()
 
198
    try:
 
199
        return _revision_graph(revision, revision_source)
 
200
    finally:
 
201
        revision_source.unlock()
 
202
 
 
203
 
 
204
def _revision_graph(revision, revision_source):
 
205
    """See revision_graph."""
 
206
    from bzrlib.tsort import topo_sort
 
207
    graph = revision_source.get_revision_graph(revision)
 
208
    # mark all no-parent revisions as being NULL_REVISION parentage.
 
209
    for node, parents in graph.items():
 
210
        if len(parents) == 0:
 
211
            graph[node] = [NULL_REVISION]
 
212
    # add NULL_REVISION to the graph
 
213
    graph[NULL_REVISION] = []
 
214
 
 
215
    # pick a root. If there are multiple roots
 
216
    # this could pick a random one.
 
217
    topo_order = topo_sort(graph.items())
 
218
    root = topo_order[0]
 
219
 
201
220
    ancestors = {}
202
221
    descendants = {}
203
 
    lines = [revision]
204
 
    root = None
205
 
    descendants[revision] = {}
206
 
    while len(lines) > 0:
207
 
        new_lines = set()
208
 
        for line in lines:
209
 
            try:
210
 
                rev = revision_source.get_revision(line)
211
 
                parents = [p.revision_id for p in rev.parents]
212
 
                if len(parents) == 0:
213
 
                    root = line
214
 
            except bzrlib.errors.NoSuchRevision:
215
 
                if line == revision:
216
 
                    raise
217
 
                parents = None
218
 
            if parents is not None:
219
 
                for parent in parents:
220
 
                    if parent not in ancestors:
221
 
                        new_lines.add(parent)
222
 
                    if parent not in descendants:
223
 
                        descendants[parent] = {}
224
 
                    descendants[parent][line] = 1
225
 
            if parents is not None:
226
 
                ancestors[line] = set(parents)
227
 
        lines = new_lines
 
222
 
 
223
    # map the descendants of the graph.
 
224
    # and setup our set based return graph.
 
225
    for node in graph.keys():
 
226
        descendants[node] = {}
 
227
    for node, parents in graph.items():
 
228
        for parent in parents:
 
229
            descendants[parent][node] = 1
 
230
        ancestors[node] = set(parents)
 
231
 
228
232
    assert root not in descendants[root]
229
233
    assert root not in ancestors[root]
230
234
    return root, ancestors, descendants
231
235
 
 
236
 
 
237
@deprecated_function(symbol_versioning.one_three)
232
238
def combined_graph(revision_a, revision_b, revision_source):
233
239
    """Produce a combined ancestry graph.
234
240
    Return graph root, ancestors map, descendants map, set of common nodes"""
235
 
    root, ancestors, descendants = revision_graph(revision_a, revision_source)
236
 
    root_b, ancestors_b, descendants_b = revision_graph(revision_b, 
237
 
                                                        revision_source)
 
241
    root, ancestors, descendants = revision_graph(
 
242
        revision_a, revision_source)
 
243
    root_b, ancestors_b, descendants_b = revision_graph(
 
244
        revision_b, revision_source)
238
245
    if root != root_b:
239
 
        raise bzrlib.errors.NoCommonRoot(revision_a, revision_b)
 
246
        raise errors.NoCommonRoot(revision_a, revision_b)
240
247
    common = set()
241
248
    for node, node_anc in ancestors_b.iteritems():
242
249
        if node in ancestors:
246
253
        ancestors[node].update(node_anc)
247
254
    for node, node_dec in descendants_b.iteritems():
248
255
        if node not in descendants:
249
 
            descendants[node] = set()
 
256
            descendants[node] = {}
250
257
        descendants[node].update(node_dec)
251
258
    return root, ancestors, descendants, common
252
259
 
253
 
def common_ancestor(revision_a, revision_b, revision_source):
 
260
 
 
261
@deprecated_function(symbol_versioning.one_three)
 
262
def common_ancestor(revision_a, revision_b, revision_source, 
 
263
                    pb=DummyProgress()):
 
264
    if None in (revision_a, revision_b):
 
265
        return None
 
266
    if NULL_REVISION in (revision_a, revision_b):
 
267
        return NULL_REVISION
 
268
    # trivial optimisation
 
269
    if revision_a == revision_b:
 
270
        return revision_a
254
271
    try:
255
 
        root, ancestors, descendants, common = \
256
 
            combined_graph(revision_a, revision_b, revision_source)
257
 
    except bzrlib.errors.NoCommonRoot:
258
 
        raise bzrlib.errors.NoCommonAncestor(revision_a, revision_b)
259
 
        
260
 
    nodes = farthest_nodes(descendants, ancestors, root)
261
 
    for node in nodes:
262
 
        if node in common:
263
 
            return node
264
 
    raise bzrlib.errors.NoCommonAncestor(revision_a, revision_b)
 
272
        try:
 
273
            pb.update('Picking ancestor', 1, 3)
 
274
            graph = revision_source.get_revision_graph_with_ghosts(
 
275
                [revision_a, revision_b])
 
276
            # Shortcut the case where one of the tips is already included in
 
277
            # the other graphs ancestry.
 
278
            ancestry_a = graph.get_ancestry(revision_a, topo_sorted=False)
 
279
            if revision_b in ancestry_a:
 
280
                return revision_b
 
281
            ancestry_b = graph.get_ancestry(revision_b, topo_sorted=False)
 
282
            if revision_a in ancestry_b:
 
283
                return revision_a
 
284
            # convert to a NULL_REVISION based graph.
 
285
            ancestors = graph.get_ancestors()
 
286
            descendants = graph.get_descendants()
 
287
            common = set(ancestry_a)
 
288
            common.intersection_update(ancestry_b)
 
289
            descendants[NULL_REVISION] = {}
 
290
            ancestors[NULL_REVISION] = []
 
291
            for root in graph.roots:
 
292
                descendants[NULL_REVISION][root] = 1
 
293
                ancestors[root].append(NULL_REVISION)
 
294
            for ghost in graph.ghosts:
 
295
                # ghosts act as roots for the purpose of finding 
 
296
                # the longest paths from the root: any ghost *might*
 
297
                # be directly attached to the root, so we treat them
 
298
                # as being such.
 
299
                # ghost now descends from NULL
 
300
                descendants[NULL_REVISION][ghost] = 1
 
301
                # that is it has an ancestor of NULL
 
302
                ancestors[ghost] = [NULL_REVISION]
 
303
                # ghost is common if any of ghosts descendants are common:
 
304
                for ghost_descendant in descendants[ghost]:
 
305
                    if ghost_descendant in common:
 
306
                        common.add(ghost)
 
307
                
 
308
            root = NULL_REVISION
 
309
            common.add(NULL_REVISION)
 
310
        except errors.NoCommonRoot:
 
311
            raise errors.NoCommonAncestor(revision_a, revision_b)
 
312
            
 
313
        pb.update('Picking ancestor', 2, 3)
 
314
        distances = node_distances (descendants, ancestors, root)
 
315
        pb.update('Picking ancestor', 3, 2)
 
316
        farthest = select_farthest(distances, common)
 
317
        if farthest is None or farthest == NULL_REVISION:
 
318
            raise errors.NoCommonAncestor(revision_a, revision_b)
 
319
    finally:
 
320
        pb.clear()
 
321
    return farthest
 
322
 
265
323
 
266
324
class MultipleRevisionSources(object):
267
325
    """Proxy that looks in multiple branches for revisions."""
 
326
 
 
327
    @symbol_versioning.deprecated_method(symbol_versioning.one_three)
268
328
    def __init__(self, *args):
269
329
        object.__init__(self)
270
330
        assert len(args) != 0
271
331
        self._revision_sources = args
272
332
 
 
333
    def revision_parents(self, revision_id):
 
334
        for source in self._revision_sources:
 
335
            try:
 
336
                return source.revision_parents(revision_id)
 
337
            except (errors.WeaveRevisionNotPresent, errors.NoSuchRevision), e:
 
338
                pass
 
339
        raise e
 
340
 
273
341
    def get_revision(self, revision_id):
274
342
        for source in self._revision_sources:
275
343
            try:
276
344
                return source.get_revision(revision_id)
277
 
            except bzrlib.errors.NoSuchRevision, e:
 
345
            except errors.NoSuchRevision, e:
278
346
                pass
279
347
        raise e
280
348
 
281
 
def get_intervening_revisions(ancestor_id, rev_id, rev_source, 
282
 
                              revision_history=None):
283
 
    """Find the longest line of descent from maybe_ancestor to revision.
284
 
    Revision history is followed where possible.
285
 
 
286
 
    If ancestor_id == rev_id, list will be empty.
287
 
    Otherwise, rev_id will be the last entry.  ancestor_id will never appear.
288
 
    If ancestor_id is not an ancestor, NotAncestor will be thrown
289
 
    """
290
 
    root, ancestors, descendants = revision_graph(rev_id, rev_source)
291
 
    if len(descendants) == 0:
292
 
        raise NoSuchRevision(rev_source, rev_id)
293
 
    if ancestor_id not in descendants:
294
 
        rev_source.get_revision(ancestor_id)
295
 
        raise bzrlib.errors.NotAncestor(rev_id, ancestor_id)
296
 
    root_descendants = all_descendants(descendants, ancestor_id)
297
 
    root_descendants.add(ancestor_id)
298
 
    if rev_id not in root_descendants:
299
 
        raise bzrlib.errors.NotAncestor(rev_id, ancestor_id)
300
 
    distances = node_distances(descendants, ancestors, ancestor_id,
301
 
                               root_descendants=root_descendants)
302
 
 
303
 
    def best_ancestor(rev_id):
304
 
        best = None
305
 
        for anc_id in ancestors[rev_id]:
306
 
            try:
307
 
                distance = distances[anc_id]
308
 
            except KeyError:
 
349
    def get_revision_graph(self, revision_id):
 
350
        # we could probe incrementally until the pending
 
351
        # ghosts list stop growing, but its cheaper for now
 
352
        # to just ask for the complete graph for each repository.
 
353
        graphs = []
 
354
        for source in self._revision_sources:
 
355
            ghost_graph = source.get_revision_graph_with_ghosts()
 
356
            graphs.append(ghost_graph)
 
357
        absent = 0
 
358
        for graph in graphs:
 
359
            if not revision_id in graph.get_ancestors():
 
360
                absent += 1
 
361
        if absent == len(graphs):
 
362
            raise errors.NoSuchRevision(self._revision_sources[0], revision_id)
 
363
 
 
364
        # combine the graphs
 
365
        result = {}
 
366
        pending = set([revision_id])
 
367
        def find_parents(node_id):
 
368
            """find the parents for node_id."""
 
369
            for graph in graphs:
 
370
                ancestors = graph.get_ancestors()
 
371
                try:
 
372
                    return ancestors[node_id]
 
373
                except KeyError:
 
374
                    pass
 
375
            raise errors.NoSuchRevision(self._revision_sources[0], node_id)
 
376
        while len(pending):
 
377
            # all the graphs should have identical parent lists
 
378
            node_id = pending.pop()
 
379
            try:
 
380
                result[node_id] = find_parents(node_id)
 
381
                for parent_node in result[node_id]:
 
382
                    if not parent_node in result:
 
383
                        pending.add(parent_node)
 
384
            except errors.NoSuchRevision:
 
385
                # ghost, ignore it.
 
386
                pass
 
387
        return result
 
388
 
 
389
    def get_revision_graph_with_ghosts(self, revision_ids):
 
390
        # query all the sources for their entire graphs 
 
391
        # and then build a combined graph for just
 
392
        # revision_ids.
 
393
        graphs = []
 
394
        for source in self._revision_sources:
 
395
            ghost_graph = source.get_revision_graph_with_ghosts()
 
396
            graphs.append(ghost_graph.get_ancestors())
 
397
        for revision_id in revision_ids:
 
398
            absent = 0
 
399
            for graph in graphs:
 
400
                    if not revision_id in graph:
 
401
                        absent += 1
 
402
            if absent == len(graphs):
 
403
                raise errors.NoSuchRevision(self._revision_sources[0],
 
404
                                            revision_id)
 
405
 
 
406
        # combine the graphs
 
407
        result = Graph()
 
408
        pending = set(revision_ids)
 
409
        done = set()
 
410
        def find_parents(node_id):
 
411
            """find the parents for node_id."""
 
412
            for graph in graphs:
 
413
                try:
 
414
                    return graph[node_id]
 
415
                except KeyError:
 
416
                    pass
 
417
            raise errors.NoSuchRevision(self._revision_sources[0], node_id)
 
418
        while len(pending):
 
419
            # all the graphs should have identical parent lists
 
420
            node_id = pending.pop()
 
421
            try:
 
422
                parents = find_parents(node_id)
 
423
                for parent_node in parents:
 
424
                    # queued or done? 
 
425
                    if (parent_node not in pending and
 
426
                        parent_node not in done):
 
427
                        # no, queue
 
428
                        pending.add(parent_node)
 
429
                result.add_node(node_id, parents)
 
430
                done.add(node_id)
 
431
            except errors.NoSuchRevision:
 
432
                # ghost
 
433
                result.add_ghost(node_id)
309
434
                continue
310
 
            if revision_history is not None and anc_id in revision_history:
311
 
                return anc_id
312
 
            elif best is None or distance > best[1]:
313
 
                best = (anc_id, distance)
314
 
        return best[0]
315
 
 
316
 
    next = rev_id
317
 
    path = []
318
 
    while next != ancestor_id:
319
 
        path.append(next)
320
 
        next = best_ancestor(next)
321
 
    path.reverse()
322
 
    return path
 
435
        return result
 
436
 
 
437
    def lock_read(self):
 
438
        for source in self._revision_sources:
 
439
            source.lock_read()
 
440
 
 
441
    def unlock(self):
 
442
        for source in self._revision_sources:
 
443
            source.unlock()
 
444
 
 
445
 
 
446
def is_reserved_id(revision_id):
 
447
    """Determine whether a revision id is reserved
 
448
 
 
449
    :return: True if the revision is is reserved, False otherwise
 
450
    """
 
451
    return isinstance(revision_id, basestring) and revision_id.endswith(':')
 
452
 
 
453
 
 
454
def check_not_reserved_id(revision_id):
 
455
    """Raise ReservedId if the supplied revision_id is reserved"""
 
456
    if is_reserved_id(revision_id):
 
457
        raise errors.ReservedId(revision_id)
 
458
 
 
459
 
 
460
def ensure_null(revision_id):
 
461
    """Ensure only NULL_REVISION is used to represent the null revision"""
 
462
    if revision_id is None:
 
463
        symbol_versioning.warn('NULL_REVISION should be used for the null'
 
464
            ' revision instead of None, as of bzr 0.91.',
 
465
            DeprecationWarning, stacklevel=2)
 
466
        return NULL_REVISION
 
467
    else:
 
468
        return revision_id
 
469
 
 
470
 
 
471
def is_null(revision_id):
 
472
    if revision_id is None:
 
473
        symbol_versioning.warn('NULL_REVISION should be used for the null'
 
474
            ' revision instead of None, as of bzr 0.90.',
 
475
            DeprecationWarning, stacklevel=2)
 
476
    return revision_id in (None, NULL_REVISION)