* commit-graph: avoid message Computing commit graph generation numbers...
@ 2019-11-05 15:40 ryenus
2019-11-05 15:59 ` Derrick Stolee
0 siblings, 1 reply; 2+ messages in thread
From: ryenus @ 2019-11-05 15:40 UTC (permalink / raw)
To: Git mailing list
Hi,
After setting fetch.writeCommitGraph to true, the message
> Computing commit graph generation numbers: 100% (2/2), done.
is always printed.
IMHO this is internal to Git and probably NOT necessary to print by
default, if really necessary, maybe an option can be provided
to turn it on. Thoughts?
Thanks
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: commit-graph: avoid message Computing commit graph generation numbers...
2019-11-05 15:40 commit-graph: avoid message Computing commit graph generation numbers ryenus
@ 2019-11-05 15:59 ` Derrick Stolee
0 siblings, 0 replies; 2+ messages in thread
From: Derrick Stolee @ 2019-11-05 15:59 UTC (permalink / raw)
To: ryenus, Git mailing list
On 11/5/2019 10:40 AM, ryenus wrote:
> Hi,
>
> After setting fetch.writeCommitGraph to true, the message
>> Computing commit graph generation numbers: 100% (2/2), done.
> is always printed.
>
> IMHO this is internal to Git and probably NOT necessary to print by
> default, if really necessary, maybe an option can be provided
> to turn it on. Thoughts?
Thanks for the report! This is a relatively simple mistake, and it
has a very simple fix [1].
I've queued the GitGitGadget PR for submission, so look forward to
that soon.
Thanks!
-Stolee
[1] https://github.com/gitgitgadget/git/pull/450
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-11-05 15:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-11-05 15:40 commit-graph: avoid message Computing commit graph generation numbers ryenus
2019-11-05 15:59 ` Derrick Stolee
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).