> >On Mon, May 10 2021, lilinchao@xxxxxxxxxx wrote: > >> From: Li Linchao <lilinchao@xxxxxxxxxx> >> >> 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@xxxxxxxxxxxxxxxxxxx/ > >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
