<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git-mergetool.sh, branch v2.1.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.1.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.1.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-06-09T21:47:07Z</updated>
<entry>
<title>git-mergetool.sh: avoid "test &lt;cond&gt; -a/-o &lt;cond&gt;"</title>
<updated>2014-06-09T21:47:07Z</updated>
<author>
<name>Elia Pinto</name>
<email>gitter.spiros@gmail.com</email>
</author>
<published>2014-06-06T14:55:51Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1cb493739540bd196848a59f9b7765b0622f725c'/>
<id>urn:sha1:1cb493739540bd196848a59f9b7765b0622f725c</id>
<content type='text'>
The construct is error-prone; "test" being built-in in most modern
shells, the reason to avoid "test &lt;cond&gt; &amp;&amp; test &lt;cond&gt;" spawning
one extra process by using a single "test &lt;cond&gt; -a &lt;cond&gt;" no
longer exists.

Signed-off-by: Elia Pinto &lt;gitter.spiros@gmail.com&gt;
Reviewed-by: Matthieu Moy &lt;Matthieu.Moy@imag.fr&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>mergetool: run prompt only if guessed tool</title>
<updated>2014-04-22T19:49:46Z</updated>
<author>
<name>Felipe Contreras</name>
<email>felipe.contreras@gmail.com</email>
</author>
<published>2014-04-21T00:17:34Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4ecc63d7f982aa65038048bcdaf49e99b428b028'/>
<id>urn:sha1:4ecc63d7f982aa65038048bcdaf49e99b428b028</id>
<content type='text'>
It's annoying to see the prompt:

  Hit return to start merge resolution tool (foo):

Every time the user does 'git mergetool' even if the user already
configured 'foo' as the wanted tool.

Display this prompt only when the user hasn't explicitly configured a
tool.

Signed-off-by: Felipe Contreras &lt;felipe.contreras@gmail.com&gt;
Acked-by: David Aguilar &lt;davvid@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'al/mergetool-printf-fix'</title>
<updated>2013-02-14T18:29:37Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-02-14T18:29:37Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ba56d7057a5da9eb3485391658ba7465c855a8f3'/>
<id>urn:sha1:ba56d7057a5da9eb3485391658ba7465c855a8f3</id>
<content type='text'>
* al/mergetool-printf-fix:
  difftool--helper: fix printf usage
  git-mergetool: print filename when it contains %
</content>
</entry>
<entry>
<title>git-mergetool: print filename when it contains %</title>
<updated>2013-02-08T21:10:06Z</updated>
<author>
<name>Asheesh Laroia</name>
<email>asheesh@asheesh.org</email>
</author>
<published>2013-02-08T01:16:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=59cf706b2381d99b39c4d1daf62da9eeccf0ec13'/>
<id>urn:sha1:59cf706b2381d99b39c4d1daf62da9eeccf0ec13</id>
<content type='text'>
If git-mergetool was invoked with files with a percent sign (%) in
their names, it would print an error.  For example, if you were
calling mergetool on a file called "%2F":

    printf: %2F: invalid directive

Do not pass random string to printf as if it were a valid format.
Use format string "%s" and pass the string as data to be formatted
instead.

Signed-off-by: Asheesh Laroia &lt;asheesh@asheesh.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-mergetool: move show_tool_help to mergetool--lib</title>
<updated>2013-01-25T19:07:30Z</updated>
<author>
<name>John Keeping</name>
<email>john@keeping.me.uk</email>
</author>
<published>2013-01-25T09:43:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4a8273a3ed1d9db48920433471f87090423d1b5e'/>
<id>urn:sha1:4a8273a3ed1d9db48920433471f87090423d1b5e</id>
<content type='text'>
This is the first step in unifying "git difftool --tool-help" and
"git mergetool --tool-help".

Signed-off-by: John Keeping &lt;john@keeping.me.uk&gt;
Signed-off-by: David Aguilar &lt;davvid@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>mergetool: style fixes</title>
<updated>2012-08-24T04:30:51Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junio@pobox.com</email>
</author>
<published>2012-08-23T05:33:15Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f8750a0ea951a328481b67b5d0558c0402f0cdfd'/>
<id>urn:sha1:f8750a0ea951a328481b67b5d0558c0402f0cdfd</id>
<content type='text'>
This script is one of the sizeable ones that tempted people to copy
its "neibouring style" in their new code, but was littered with
styles incompatible with our style guide.

 - use one tab, not four spaces, per indent level;

 - long lines can be wrapped after '|', '&amp;&amp;', or '||' for
   readability.

 - structures like "if .. then .. else .. fi", "while .. do .. done"
   are split into lines in such a way that does not require
   unnecessary semicolon.

 - case, esac and case-arms align at the same column.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>mergetool: support --tool-help option like difftool does</title>
<updated>2012-07-23T21:42:39Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2012-07-23T21:16:12Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=109859e27445a3ad12eb8f8cba43a5aa8f701733'/>
<id>urn:sha1:109859e27445a3ad12eb8f8cba43a5aa8f701733</id>
<content type='text'>
This way we do not have to risk the list of tools going out of sync
between the implementation and the documentation.

In the same spirit as bf73fc2 (difftool: print list of valid tools
with '--tool-help', 2012-03-29), trim the list of merge backends in
the documentation.  We do not want to have a complete list of valid
tools; we only want a list to help people guess what kind of things
the tools do to be specified there, and refer them to --tool-help
for a complete list.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>mergetool: Provide an empty file when needed</title>
<updated>2012-01-23T19:50:22Z</updated>
<author>
<name>David Aguilar</name>
<email>davvid@gmail.com</email>
</author>
<published>2012-01-20T07:47:35Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ec245ba04944473ba6657a826643ef90ed02b51d'/>
<id>urn:sha1:ec245ba04944473ba6657a826643ef90ed02b51d</id>
<content type='text'>
Some merge tools cannot cope when $LOCAL, $BASE, or $REMOTE are missing.
$BASE can be missing when two branches independently add the same
filename.

Provide an empty file to make these tools happy.

When a delete/modify conflict occurs, $LOCAL and $REMOTE can also be
missing. We have special case code to handle such case so this change
may not affect that codepath, but try to be consistent and create an
empty file for them anyway.

Reported-by: Jason Wenger &lt;jcwenger@gmail.com&gt;
Signed-off-by: David Aguilar &lt;davvid@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jm/mergetool-pathspec'</title>
<updated>2011-10-10T22:56:18Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2011-10-10T22:56:18Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=5fbdb9c2e8cc7226d9a9e7de5ad09ac5f0a0b906'/>
<id>urn:sha1:5fbdb9c2e8cc7226d9a9e7de5ad09ac5f0a0b906</id>
<content type='text'>
* jm/mergetool-pathspec:
  mergetool: no longer need to save standard input
  mergetool: Use args as pathspec to unmerged files
</content>
</entry>
<entry>
<title>mergetool: no longer need to save standard input</title>
<updated>2011-09-26T16:39:45Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2011-09-16T20:19:33Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=6d9990a959c0168ab6dfe75236980f95ac512ce5'/>
<id>urn:sha1:6d9990a959c0168ab6dfe75236980f95ac512ce5</id>
<content type='text'>
Earlier code wanted to run merge_file and prompt_after_failed_merge
both of which wanted to read from the standard input of the entire
script inside a while loop, which read from a pipe, and in order to
do so, it redirected the original standard input to another file
descriptor. We no longer need to do so after the previous change.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
