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

Re: [PATCH 1/3] doc/git-config: explain --file instead of referring to GIT_CONFIG

From
Taylor Blau <me@ttaylorr.com>
Date
Jul 14, 2021, 22:26 UTC
Message-ID
<YO9kfhr3u/Xx1FQc@nand.local>
In-Reply-To
<YO9j6hrJFFTSEd6j@coredump.intra.peff.net>
On Wed, Jul 14, 2021 at 06:23:38PM -0400, Jeff King wrote:
Show 7 quoted lines
> On Wed, Jul 14, 2021 at 06:20:30PM -0400, Taylor Blau wrote:
> > Perhaps `.git/config` is clear enough; I definitely don't feel strongly
> > about it.
>
> Yes, I noticed that, too. This is copied verbatim from the other
> options, so if we were to do that, it should be done on top across the
> whole file.

I'm dubious that s,.git/config,$GIT_DIR/config,g would be an improvement in clarity if done across the whole file, and I'd rather be consistent than not. So I think that it's fine as is.

Thanks, Taylor

Previous: Jeff KingNext: Martin Ågren
Message 9 of 16 in “Bug report: GIT_CONFIG and user.email/name”
  1. German LashevichJul 14, 2021
  2. Jeff KingJul 14, 2021
  3. German LashevichJul 14, 2021
  4. Junio C HamanoJul 14, 2021
  5. Jeff KingJul 14, 2021
  6. 1/3 doc/git-config: explain --file instead of referring to GIT_CONFIGJeff King, Jul 14, 2021
  7. Taylor BlauJul 14, 2021
  8. Jeff KingJul 14, 2021
  9. Taylor BlauJul 14, 2021
  10. Martin ÅgrenJul 20, 2021
  11. Jeff KingJul 20, 2021
  12. Jeff KingJul 20, 2021
  13. Junio C HamanoJul 20, 2021
  14. 2/3 doc/git-config: clarify GIT_CONFIG environment variableJeff King, Jul 14, 2021
  15. 3/3 doc/git-config: simplify "override" advice for FILES sectionJeff King, Jul 14, 2021
  16. Taylor BlauJul 14, 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.