Re: [PATCH 1/4] gitweb: Add GIT favicon, assuming image/png type
Show 12 quoted lines
> Jakub Narebski <jnareb@gmail.com> writes:
>
>> Jakub Narebski wrote:
>>
>>> index 199fbe87384cd3f4686916277dd124cefc751e8a..f3b762de92e579300eba3f53ee26b2a80b4e76c7 100644
>>
>> Hmmm... I do wonder why --binary implies --full-index option.
>
> Patch safety. Yours happened to be a creation patch so it might
> not matter in this particular case, but we would want to be
> reasonably sure there is no ambiguity in the preimage when you
> are doing a modification patch.
Couldn't we do this (show full sha1) only for the binary part of patch?
--
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git