Konstantin Ryabitsev [off-list ref] writes:
I'm not sure why --full-index was added there -- I don't think it's needed for
--binary?
Correct. For the purpose of that quoted commit, giving "--binary"
should be sufficient. It would avoid "Binary files differ" that is
useless in the context of format-patch, and when a binary patch is
given, it will automatically get the full length blob object name,
without --full-index.
Thanks.