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

Re: [PATCH 1/2] mergetool: Add tests for filename with whitespace

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 22, 2011, 21:11 UTC
Message-ID
<7vaad9607h.fsf@alter.siamese.dyndns.org>
In-Reply-To
<734F376D-0CF5-4417-8DC2-8A46AA05D995@JonathonMah.com>
Jonathon Mah <me@JonathonMah.com> writes:
> Signed-off-by: Jonathon Mah <me@JonathonMah.com>
> ---
Missing log message. What is the purpose of these new tests?

You found a bug in the current implementation of mergetool (and then fixed with your next patch)? Your earlier unpublished attempts for your next patch did not work for paths with IFS in it, and you wanted to make sure that future changes to your next patch will not be broken?

Previous: Jonathon Mah
Message 2 of 2 in “mergetool: Add tests for filename with whitespace”
  1. 1/2 mergetool: Add tests for filename with whitespaceJonathon Mah, Jun 22, 2011
  2. Junio C HamanoJun 22, 2011

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.