Re: [PATCH 01/11] multi-pack-index: add 'verify' verb
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Sep 5, 2018, 18:59 UTC
- Message-ID
- <CAPig+cR4vEcMBe4REd4Z=WMG-MR4PUnyNNBxoJacUXzB8ho6fg@mail.gmail.com>
- In-Reply-To
- <8dc38afe2b4d73e940daf9a0ff7f9c0a38802d95.1536158789.git.gitgitgadget@gmail.com>
On Wed, Sep 5, 2018 at 10:46 AM Derrick Stolee via GitGitGadget <gitgitgadget@gmail.com> wrote:
Show 11 quoted lines
> The multi-pack-index builtin writes multi-pack-index files, and
> uses a 'write' verb to do so. Add a 'verify' verb that checks this
> file matches the contents of the pack-indexes it replaces.
> [...]
> Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
> ---
> diff --git a/builtin/multi-pack-index.c b/builtin/multi-pack-index.c
> @@ -5,7 +5,7 @@
> static char const * const builtin_multi_pack_index_usage[] = {
> - N_("git multi-pack-index [--object-dir=<dir>] write"),
> + N_("git multi-pack-index [--object-dir=<dir>] [write|verify]"),Nit: The square brackets make the verb optional. You probably want parenthesis to indicate that one or the other is required:
git multi-pack-index [--object-dir=<dir>] (write|verify)