Re: [PATCH v3 24/24] midx: clear midx on repack

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Jul 6, 2018, 05:52 UTC
Message-ID: <CAPig+cRm0VykJWvMLqaKDDRk1mcG_kDDPeo64jJCoVZ1i+k9wg@mail.gmail.com>
In-Reply-To: <20180706005321.124643-25-dstolee@microsoft.com>

On Thu, Jul 5, 2018 at 8:54 PM Derrick Stolee <stolee@gmail.com> wrote:

Show 10 quoted lines

> If a 'git repack' command replaces existing packfiles, then we must
> clear the existing multi-pack-index before moving the packfiles it
> references.
>
> Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
> ---
>  builtin/repack.c |  8 ++++++++
>  midx.c           | 12 ++++++++++++
>  midx.h           |  1 +
>  3 files changed, 21 insertions(+)

This seems like a pretty important bit of functionality. Is there any way to add a test of it to t5319 or would it be too difficult (or not important enough)?

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.