Merge lp:~cjwatson/brz-loom/breezy-3.0.0 into lp:brz-loom

Proposed by Colin Watson
Status: Merged
Merged at revision: 164
Proposed branch: lp:~cjwatson/brz-loom/breezy-3.0.0
Merge into: lp:brz-loom
Diff against target: 62 lines (+9/-10)
1 file modified
branch.py (+9/-10)
To merge this branch: bzr merge lp:~cjwatson/brz-loom/breezy-3.0.0
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+368786@code.launchpad.net

Commit message

Update to Breezy 3.0.0.

Description of the change

There were a couple of breaking API changes in Breezy since the initial brz-loom port that didn't get handled.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Thanks!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'branch.py'
--- branch.py 2018-05-20 14:06:26 +0000
+++ branch.py 2019-06-13 16:18:52 +0000
@@ -33,7 +33,6 @@
33 commit as _mod_commit,33 commit as _mod_commit,
34 controldir,34 controldir,
35 errors,35 errors,
36 fetch as _mod_fetch,
37 symbol_versioning,36 symbol_versioning,
38 trace,37 trace,
39 ui,38 ui,
@@ -42,6 +41,7 @@
42from breezy.bzr import (41from breezy.bzr import (
43 branch as _mod_bzrbranch,42 branch as _mod_bzrbranch,
44 bzrdir,43 bzrdir,
44 fetch as _mod_fetch,
45 inventory as _mod_inventory,45 inventory as _mod_inventory,
46 inventorytree as _mod_inventorytree,46 inventorytree as _mod_inventorytree,
47 remote,47 remote,
@@ -174,27 +174,26 @@
174 self._loom_stream = loom_stream174 self._loom_stream = loom_stream
175 self._loom_sha1 = loom_sha1175 self._loom_sha1 = loom_sha1
176 176
177 def get_file(self, path, file_id):177 def get_file(self, path):
178 """Get the content of file_id from this tree.178 """Get the content of 'path' from this tree.
179179
180 As usual this must be for the single existing file 'loom'.180 As usual this must be for the single existing file 'loom'.
181 """181 """
182 return self._loom_stream182 return self._loom_stream
183183
184 def get_file_with_stat(self, path, file_id=None):184 def get_file_with_stat(self, path):
185 return (self.get_file(path, file_id), None)185 return (self.get_file(path), None)
186186
187 def get_file_sha1(self, path, file_id, stat_value=None):187 def get_file_sha1(self, path, stat_value=None):
188 """Get the sha1 for a file. 188 """Get the sha1 for a file.
189189
190 This tree only has one file, so it MUST be present!190 This tree only has one file, so it MUST be present!
191 """191 """
192 assert path == 'loom'192 assert path == 'loom'
193 assert file_id == 'loom_meta_tree'
194 return self._loom_sha1193 return self._loom_sha1
195194
196 def is_executable(self, file_id, path):195 def is_executable(self, path):
197 """get the executable status for file_id.196 """get the executable status for 'path'.
198 197
199 Nothing in a LoomMetaTree is executable.198 Nothing in a LoomMetaTree is executable.
200 """199 """
@@ -364,7 +363,7 @@
364 if revisionid is empty:, this is a new, empty branch.363 if revisionid is empty:, this is a new, empty branch.
365 """364 """
366 tree = self.repository.revision_tree(rev_id)365 tree = self.repository.revision_tree(rev_id)
367 lines = tree.get_file('loom', 'loom_meta_tree').read().split('\n')366 lines = tree.get_file('loom').read().split('\n')
368 assert lines[0] == 'Loom meta 1'367 assert lines[0] == 'Loom meta 1'
369 return lines[1:-1]368 return lines[1:-1]
370369

Subscribers

People subscribed via source and target branches

to all changes: