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

Re: [PATCH] setup.c: fix comment about order of .git directory discovery

From
Jeff King <peff@peff.net>
Date
Dec 8, 2017, 10:02 UTC
Message-ID
<20171208100214.GB1899@sigill.intra.peff.net>
In-Reply-To
<20171207085949.19519-1-szeder.dev@gmail.com>
On Thu, Dec 07, 2017 at 09:59:49AM +0100, SZEDER Gábor wrote:
Show 5 quoted lines
> Since gitfiles were introduced in b44ebb19e (Add platform-independent
> .git "symlink", 2008-02-20) the order of checks during .git directory
> discovery is: gitfile, gitdir, bare repo.  However, that commit did
> only partially update the in-code comment describing this order,
> missing the last line which still puts gitdir before gitfile.

I was confused at first, because your patch only changes to ".git/" to ".git". So there is no ordering change, only a swapping out of one for the other. But that pushes done the ".git/" into the "etc." which is below.

So I think this makes sense.
Thanks.
-Peff
Previous: SZEDER Gábor
Message 2 of 2 in “setup.c: fix comment about order of .git directory discovery”
  1. setup.c: fix comment about order of .git directory discoverySZEDER Gábor, Dec 7, 2017
  2. Jeff KingDec 8, 2017

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.