[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [PATCH] Document the textconv filter.
From:       Johannes Sixt <johannes.sixt () telecom ! at>
Date:       2008-09-28 11:07:21
Message-ID: 200809281307.21409.johannes.sixt () telecom ! at
[Download RAW message or body]

On Sonntag, 28. September 2008, Matthieu Moy wrote:
> +Converting files to text before a diff
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +The attribute `textconv` affects 'git diff' in a way similar to the
> +`diff` attribute, but with `textconv`, the user provides only a way to
> +convert the file into text, and git takes care of doing the diff as
> +usual (i.e. other options of 'git diff' such as '--color' remain
> +available).
> +
> +The value of `textconv` must be a string, which is the textconv
> +driver.

Wouldn't it be much more useful to have git parse stdout of the custom diff 
tool in order to colorize it? Of course, this would mean that external diff 
tools are more complicated and their stdout has to conform mostly to the git 
diff syntax. But:

> +To tell git to use the `exif` filter for jpeg images, use:
> +
> +----------------------------------------------------------------
> +*.jpg   textconv=exif
> +----------------------------------------------------------------

In this very example it would be possible that the external diff driver shows 
the differences in the image in a window and also produces EXIF information 
on stdout.

-- Hannes
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]