Re: [PATCH] Add post-merge hook.

["Josh England" <jjengla@sandia.gov>]

On Tue, 2007-09-04 at 13:03 -0700, Junio C Hamano wrote:
> >> >>  * Do you want to run the post-merge hook even for a squash
> >> >>    merge?
> >> >
> >> > Yes.  I'd like to run it at any time that the working tree might be
> >> > updated.
> >> 
> >> If that is the case, perhaps your hook may want to get a
> >> parameter to tell it what kind of "git-merge" invocation it was?
> >> Squash merge does not even advance the HEAD and is of a very
> >> different nature from a normal merge.
> >
> > OK.  Should it just pass in a flag (squash or normal), or are there
> > other merge types it should need to know about.
> 
> I suspect you have thought abuot the issues involved longer than
> I have ;-), so you should take whatever I say with grain of
> salt, but I think you would also want to know fast-forwards and
> up-to-dates if the hook wants to be generic, not "for Josh's
> workflow only".

Generic is great, I'm just trying to figure out when/why someone would
need to know the exact type of merge operation used.  The hook should be
generic, yet not require an end user to know any git internals not
explicitly mentioned in the git-merge man page.  I'm thinking that it
will be sufficient to pass a flag indicating whether the working tree
has been modified or not.  The flag can be set for normal merge and
fast-forward merges, and unset for up-to-dates and squash merges.  I
don't really know git internals myself.  Am I missing anything?

-JE