Re: [PATCH v2 0/3] diff: fix --exit-code with external diff

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 5, 2024, 16:47 UTC
Message-ID: <xmqqed9buyj5.fsf@gitster.g>
In-Reply-To: <6fa51a62-5dc1-4865-9f79-eaf7d65189ab@web.de>

René Scharfe <l.s.r@web.de> writes:

>   t4020: test exit code with external diffs
>   userdiff: add and use struct external_diff
>   diff: let external diffs report that changes are uninteresting

OK, we now need to mark each external diff driver if we want to honor their exit status, but that awkwardness is to be blamed on the original design. Thanks for addressing the misdesign I made years ago in the mildest way possible not to introduce any unnecessary regressions.

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