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

Re: What should mergetool do with --no-prompt?

From
Charles Bailey <charles@hashpling.org>
Date
Aug 14, 2012, 07:18 UTC
Message-ID
<20120814071823.GA21031@hashpling.org>
In-Reply-To
<CAJDDKr5TK910n603jcmoq6WoaLL9DX9hgwF3Y+MmjngMpAXPQw@mail.gmail.com>
On Tue, Aug 14, 2012 at 12:07:26AM -0700, David Aguilar wrote:
Show 12 quoted lines
> Right now there are two code paths, resolving deletion conflicts
> and resolving symlink conflicts, in git-mergetool that do not
> honor --no-prompt.  They force user-interaction with the shell
> even though the caller (such as a program) said that they do
> not want to be prompted.
> 
> This was an oversight from when this option was first added.
> 
> I think a simple and sensible thing to do would be for mergetool
> to skip over these entries when --no-prompt is supplied.
> 
> Does this sound like a good idea?

--no-prompt is designed to remove the prompt before launching a mergetool. This is because it is mostly pointless but does provide a convenient point to interrupt (C-c) a large multifile conflict resolution.

It was never supposed to be a batch mode switch. By it's very nature mergetool is interactive so I don't see any advantage to pretending otherwise.

If the documentation indicates otherwise then it's my opinion that this is what needs to be fixed.

Charles.
Previous: David AguilarNext: Junio C Hamano
Message 2 of 6 in “What should mergetool do with --no-prompt?”
  1. David AguilarAug 14, 2012
  2. Charles BaileyAug 14, 2012
  3. Junio C HamanoAug 14, 2012
  4. Charles BaileyAug 14, 2012
  5. Junio C HamanoAug 14, 2012
  6. David AguilarAug 14, 2012

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.