184
by mbp at sourcefrog
pychecker fixups |
1 |
# (C) 2005 Canonical
|
1
by mbp at sourcefrog
import from baz patch-364 |
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 |
||
1185.16.40
by Martin Pool
todo |
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.
|
|
1
by mbp at sourcefrog
import from baz patch-364 |
19 |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
20 |
import bzrlib.errors |
974.1.87
by Aaron Bentley
Refactored and documented graph stuff |
21 |
from bzrlib.graph import node_distances, select_farthest, all_descendants |
1185.16.39
by Martin Pool
- constraints on revprops |
22 |
from bzrlib.osutils import contains_whitespace |
1534.9.1
by Aaron Bentley
Added progress bars to merge |
23 |
from bzrlib.progress import DummyProgress |
8
by mbp at sourcefrog
store committer's timezone in revision and show |
24 |
|
974.1.90
by Aaron Bentley
Switched NULL revision ID to 'null:' per robertc's suggestion |
25 |
NULL_REVISION="null:" |
974.1.89
by Aaron Bentley
Fixed merging with multiple roots, by using null as graph root. |
26 |
|
802
by Martin Pool
- Remove XMLMixin class in favour of simple pack_xml, unpack_xml functions |
27 |
class Revision(object): |
1
by mbp at sourcefrog
import from baz patch-364 |
28 |
"""Single revision on a branch.
|
29 |
||
30 |
Revisions may know their revision_hash, but only once they've been
|
|
31 |
written out. This is not stored because you cannot write the hash
|
|
32 |
into the file it describes.
|
|
33 |
||
697
by Martin Pool
- write out parent list for new revisions |
34 |
After bzr 0.0.5 revisions are allowed to have multiple parents.
|
696
by Martin Pool
- Refactor revision deserialization code |
35 |
|
1313
by Martin Pool
- rename to Revision.parent_ids to avoid confusion with old usage |
36 |
parent_ids
|
37 |
List of parent revision_ids
|
|
1185.16.39
by Martin Pool
- constraints on revprops |
38 |
|
39 |
properties
|
|
40 |
Dictionary of revision properties. These are attached to the
|
|
41 |
revision as extra metadata. The name must be a single
|
|
42 |
word; the value can be an arbitrary string.
|
|
1
by mbp at sourcefrog
import from baz patch-364 |
43 |
"""
|
696
by Martin Pool
- Refactor revision deserialization code |
44 |
|
1185.16.35
by Martin Pool
- stub for revision properties |
45 |
def __init__(self, revision_id, properties=None, **args): |
1092.2.25
by Robert Collins
support ghosts in commits |
46 |
self.revision_id = revision_id |
1185.16.35
by Martin Pool
- stub for revision properties |
47 |
self.properties = properties or {} |
1185.16.39
by Martin Pool
- constraints on revprops |
48 |
self._check_properties() |
1313
by Martin Pool
- rename to Revision.parent_ids to avoid confusion with old usage |
49 |
self.parent_ids = [] |
1311
by Martin Pool
- remove RevisionReference; just hold parent ids directly |
50 |
self.parent_sha1s = [] |
1185.42.6
by Jelmer Vernooij
Don't clear Revision.parent_ids after it has been set from the arguments |
51 |
self.__dict__.update(args) |
696
by Martin Pool
- Refactor revision deserialization code |
52 |
|
1
by mbp at sourcefrog
import from baz patch-364 |
53 |
def __repr__(self): |
184
by mbp at sourcefrog
pychecker fixups |
54 |
return "<Revision id %s>" % self.revision_id |
1
by mbp at sourcefrog
import from baz patch-364 |
55 |
|
1185
by Martin Pool
- add xml round-trip test for revisions |
56 |
def __eq__(self, other): |
57 |
if not isinstance(other, Revision): |
|
58 |
return False |
|
1092.2.20
by Robert Collins
symlink and weaves, whaddya know |
59 |
# FIXME: rbc 20050930 parent_ids are not being compared
|
60 |
return ( |
|
61 |
self.inventory_sha1 == other.inventory_sha1 |
|
1185
by Martin Pool
- add xml round-trip test for revisions |
62 |
and self.revision_id == other.revision_id |
63 |
and self.timestamp == other.timestamp |
|
64 |
and self.message == other.message |
|
65 |
and self.timezone == other.timezone |
|
1185.16.35
by Martin Pool
- stub for revision properties |
66 |
and self.committer == other.committer |
67 |
and self.properties == other.properties) |
|
1185
by Martin Pool
- add xml round-trip test for revisions |
68 |
|
69 |
def __ne__(self, other): |
|
70 |
return not self.__eq__(other) |
|
71 |
||
1185.16.39
by Martin Pool
- constraints on revprops |
72 |
def _check_properties(self): |
73 |
"""Verify that all revision properties are OK.
|
|
74 |
"""
|
|
75 |
for name, value in self.properties.iteritems(): |
|
76 |
if not isinstance(name, basestring) or contains_whitespace(name): |
|
77 |
raise ValueError("invalid property name %r" % name) |
|
78 |
if not isinstance(value, basestring): |
|
79 |
raise ValueError("invalid property value %r for %r" % |
|
80 |
(name, value)) |
|
81 |
||
1534.4.49
by Robert Collins
Provide a revision.get_history(repository) method for generating a synthetic revision history. |
82 |
def get_history(self, repository): |
83 |
"""Return the canonical line-of-history for this revision.
|
|
84 |
||
85 |
If ghosts are present this may differ in result from a ghost-free
|
|
86 |
repository.
|
|
87 |
"""
|
|
88 |
current_revision = self |
|
89 |
reversed_result = [] |
|
90 |
while current_revision is not None: |
|
91 |
reversed_result.append(current_revision.revision_id) |
|
92 |
if not len (current_revision.parent_ids): |
|
93 |
reversed_result.append(None) |
|
94 |
current_revision = None |
|
95 |
else: |
|
96 |
next_revision_id = current_revision.parent_ids[0] |
|
97 |
current_revision = repository.get_revision(next_revision_id) |
|
98 |
reversed_result.reverse() |
|
99 |
return reversed_result |
|
100 |
||
1268
by Martin Pool
- is_ancestor now works by looking at the Branch's stored ancestry |
101 |
|
102 |
def is_ancestor(revision_id, candidate_id, branch): |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
103 |
"""Return true if candidate_id is an ancestor of revision_id.
|
1268
by Martin Pool
- is_ancestor now works by looking at the Branch's stored ancestry |
104 |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
105 |
A false negative will be returned if any intermediate descendent of
|
106 |
candidate_id is not present in any of the revision_sources.
|
|
810
by Martin Pool
- New validate_revision_id function |
107 |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
108 |
revisions_source is an object supporting a get_revision operation that
|
109 |
behaves like Branch's.
|
|
110 |
"""
|
|
1185.67.2
by Aaron Bentley
Renamed Branch.storage to Branch.repository |
111 |
return candidate_id in branch.repository.get_ancestry(revision_id) |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
112 |
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
113 |
|
114 |
def iter_ancestors(revision_id, revision_source, only_present=False): |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
115 |
ancestors = (revision_id,) |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
116 |
distance = 0 |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
117 |
while len(ancestors) > 0: |
118 |
new_ancestors = [] |
|
119 |
for ancestor in ancestors: |
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
120 |
if not only_present: |
121 |
yield ancestor, distance |
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
122 |
try: |
123 |
revision = revision_source.get_revision(ancestor) |
|
124 |
except bzrlib.errors.NoSuchRevision, e: |
|
125 |
if e.revision == revision_id: |
|
126 |
raise
|
|
127 |
else: |
|
128 |
continue
|
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
129 |
if only_present: |
130 |
yield ancestor, distance |
|
1313
by Martin Pool
- rename to Revision.parent_ids to avoid confusion with old usage |
131 |
new_ancestors.extend(revision.parent_ids) |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
132 |
ancestors = new_ancestors |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
133 |
distance += 1 |
134 |
||
135 |
||
136 |
def find_present_ancestors(revision_id, revision_source): |
|
1133
by Martin Pool
doc |
137 |
"""Return the ancestors of a revision present in a branch.
|
138 |
||
139 |
It's possible that a branch won't have the complete ancestry of
|
|
140 |
one of its revisions.
|
|
141 |
||
142 |
"""
|
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
143 |
found_ancestors = {} |
144 |
anc_iter = enumerate(iter_ancestors(revision_id, revision_source, |
|
145 |
only_present=True)) |
|
146 |
for anc_order, (anc_id, anc_distance) in anc_iter: |
|
147 |
if not found_ancestors.has_key(anc_id): |
|
148 |
found_ancestors[anc_id] = (anc_order, anc_distance) |
|
149 |
return found_ancestors |
|
150 |
||
1153
by Martin Pool
- clean up some code in revision.py |
151 |
|
152 |
def __get_closest(intersection): |
|
153 |
intersection.sort() |
|
154 |
matches = [] |
|
155 |
for entry in intersection: |
|
156 |
if entry[0] == intersection[0][0]: |
|
157 |
matches.append(entry[2]) |
|
158 |
return matches |
|
159 |
||
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
160 |
|
974.1.65
by Aaron Bentley
Cleanup and test-fixing |
161 |
def old_common_ancestor(revision_a, revision_b, revision_source): |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
162 |
"""Find the ancestor common to both revisions that is closest to both.
|
163 |
"""
|
|
164 |
from bzrlib.trace import mutter |
|
165 |
a_ancestors = find_present_ancestors(revision_a, revision_source) |
|
166 |
b_ancestors = find_present_ancestors(revision_b, revision_source) |
|
167 |
a_intersection = [] |
|
168 |
b_intersection = [] |
|
169 |
# a_order is used as a tie-breaker when two equally-good bases are found
|
|
170 |
for revision, (a_order, a_distance) in a_ancestors.iteritems(): |
|
171 |
if b_ancestors.has_key(revision): |
|
172 |
a_intersection.append((a_distance, a_order, revision)) |
|
173 |
b_intersection.append((b_ancestors[revision][1], a_order, revision)) |
|
1185.31.4
by John Arbash Meinel
Fixing mutter() calls to not have to do string processing. |
174 |
mutter("a intersection: %r", a_intersection) |
175 |
mutter("b intersection: %r", b_intersection) |
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
176 |
|
1153
by Martin Pool
- clean up some code in revision.py |
177 |
a_closest = __get_closest(a_intersection) |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
178 |
if len(a_closest) == 0: |
179 |
return None |
|
1153
by Martin Pool
- clean up some code in revision.py |
180 |
b_closest = __get_closest(b_intersection) |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
181 |
assert len(b_closest) != 0 |
1185.31.4
by John Arbash Meinel
Fixing mutter() calls to not have to do string processing. |
182 |
mutter ("a_closest %r", a_closest) |
183 |
mutter ("b_closest %r", b_closest) |
|
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
184 |
if a_closest[0] in b_closest: |
185 |
return a_closest[0] |
|
186 |
elif b_closest[0] in a_closest: |
|
187 |
return b_closest[0] |
|
188 |
else: |
|
1154
by Martin Pool
- fix imports for moved errors |
189 |
raise bzrlib.errors.AmbiguousBase((a_closest[0], b_closest[0])) |
974.1.35
by aaron.bentley at utoronto
Added revision-based common-ancestor checking |
190 |
return a_closest[0] |
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
191 |
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
192 |
def revision_graph(revision, revision_source): |
974.1.66
by Aaron Bentley
more cleanups, docs, sorting stuff |
193 |
"""Produce a graph of the ancestry of the specified revision.
|
194 |
Return root, ancestors map, descendants map
|
|
195 |
||
196 |
TODO: Produce graphs with the NULL revision as root, so that we can find
|
|
197 |
a common even when trees are not branches don't represent a single line
|
|
198 |
of descent.
|
|
1482
by Robert Collins
'revision.revision_graph can handle having only partial history. |
199 |
RBC: 20051024: note that when we have two partial histories, this may not
|
200 |
be possible. But if we are willing to pretend :)... sure.
|
|
974.1.66
by Aaron Bentley
more cleanups, docs, sorting stuff |
201 |
"""
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
202 |
ancestors = {} |
203 |
descendants = {} |
|
204 |
lines = [revision] |
|
205 |
root = None |
|
974.1.61
by abentley
fixes for common ancestor picker |
206 |
descendants[revision] = {} |
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
207 |
while len(lines) > 0: |
208 |
new_lines = set() |
|
209 |
for line in lines: |
|
974.1.89
by Aaron Bentley
Fixed merging with multiple roots, by using null as graph root. |
210 |
if line == NULL_REVISION: |
211 |
parents = [] |
|
212 |
root = NULL_REVISION |
|
213 |
else: |
|
214 |
try: |
|
215 |
rev = revision_source.get_revision(line) |
|
1390
by Robert Collins
pair programming worx... merge integration and weave |
216 |
parents = list(rev.parent_ids) |
974.1.89
by Aaron Bentley
Fixed merging with multiple roots, by using null as graph root. |
217 |
if len(parents) == 0: |
218 |
parents = [NULL_REVISION] |
|
219 |
except bzrlib.errors.NoSuchRevision: |
|
220 |
if line == revision: |
|
221 |
raise
|
|
222 |
parents = None |
|
974.1.64
by Aaron Bentley
Handled ancestors that are missing when finding a base |
223 |
if parents is not None: |
224 |
for parent in parents: |
|
225 |
if parent not in ancestors: |
|
226 |
new_lines.add(parent) |
|
227 |
if parent not in descendants: |
|
228 |
descendants[parent] = {} |
|
229 |
descendants[parent][line] = 1 |
|
230 |
if parents is not None: |
|
231 |
ancestors[line] = set(parents) |
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
232 |
lines = new_lines |
1482
by Robert Collins
'revision.revision_graph can handle having only partial history. |
233 |
if root is None: |
234 |
# The history for revision becomes inaccessible without
|
|
235 |
# actually hitting a no-parents revision. This then
|
|
236 |
# makes these asserts below trigger. So, if root is None
|
|
237 |
# determine the actual root by walking the accessible tree
|
|
238 |
# and then stash NULL_REVISION at the end.
|
|
239 |
root = NULL_REVISION |
|
240 |
descendants[root] = {} |
|
241 |
# for every revision, check we can access at least
|
|
242 |
# one parent, if we cant, add NULL_REVISION and
|
|
243 |
# a link
|
|
244 |
for rev in ancestors: |
|
245 |
if len(ancestors[rev]) == 0: |
|
246 |
raise RuntimeError('unreachable code ?!') |
|
247 |
ok = False |
|
248 |
for parent in ancestors[rev]: |
|
249 |
if parent in ancestors: |
|
250 |
ok = True |
|
251 |
if ok: |
|
252 |
continue
|
|
253 |
descendants[root][rev] = 1 |
|
254 |
ancestors[rev].add(root) |
|
255 |
ancestors[root] = set() |
|
974.1.63
by Aaron Bentley
Fixed graph-generation |
256 |
assert root not in descendants[root] |
257 |
assert root not in ancestors[root] |
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
258 |
return root, ancestors, descendants |
259 |
||
1092.3.4
by Robert Collins
update symlink branch to integration |
260 |
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
261 |
def combined_graph(revision_a, revision_b, revision_source): |
974.1.66
by Aaron Bentley
more cleanups, docs, sorting stuff |
262 |
"""Produce a combined ancestry graph.
|
263 |
Return graph root, ancestors map, descendants map, set of common nodes"""
|
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
264 |
root, ancestors, descendants = revision_graph(revision_a, revision_source) |
265 |
root_b, ancestors_b, descendants_b = revision_graph(revision_b, |
|
266 |
revision_source) |
|
974.1.80
by Aaron Bentley
Improved merge error handling and testing |
267 |
if root != root_b: |
268 |
raise bzrlib.errors.NoCommonRoot(revision_a, revision_b) |
|
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
269 |
common = set() |
270 |
for node, node_anc in ancestors_b.iteritems(): |
|
271 |
if node in ancestors: |
|
272 |
common.add(node) |
|
273 |
else: |
|
274 |
ancestors[node] = set() |
|
275 |
ancestors[node].update(node_anc) |
|
276 |
for node, node_dec in descendants_b.iteritems(): |
|
277 |
if node not in descendants: |
|
1185.8.1
by Aaron Bentley
Ensured combined_graph is order-insensitive |
278 |
descendants[node] = {} |
974.1.63
by Aaron Bentley
Fixed graph-generation |
279 |
descendants[node].update(node_dec) |
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
280 |
return root, ancestors, descendants, common |
281 |
||
1092.3.4
by Robert Collins
update symlink branch to integration |
282 |
|
1534.9.1
by Aaron Bentley
Added progress bars to merge |
283 |
def common_ancestor(revision_a, revision_b, revision_source, |
284 |
pb=DummyProgress()): |
|
974.1.80
by Aaron Bentley
Improved merge error handling and testing |
285 |
try: |
1534.9.1
by Aaron Bentley
Added progress bars to merge |
286 |
try: |
287 |
pb.update('Picking ancestor', 1, 3) |
|
288 |
root, ancestors, descendants, common = \ |
|
289 |
combined_graph(revision_a, revision_b, revision_source) |
|
290 |
except bzrlib.errors.NoCommonRoot: |
|
291 |
raise bzrlib.errors.NoCommonAncestor(revision_a, revision_b) |
|
292 |
||
293 |
pb.update('Picking ancestor', 2, 3) |
|
294 |
distances = node_distances (descendants, ancestors, root) |
|
295 |
pb.update('Picking ancestor', 3, 2) |
|
296 |
farthest = select_farthest(distances, common) |
|
297 |
if farthest is None or farthest == NULL_REVISION: |
|
298 |
raise bzrlib.errors.NoCommonAncestor(revision_a, revision_b) |
|
299 |
finally: |
|
300 |
pb.clear() |
|
974.1.87
by Aaron Bentley
Refactored and documented graph stuff |
301 |
return farthest |
974.1.60
by aaron.bentley at utoronto
Initial import of common-ancestor detection |
302 |
|
303 |
||
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
304 |
class MultipleRevisionSources(object): |
1153
by Martin Pool
- clean up some code in revision.py |
305 |
"""Proxy that looks in multiple branches for revisions."""
|
974.1.26
by aaron.bentley at utoronto
merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 |
306 |
def __init__(self, *args): |
307 |
object.__init__(self) |
|
308 |
assert len(args) != 0 |
|
309 |
self._revision_sources = args |
|
310 |
||
311 |
def get_revision(self, revision_id): |
|
312 |
for source in self._revision_sources: |
|
313 |
try: |
|
314 |
return source.get_revision(revision_id) |
|
315 |
except bzrlib.errors.NoSuchRevision, e: |
|
316 |
pass
|
|
317 |
raise e |
|
974.2.7
by aaron.bentley at utoronto
Merged from bzr.24 |
318 |
|
319 |
def get_intervening_revisions(ancestor_id, rev_id, rev_source, |
|
320 |
revision_history=None): |
|
321 |
"""Find the longest line of descent from maybe_ancestor to revision.
|
|
322 |
Revision history is followed where possible.
|
|
323 |
||
324 |
If ancestor_id == rev_id, list will be empty.
|
|
325 |
Otherwise, rev_id will be the last entry. ancestor_id will never appear.
|
|
326 |
If ancestor_id is not an ancestor, NotAncestor will be thrown
|
|
327 |
"""
|
|
974.1.73
by Aaron Bentley
Reimplemented get_intervening_revisions for better scalability |
328 |
root, ancestors, descendants = revision_graph(rev_id, rev_source) |
329 |
if len(descendants) == 0: |
|
330 |
raise NoSuchRevision(rev_source, rev_id) |
|
331 |
if ancestor_id not in descendants: |
|
332 |
rev_source.get_revision(ancestor_id) |
|
333 |
raise bzrlib.errors.NotAncestor(rev_id, ancestor_id) |
|
334 |
root_descendants = all_descendants(descendants, ancestor_id) |
|
335 |
root_descendants.add(ancestor_id) |
|
336 |
if rev_id not in root_descendants: |
|
337 |
raise bzrlib.errors.NotAncestor(rev_id, ancestor_id) |
|
338 |
distances = node_distances(descendants, ancestors, ancestor_id, |
|
339 |
root_descendants=root_descendants) |
|
340 |
||
341 |
def best_ancestor(rev_id): |
|
342 |
best = None |
|
343 |
for anc_id in ancestors[rev_id]: |
|
344 |
try: |
|
345 |
distance = distances[anc_id] |
|
346 |
except KeyError: |
|
347 |
continue
|
|
348 |
if revision_history is not None and anc_id in revision_history: |
|
349 |
return anc_id |
|
350 |
elif best is None or distance > best[1]: |
|
351 |
best = (anc_id, distance) |
|
352 |
return best[0] |
|
353 |
||
354 |
next = rev_id |
|
355 |
path = [] |
|
356 |
while next != ancestor_id: |
|
357 |
path.append(next) |
|
358 |
next = best_ancestor(next) |
|
359 |
path.reverse() |
|
360 |
return path |