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

Re: [filter-repo PATCH] Don't crash on multi-line config values

From
THToke Høiland-Jørgensen <toke@toke.dk>
Date
Jan 28, 2025, 18:06 UTC
Message-ID
<87sep2olti.fsf@toke.dk>
In-Reply-To
<20241214181306.296673-1-toke@toke.dk>
Toke Høiland-Jørgensen <toke@toke.dk> writes:
Show 5 quoted lines
> The parsing of the output of `git config --list` fails if any of the
> config values contain newlines. Fix this by using the --null parameter
> to `git config`, which is designed for this purpose.
>
> Add a simple test that causes the crash pre-patch.
Hi Elijah

Any chance you could merge this? Seems a couple of others found this bug as well, there are two open PRs on github with alternative fixes for the same issue...

-Toke
Previous: Toke Høiland-JørgensenNext: Elijah Newren
Message 2 of 3 in “Don't crash on multi-line config values”
  1. Don't crash on multi-line config valuesToke Høiland-Jørgensen, Dec 14, 2024
  2. Toke Høiland-JørgensenJan 28, 2025
  3. Elijah NewrenJan 28, 2025

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.