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

Re: Re: [PATCH] builtin/gc: warn when core.commitGraph is disabled

From
Llilinchao@oschina.cn <lilinchao@oschina.cn>
Date
May 13, 2021, 08:17 UTC
Message-ID
<b87b12b4b3c311eba1fd0024e87935e7@oschina.cn>
In-Reply-To
<87tunau7ia.fsf@evledraar.gmail.com>
Show 17 quoted lines
>
>On Mon, May 10 2021, lilinchao@oschina.cn wrote:
>
>> From: Li Linchao <lilinchao@oschina.cn>
>>
>> Throw warning message when core.commitGraph is disabled in commit-graph
>> maintenance task.
>
>Won't this cause the gc.log issue noted in
>https://lore.kernel.org/git/87r1l27rae.fsf@evledraar.gmail.com/
>
>More importantly, I don't think this UX makes sense. We said we didn't
>want it, so why warn about it?
>
>Maybe there are good reasons to, but this commit message / patch doesn't
>make the case for it...
>
Uh, well, maybe I should argue for this patch a bit more.
First this is in git maintenance task, I've read the link you post, and I feel it has nothing to do with maintenance task.
Second I hope the `commit-graph` task can do the same thing with `incremental repack` task that to warn user when the related necessary setting is not yet ready, instead of running quietly, but doing nothing.
Thanks
Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 3 of 6 in “builtin/gc: warn when core.commitGraph is disabled”
  1. builtin/gc: warn when core.commitGraph is disabledlilinchao@oschina.cn, May 10, 2021
  2. Ævar Arnfjörð BjarmasonMay 10, 2021
  3. lilinchao@oschina.cnMay 13, 2021
  4. Ævar Arnfjörð BjarmasonMay 13, 2021
  5. Junio C HamanoMay 10, 2021
  6. lilinchao@oschina.cnMay 11, 2021

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.