[PATCH v3 5/5] tools: Update to use 'hasher'

Stephen Finucane stephen at that.guru
Fri Dec 2 21:18:24 AEDT 2016


On Fri, 2016-12-02 at 15:42 +1100, Daniel Axtens wrote:
> Hi,
> 
> >      local hash
> > -    hash=$(git show -C $1 | python $PWDIR/parser.py --hash)
> > +    hash=$(git diff "$1~..$1" | python $pwpath/hasher.py)

That's a good point - I don't think the existing code handled merge
commits and neither will this...

> Does this function appropriately in the presence of merge commits?
> ISTR
> they had interesting behaviour and there was a difference between
> using
> ~ and using ^, and I just wanted to flag that for consideration...

That's a very good point - I had no idea these did different things but
it would appear so:

  http://stackoverflow.com/a/12527561/613428

I've gone with '~' for now to fix Tom's issue, but I'm going to
investigate this over the weekend and submit a follow-up patch if
necessary.

Stephen


More information about the Patchwork mailing list