On Sun, Aug 06, 2017 at 06:47:06PM -0700, Shawn Pearce wrote:

> Changes from v5:
> - extensions.refStorage = reftable is used to select this format.

Thanks, I think this is a better scheme going forward. Just a few notes
on compatibility while I'm thinking about it:

  - existing versions will complain that they don't know what the
    "refStorage" extension is

  - presumably we'd add new code that recognizes the extension, and then
    makes sure the value is something we understand.

  - then we'd finally mark "reftable" as understood once we had an
    implementation. We _could_ then also check other config (like
    "reftable.*") and complain about unknown keys. But I think we could
    declare any such keys as optional, and just rely on the version
    number inside the reftable file for parsing it.

-Peff

Reply via email to