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

Re: [PATCH v1 1/2] api-trace2.txt: print config key-value pair

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 25, 2022, 19:07 UTC
Message-ID
<xmqqczdt3tb6.fsf@gitster.g>
In-Reply-To
<220722.86fsits91m.gmgdl@evledraar.gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 30 quoted lines
> On Fri, Jul 22 2022, tenglong.tl wrote:
> ...
>> +Print Configs::
>> +
>> +	  Dump "interesting" config values to trace2 log.
>> ++
>> +The environment variable `GIT_TRACE2_CONFIG_PARAMS` and configuration
>> +`trace2.configparams` can be used to output config values which you care
>> +about(see linkgit:git-config[1). For example:
>
> I didn't notice this before, but this is an addition to a long section
> where the examples are ------- delimited, starting with "in this
> example.." usually.
>
> So this "print configs" seems like on odd continuation. Shouldn't this
> copy the template of "Thread Events::" above. I.e. something like (I
> have not tried to asciidoc render this):
> 	
> 	Config (def param) Events::
> 		We can optionally emit configuration events, see
> 		`trace2.configParams` in linkgit:git-config[1] for how to enable
> 		it.
> 	+
> 	< your example below would follow this>
>
> I.e. re my earlier mention of git-config we it explains
> GIT_TRACE2_CONFIG_PARAMS, so perhaps it suffices to just link to
> linkgit:git-config[1] for that.
>
> Also a nit: trace2.configParams, not trace2.configparams.

These may be worth fixing, regardless of the "adding scope in the example needs to wait until 2/2" I pointed out separately.

Thanks.
Previous: Ævar Arnfjörð BjarmasonNext: tenglong.tl
Message 9 of 22 in “tr2: shows the scope unconditionally with config”
  1. 0/2 tr2: shows the scope unconditionally with configtenglong.tl, Jul 21, 2022
  2. 1/2 api-trace2.txt: print config key-value pairtenglong.tl, Jul 21, 2022
  3. 2/2 tr2: shows scope unconditionally in addition to key-value pairtenglong.tl, Jul 21, 2022
  4. Junio C HamanoJul 21, 2022
  5. tenglong.tlJul 22, 2022
  6. 0/2 tr2: shows the scope unconditionally with configtenglong.tl, Jul 22, 2022
  7. 1/2 api-trace2.txt: print config key-value pairtenglong.tl, Jul 22, 2022
  8. Ævar Arnfjörð BjarmasonJul 22, 2022
  9. Junio C HamanoJul 25, 2022
  10. tenglong.tlAug 1, 2022
  11. Junio C HamanoAug 5, 2022
  12. Teng LongAug 8, 2022
  13. Junio C HamanoJul 22, 2022
  14. tenglong.tlJul 23, 2022
  15. Junio C HamanoJul 23, 2022
  16. tenglong.tlJul 25, 2022
  17. 2/2 tr2: shows scope unconditionally in addition to key-value pairtenglong.tl, Jul 22, 2022
  18. 0/2 tr2: shows the scope unconditionally with configTeng Long, Aug 12, 2022
  19. 1/2 api-trace2.txt: print config key-value pairTeng Long, Aug 12, 2022
  20. 2/2 tr2: shows scope unconditionally in addition to key-value pairTeng Long, Aug 12, 2022
  21. Junio C HamanoAug 12, 2022
  22. Junio C HamanoAug 19, 2022

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.