[Buildroot] [PATCH] ser2net: Add a hash file
Vicente Olivert Riera
Vincent.Riera at imgtec.com
Wed Oct 8 09:25:36 UTC 2014
Dear Thomas Petazzoni,
On 10/08/2014 10:20 AM, Thomas Petazzoni wrote:
> Dear Vicente Olivert Riera,
>
> On Wed, 8 Oct 2014 09:57:39 +0100, Vicente Olivert Riera wrote:
>
>>> Why having all those hashes?
>>
>> Why not? Those are all the hashes supported in Buildroot. The more we
>> have the better.
>
> I disagree: more hashes means more work to do when bumping the package.
> Either one strong hash, or two weaker hashes should be sufficient IMO.
More work? I disagree. I created the hash file with a one-liner command:
for i in md5 sha1 sha224 sha256 sha384 sha512; do echo $i `${i}sum
ser2net-2.10.0.tar.gz` >> ../package/ser2net/ser2net.hash; done
> Something to be discussed at the meeting maybe?
>
>>> And the comment indicating how they have
>>> been found is missing (locally computed? found on the upstream web
>>> site).
>>
>> I haven't added any comment because those hashes were locally computed.
>> Do I need to add a comment explain that?
>
> Yes, I believe so.
No problem, I can resend the patch adding that comment. Or maybe it's
easier if you or Peter do it, since is a trivial change and you are the
ones who will commit the patch at the end.
> Best regards,
>
> Thomas
>
--
Vincent
More information about the buildroot
mailing list