git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] commit-graph: add verify changed paths option

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 31, 2020, 17:14 UTC
Message-ID
<xmqqh7tn4neo.fsf@gitster.c.googlers.com>
In-Reply-To
<pull.687.git.1596181765336.gitgitgadget@gmail.com>
"Son Luong Ngoc via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 5 quoted lines
> From: Son Luong Ngoc <sluongng@gmail.com>
>
> Add '--has-changed-paths' option to 'git commit-graph verify' subcommand
> to validate whether the commit-graph was written with '--changed-paths'
> option.

The implementation seems to be only about "does this section exist?" and not "does this section have healthy/uncorrupted data?", which feels a bit strange for "verify". Instead of setting ourselves up to having to add "--has-this-section" and "--has-that-section" every time a new kind of data is added to the system, how about giving the verify command an option to list all the sections found in the file, or a separate "git commit-graph list-sections" subcommand?

Previous: Christian CouderNext: Taylor Blau
Message 3 of 8 in “commit-graph: add verify changed paths option”
  1. commit-graph: add verify changed paths optionSon Luong Ngoc via GitGitGadget, Jul 31, 2020
  2. Christian CouderJul 31, 2020
  3. Junio C HamanoJul 31, 2020
  4. Taylor BlauJul 31, 2020
  5. Jeff KingJul 31, 2020
  6. Taylor BlauJul 31, 2020
  7. Jeff KingJul 31, 2020
  8. Son Luong NgocJul 31, 2020

Read the whole thread, see it on lore, or plain text.

$ 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.