On Wed, 2009-08-12 at 02:03 +0000, John A Meinel wrote:
> We no longer fetch using Packer, correct.
> This code *is* used during pack operations (bzr pack and autopack)
>
> I'll admit to not tracing to see for sure that _revision_keys was None,
> and you could be right.
Conversely I haven't debugged to be confident its not getting
accidentally set in 1.9 formats. It would make sense to me to remove the
support for revision_keys as we're not using it to fetch anymore (though
perhaps reconcilechecker needs it still..)
On Wed, 2009-08-12 at 02:03 +0000, John A Meinel wrote:
> We no longer fetch using Packer, correct.
> This code *is* used during pack operations (bzr pack and autopack)
>
> I'll admit to not tracing to see for sure that _revision_keys was None,
> and you could be right.
Conversely I haven't debugged to be confident its not getting
accidentally set in 1.9 formats. It would make sense to me to remove the
support for revision_keys as we're not using it to fetch anymore (though
perhaps reconcilechecker needs it still..)
-Rob