<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/git-mergetool.sh, branch v1.8.3.3</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.8.3.3</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.8.3.3'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2013-02-14T18:29:37Z</updated>
<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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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.stealer.net/cgit.cgi/user/sven/git.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>
<entry>
<title>mergetool: Use args as pathspec to unmerged files</title>
<updated>2011-09-26T16:39:36Z</updated>
<author>
<name>Jonathon Mah</name>
<email>me@JonathonMah.com</email>
</author>
<published>2011-09-16T02:12:10Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=3e8e691abe4e1cce73a8a2ef413dada0278e7b3b'/>
<id>urn:sha1:3e8e691abe4e1cce73a8a2ef413dada0278e7b3b</id>
<content type='text'>
Mergetool now treats its path arguments as a pathspec (like other git
subcommands), restricting action to the given files and directories.
Files matching the pathspec are filtered so mergetool only acts on
unmerged paths; previously it would assume each path argument was in an
unresolved state, and get confused when it couldn't check out their
other stages.

Running "git mergetool subdir" will prompt to resolve all conflicted
blobs under subdir.

Signed-off-by: Jonathon Mah &lt;me@JonathonMah.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>git-mergetool: check return value from read</title>
<updated>2011-09-20T00:41:49Z</updated>
<author>
<name>Jay Soffian</name>
<email>jaysoffian@gmail.com</email>
</author>
<published>2011-09-19T23:40:52Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=e622f41dcd97861796c7529712170f9eae07dc56'/>
<id>urn:sha1:e622f41dcd97861796c7529712170f9eae07dc56</id>
<content type='text'>
Mostly fixed already by 6b44577 (mergetool: check return value
from read, 2011-07-01). Catch two uses it missed.

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