threads / discuss / 17340

diff settings

Subject: diff settings

## tl;dr

13 messages between Jan 24, 2009 and Jan 31, 2009.

replies: 12people: 6as markdown or json

Keith Cascio· Jan 24, 2009, 17:38 UTC · lore

How do I configure my local git so that diff always obeys a particular option, e.g. "-w", without needing to type it on the command line each time?

Thanks, Keith

Jeff King· Jan 24, 2009, 19:22 UTC · re: Keith Cascio · lore

Re: diff settings

On Sat, Jan 24, 2009 at 09:38:18AM -0800, Keith Cascio wrote:
> How do I configure my local git so that diff always obeys a particular
> option, e.g. "-w", without needing to type it on the command line each
> time?

AFAIK, there is not a way to do it in the general case (though there are specific config options for things like rename support). When I refactored the gitattributes diff=$foo code a few months back, I tried to keep in mind the possibility of a "diff options" field. But I never actually implemented anything, as I don't personally ever use diff options in such a way.

If you're interested, it would probably not be too hard to implement (see userdiff.[ch], and the external and textconv code in diff.c for a start).

-Peff
Teemu Likonen· Jan 24, 2009, 19:33 UTC · re: Keith Cascio · lore

Re: diff settings

Keith Cascio (2009-01-24 09:38 -0800) wrote:
> How do I configure my local git so that diff always obeys a particular
> option, e.g. "-w", without needing to type it on the command line each
> time?
It seems that the answer is "no", but aliases are a work-around:
    git config --global alias.dff "diff -w"
Then "git dff" is your new "git diff -w". :-)
Keith Cascio· Jan 24, 2009, 20:01 UTC · re: Teemu Likonen · lore

Re: diff settings

Show 5 quoted lines
> It seems that the answer is "no", but aliases are a work-around:
> 
>     git config --global alias.dff "diff -w"
> 
> Then "git dff" is your new "git diff -w". :-)

That feature is gonna come in handy. It doesn't 100% fulfill my needs here but I like it! Thanks, I did not know about alias.*

Ted Pavlic· Jan 25, 2009, 21:02 UTC · re: Keith Cascio · lore

Re: diff settings

Show 6 quoted lines
>>      git config --global alias.dff "diff -w"
>>
>> Then "git dff" is your new "git diff -w". :-)
>
> That feature is gonna come in handy.  It doesn't 100% fulfill my needs here but
> I like it!  Thanks, I did not know about alias.*

Mercurial has a "defaults.*" that does exactly what you want. That is, you can say

[defaults] diff = -w

and "hg diff" will automatically do "hg diff -w". Such a feature might be a nice addition to git.

--Ted
-- 
Ted Pavlic <ted@tedpavlic.com>

   Please visit my ALS association page:
         http://web.alsa.org/goto/tedpavlic
   My family appreciates your support in the fight to defeat ALS.
Keith Cascio· Jan 25, 2009, 21:11 UTC · re: Ted Pavlic · lore

Re: diff settings

On Sun, 25 Jan 2009, Ted Pavlic wrote:
Show 8 quoted lines
> Mercurial has a "defaults.*" that does exactly what you want. That is, you can
> say
> 
> [defaults]
> diff = -w
> 
> and "hg diff" will automatically do "hg diff -w". Such a feature might be a 
> nice addition to git.

Thank you, that is very interesting. I just submitted a patch this morning that does exactly what you describe, but I called it "primer" instead of "defaults" because it seemed more explicit. Check it out.

                            -- Keith
Ted Pavlic· Jan 26, 2009, 02:06 UTC · re: Keith Cascio · lore

Re: diff settings

Show 8 quoted lines
>> Mercurial has a "defaults.*" that does exactly what you want. That is, you can
>> [defaults]
>> diff = -w
>> and "hg diff" will automatically do "hg diff -w". Such a feature might be a
>> nice addition to git.
> Thank you, that is very interesting.  I just submitted a patch this morning that
> does exactly what you describe, but I called it "primer" instead of "defaults"
> because it seemed more explicit.  Check it out.

I saw that. However, in Mercurial's case, the [defaults] section applies to all commands (including new ones introduced by extensions). That is, it looks like your patch adds a diff.primer, but Mercurial would add a primer.* that allows you to have a primer.diff, primer.commit, etc.

For example, wouldn't it be nice to have something like...

[defaults] diff = -w commit = -a

etc.? Right now the only way you can do that is with aliases (e.g., I have an alias from "ci" to "commit -a").

This task is pretty easy in Mercurial because Mercurial porcelains are implemented as modules that are all executed through the central "hg" command. This isn't the case with git.

(on a somewhat unrelated note, Mercurial does have a [diff] section that allows for, say, forcing the use of git diff and other things)

--Ted
-- 
Ted Pavlic <ted@tedpavlic.com>

   Please visit my ALS association page:
         http://web.alsa.org/goto/tedpavlic
   My family appreciates your support in the fight to defeat ALS.
Ted Pavlic· Jan 29, 2009, 16:26 UTC · re: Ted Pavlic · lore

Re: diff settings

