commit in brisbane core?

Robert Collins robert.collins at canonical.com
Thu Apr 2 22:25:33 BST 2009


Could we remove this change please :) its a bogus condition to test for.

         if self.new_inventory is None:
+            # XXX: Using these asserts causes test failures. However, at least
+            #      "self._recording_deletes" seems like a useful check to do,
+            #      as it ensure the delta is completely valid. Most likely this
+            #      just exposes that the test suite isn't using CommitBuilder
+            #      100% correctly.
+            # if (not self.repository._format._commit_inv_deltas
+            #     or not self._recording_deletes):
+            #     raise AssertionError('new_inventory is None, but we did not'
+            #         ' set the flag that the repository format supports'
+            #         ' partial inventory generation.')


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090403/766e8d22/attachment.pgp 


More information about the bazaar mailing list