Re: [PATCH v5 00/30] Builtin FSMonitor Part 2

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 17, 2022, 19:36 UTC
Message-ID: <xmqqpmnl8evc.fsf@gitster.g>
In-Reply-To: <nycvar.QRO.7.76.6.2202171655390.348@tvgsbejvaqbjf.bet>

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

Show 6 quoted lines

>> In this version I removed the core.useBuiltinFSMonitor config setting and
>> instead extended the existing core.fsmonitor.
>
> I am somewhat surprised that a reviewer suggested this, as it breaks the
> common paradigm we use to allow using several Git versions on the same
> worktree.

I do not think sharing the same repository with different versions of Git was considered as a possible source of problems during the review discussion.

https://lore.kernel.org/git/74282d08-aaeb-0a1e-cad3-1de17d59b4d1@jeffhostetler.com/

I am not saying that we should not consider it; I am just stating the fact that there was nobody who raised as a potential issue during the discussion that lead to the cited message.

$ 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.