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

Re: [PATCH] git-mergetool--lib.sh: fix mergetool.<tool>.* configurations ignored for known tools

From
Sylvain Rabot <sylvain@abstraction.fr>
Date
Jun 5, 2010, 22:42 UTC
Message-ID
<1275777749.18270.23.camel@kheops>
In-Reply-To
<m2fx11n8rp.fsf@igel.home>
On Sat, 2010-06-05 at 11:11 +0200, Andreas Schwab wrote:
Show 12 quoted lines
> Sylvain Rabot <sylvain@abstraction.fr> writes:
> 
> > +			if !test -f "$merge_tool_path/$merge_tool_cmd_base"; then
> 
> Missing space after '!'.
> 
> > +			elif !test -x "$merge_tool_path/$merge_tool_cmd_base"; then
> 
> Likewise.
> 
> Andreas.
> 
Thanks, 

I have updated the patch, you can find it here git://git.abstraction.fr/~sylvain/git.git in the mergetool-lib branch.

http://git.abstraction.fr/~sylvain/?p=git.git;a=commitdiff;h=905bfb5cea0750a67bf9bcc2baf22079054742fa
-- 
Sylvain Rabot <sylvain@abstraction.fr>
Previous: Andreas SchwabNext: David Aguilar
Message 4 of 9 in “git-mergetool--lib.sh: fix mergetool.<tool>.* configurations ignored for known tools”
  1. git-mergetool--lib.sh: fix mergetool.<tool>.* configurations ignored for known toolsSylvain Rabot, Jun 5, 2010
  2. git-mergetool--lib.sh: fix mergetool.<tool>.* configurations ignored for known toolsSylvain Rabot, Jun 5, 2010
  3. Andreas SchwabJun 5, 2010
  4. Sylvain RabotJun 5, 2010
  5. David AguilarJun 8, 2010
  6. Junio C HamanoJun 8, 2010
  7. Sylvain RabotJun 11, 2010
  8. Charles BaileyJun 9, 2010
  9. Sylvain RabotJun 11, 2010

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.