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

Re: [PATCH v3] config: support remote name in includeIf.hasconfig condition

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 3, 2024, 07:28 UTC
Message-ID
<xmqq8qsxcjjp.fsf@gitster.g>
In-Reply-To
<20241020173216.40852-2-ken@kmatsui.me>
Ken Matsui <ken@kmatsui.me> writes:
Show 8 quoted lines
> Changes in v3:
>
> * Updated the description based on Ramsay's review.
> * Added more tests.
>
> Changes in v2:
>
> * Updated the description based on Kristoffer's review.

This has never seen any serious review since its original iterations were posted late October. Anybody interested [*]?

Otherwise since it is approaching to be more than a month and a half old, I am going to mark the topic to be discarded.

Thanks.
[Footnote]
 * I ask because apparently I am not interested at all.  At least
   not yet, with what I saw in the previous postings in this thread.
   My take on it is that the "if any remote has this URL" change
   earlier did make some sort of sense, but I do not see the utility
   of this change.  If we need to access both of these two remote
   repositories, "to access github, we need these definition, and to
   access k.org, we need those, so include them" do make sense, but
   because these include do *not* limit their effectiveness upon
   which remote each Git operation accesses (i.e. You cannot say
   "use this configuration setting while running 'git fetch origin'
   but use that other setting while running 'git push github'"), it
   does not make much sense to say "if 'origin' is X, then do this",
   at least to me.
Previous: Ken Matsui
Message 10 of 10 in “config: support remote name in includeIf.hasconfig condition”
  1. config: support remote name in includeIf.hasconfig conditionKen Matsui, Oct 20, 2024
  2. Kristoffer HaugsbakkOct 20, 2024
  3. Ken MatsuiOct 20, 2024
  4. config: support remote name in includeIf.hasconfig conditionKen Matsui, Oct 20, 2024
  5. Ramsay JonesOct 20, 2024
  6. Ken MatsuiOct 20, 2024
  7. Ken MatsuiOct 20, 2024
  8. config: support remote name in includeIf.hasconfig conditionKen Matsui, Oct 20, 2024
  9. Ken MatsuiNov 6, 2024
  10. Junio C HamanoDec 3, 2024

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.