On Fri, Aug 18, 2023 at 04:59:27PM -0700, Junio C Hamano wrote:
Paul Watson reported that "diff --no-index --exit-code
--ignore-all-space" does not work when used with "--shortstat". It
turns out that this is not limited to "--no-index" mode, and it is
not limited to "--shortstat". Anything that does not use the "--patch"
machinery to discover the content level differences ignored --exit-code
when used with options like "-w" and always exited with 0. In fact,
even the "--patch" machinery was slightly faulty in corner cases.
And here is another round to fix it.
Previous one is at https://lore.kernel.org/git/20230817222949.3835424-1-gitster@pobox.com/ (local)
This looks good to me. I left some musings, but I don't think there is
anything that merits a re-roll (you might want to fix-up a commit
message typo locally).
Thanks for running with it. It's much nicer than v1. :)
-Peff