Re: is there a reason pre-commit.sample uses "git diff-index"?

From: Robert P. J. Day <rpjday@crashcourse.ca>
Date: May 31, 2018, 17:27 UTC
Message-ID: <alpine.LFD.2.21.1805311323220.17047@localhost.localdomain>
In-Reply-To: <CACsJy8CP=1_bHAzs+mveaJvbOOzr9vny-2xz6kM4cEcCYWmOOw@mail.gmail.com>

On Thu, 31 May 2018, Duy Nguyen wrote:

Show 17 quoted lines

> On Thu, May 31, 2018 at 6:38 PM, Robert P. J. Day <rpjday@crashcourse.ca> wrote:
> >
> >   was going over some hooks and writing some tutorials for some of
> > the commit-related, client-side hooks, and was wondering (perhaps
> > stupidly) why the pre-commit.sample hook uses, as its last line:
> >
> >   exec git diff-index --check --cached $against --
> >            ^^^^^^^^^^
> >
> > as in, could this not be done equivalently with just git diff, not
> > git diff-index? i just did a quick test and it seems to do the
> > same thing, but i've never taken a close look at git diff-index
> > before so i may just be clueless about some important distinction.
>
> git diff-index is "plumbing", designed for writing scripts. "git
> diff" on the other hand is for users and its behavior may change
> even if it breaks backward compatibility.

  ah, this was a philosophical underpinning i was unaware of. i see
occasional explanations of git porcelain versus plumbing, but i don't
recall anyone simply stating that the plumbing is meant to have a
long-term stability that is not guaranteed for the porcelain.

  in any event, this does mean that, stability issues aside, "git
diff" would apparently have worked just fine for that hook.

rday

-- 
========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                  http://crashcourse.ca/dokuwiki

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================