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

Re: [PATCH] t7500-commit.sh: do not call test_set_editor unnecessarily, it's confusing

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 29, 2008, 09:46 UTC
Message-ID
<7vmyefco11.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<1230542658-9758-1-git-send-email-dato@net.com.org.es>
Adeodato Simó <dato@net.com.org.es> writes:
> I was reading this test case, and it took a small bit to figure out the
> editor was not being used at all. I hope there was no hidden reason for
> it to be there, and it can go away.

That 'zort' came from 1320857 (builtin-commit: fix --signoff, 2007-11-11), and I _think_ it is trying to make sure that presense of "-F -" makes the editor not to trigger.

Dscho?
Previous: Adeodato SimóNext: Adeodato Simó
Message 2 of 9 in “t7500-commit.sh: do not call test_set_editor unnecessarily, it's confusing”
  1. t7500-commit.sh: do not call test_set_editor unnecessarily, it's confusingAdeodato Simó, Dec 29, 2008
  2. Junio C HamanoDec 29, 2008
  3. Adeodato SimóDec 29, 2008
  4. Junio C HamanoDec 29, 2008
  5. Johannes SchindelinDec 30, 2008
  6. t7501-commit.sh: explicitly check that -F prevents invoking the editorAdeodato Simó, Jan 9, 2009
  7. Johannes SchindelinJan 10, 2009
  8. Adeodato SimóJan 10, 2009
  9. Johannes SchindelinJan 10, 2009

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.