John - that code uses None to mean 'copy all items'. Remember that Packer /had/ two use cases:
- fetch by making a pack
- packing
When doing a pack, self._revision_keys is None, and all inventories are preserved (in knit-style pack repos). When fetching by making a pack, we want to copy only those revisions.
This is essentially dead code now though, as we no longer fetch using packer, do we?
John - that code uses None to mean 'copy all items'. Remember that Packer /had/ two use cases:
- fetch by making a pack
- packing
When doing a pack, self._revision_keys is None, and all inventories are preserved (in knit-style pack repos). When fetching by making a pack, we want to copy only those revisions.
This is essentially dead code now though, as we no longer fetch using packer, do we?