> This task is pretty easy in Mercurial because Mercurial porcelains are
> implemented as modules that are all executed through the central "hg"
> command. This isn't the case with git.

(on the other hand, having a "[defaults]" section that only applies when commands are called via "git" might be nice. That way you can use the "git-CMD" when you want to use CMD without the "[defaults]"... perhaps?)

-- 
Ted Pavlic <ted@tedpavlic.com>

   Please visit my ALS association page:
         http://web.alsa.org/goto/tedpavlic
   My family appreciates your support in the fight to defeat ALS.
Johannes Schindelin· Jan 29, 2009, 16:33 UTC · re: Ted Pavlic · lore

Re: diff settings

Hi,
On Thu, 29 Jan 2009, Ted Pavlic wrote:
Show 7 quoted lines
> > This task is pretty easy in Mercurial because Mercurial porcelains are
> > implemented as modules that are all executed through the central "hg"
> > command. This isn't the case with git.
> 
> (on the other hand, having a "[defaults]" section that only applies when
> commands are called via "git" might be nice. That way you can use the
> "git-CMD" when you want to use CMD without the "[defaults]"... perhaps?)

Nope, the dash form is deprecated, and once y'all out there finally do not write it any more, we can start thinking about _not_ hardlinking the builtins to their dashed form anymore.

Believe it or not, we already have a distinction between what is called from scripts vs from humans: plumbing vs porcelain. So you can set the defaults for porcelain as much as you want, but please leave plumbing alone.

Ciao, Dscho

Keith Cascio· Jan 29, 2009, 18:46 UTC · re: Johannes Schindelin · lore

Re: diff settings

On Thu, 29 Jan 2009, Johannes Schindelin wrote:
> we already have a distinction between what is called from scripts vs from 
> humans: plumbing vs porcelain.  So you can set the defaults for porcelain as 
> much as you want, but please leave plumbing alone.
May we consider "git diff" Porcelain and "git diff-{files,index,tree}" plumbing?
Nanako Shiraishi· Jan 29, 2009, 21:07 UTC · re: Keith Cascio · lore

Re: diff settings

Quoting Keith Cascio <keith@CS.UCLA.EDU>:
Show 7 quoted lines
> On Thu, 29 Jan 2009, Johannes Schindelin wrote:
>
>> we already have a distinction between what is called from scripts vs from 
>> humans: plumbing vs porcelain.  So you can set the defaults for porcelain as 
>> much as you want, but please leave plumbing alone.
>
> May we consider "git diff" Porcelain and "git diff-{files,index,tree}" plumbing?
Sure we may. To see the list of commands with categories, you can consult
http://www.kernel.org/pub/software/scm/git/docs/
Junio, I think you could have stated this a bit more clearly in your message
http://thread.gmane.org/gmane.comp.version-control.git/107069/focus=107165
to avoid confusion, especially when you know you are talking to somebody new to git.
-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/
Ted Pavlic· Jan 31, 2009, 19:47 UTC · lore

Re: diff settings

> Thank you for alerting us to the Hg design.  I can appreciate the elegance of
> it, and I'm always in favor of the most general possible implementation.  But
> there is a subtle difference between diff.primer and primer.diff.

As I already discussed, Mercurial has both "diff.primer" and "primer.diff" precisely because "diff" settings affects so many Hg commands.

In particular, in my .hgrc, I have:

[diff] git = 1

which causes all Mercurial commands that need to generate a diff to use gitdiff rather than truediff. However, if I *wanted* to apply a set of flags to a particular command, I could (using the equivalent "git" commands)...

[defaults] pull = --rebase commit = -a format-patch = -M

So I see a purpose for both *.defaults and defaults.*. Of course, aliases are also nice (IIRC, Mercurial doesn't have "aliases", but I could be wrong). Mercurial users are very happy with having both available. I imagine git people would be too.

--Ted
-- 
Ted Pavlic <ted@tedpavlic.com>

   Please visit my ALS association page:
         http://web.alsa.org/goto/tedpavlic
   My family appreciates your support in the fight to defeat ALS.
Keith Cascio· Jan 31, 2009, 23:10 UTC · re: Ted Pavlic · lore

Re: diff settings

On Sat, 31 Jan 2009, Ted Pavlic wrote:
> As I already discussed, Mercurial has both "diff.primer" and "primer.diff" 
> precisely because "diff" settings affects so many Hg commands.
Yes I remember you did mention that.
Show 14 quoted lines
> In particular, in my .hgrc, I have:
> [diff]
> git = 1
> which causes all Mercurial commands that need to generate a diff to use
> gitdiff rather than truediff. However, if I *wanted* to apply a set of flags
> to a particular command, I could (using the equivalent "git" commands)...
> [defaults]
> pull = --rebase
> commit = -a
> format-patch = -M
> So I see a purpose for both *.defaults and defaults.*. Of course, aliases are 
> also nice (IIRC, Mercurial doesn't have "aliases", but I could be wrong). 
> Mercurial users are very happy with having both available. I imagine git 
> people would be too.

I like it! I'm all for it. Perfect opportunity for future work. In fact, I think I already glanced at the part of the code that's the right place to make that stuff happen.

                                      -- Keith

← back to recent threads