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

Re: [PATCH 4/4] t7800: "defaults" is no longer a builtin tool name

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Feb 17, 2013, 11:23 UTC
Message-ID
<20130217112334.GH6759@elie.Belkin>
In-Reply-To
<1360993666-81308-4-git-send-email-davvid@gmail.com>
David Aguilar wrote:
Show 5 quoted lines
> t7800 tests that configured commands can override builtins,
> but this test was not adjusted when the "defaults" file was
> removed because the test continued to pass.
>
> Adjust the test to use the everlasting "vimdiff"
Heh. :)
>                                                  tool name
> instead of "defaults" so that it correctly tests against a tool
> that is known by mergetool--lib.
Makes sense.  Thanks for a pleasant read.

Good night, Jonathan

Previous: David AguilarNext: Jonathan Nieder
Message 5 of 7 in “difftool: silence uninitialized variable warning”
  1. 1/4 difftool: silence uninitialized variable warningDavid Aguilar, Feb 16, 2013
  2. 2/4 t7800: Update copyright noticeDavid Aguilar, Feb 16, 2013
  3. 3/4 t7800: modernize testsDavid Aguilar, Feb 16, 2013
  4. 4/4 t7800: "defaults" is no longer a builtin tool nameDavid Aguilar, Feb 16, 2013
  5. Jonathan NiederFeb 17, 2013
  6. Jonathan NiederFeb 17, 2013
  7. David AguilarFeb 16, 2013

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.