[RFC] Inventory refactoring (brisbane)

[Robert Collins]

On Mon, 2009-04-06 at 18:03 +1000, Ian Clatworthy wrote:


> As this is arguably the critical path for merging brisbane-core
> and I'm offline tomorrow for medical reasons, can someone else
> please consider this problem, and if they agree with me, write
> these missing tests ASAP?
> 
> Ian C.
> 
> PS: I'm happy with the code that is there and I've spent a lot of
> time reviewing and tweaking it in recent weeks. The issue is the
> stuff which is *not* there.

My inclination is to look at this as we polish the new format; for the
following reasons:
 - the interface tests are not failing without those methods
 - some blackbox tests may need them
 - until CHK based repositories are widely available we run a low risk
   of a method on Inventory not being present on CHKInventory
 - inventories are not the recommended public interface for tree's 
   anyway. [they are there and supported but not the defacto interface]
 - when I checked loggerhead, one of the plugins most likely to need
   all the inventory data, it worked ok.

-Rob
-------------- 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/20090406/46be2bbb/attachment.pgp