Re: [gentoo-dev] [PATCH 0/3] glep-0074: clarifications about size and checksum fields

2022-09-25 Thread Michał Górny
On Fri, 2022-09-23 at 17:58 +, Sheng Yu wrote:
> Hi,
> 
> The hash does not need to be lowercase. It can be a quick fix in portage to 
> accept any case.
> 

I'm all for fixing Portage but I don't think we should be breaking
backwards compatibility over this.

-- 
Best regards,
Michał Górny




Re: [gentoo-dev] [PATCH 0/3] glep-0074: clarifications about size and checksum fields

2022-09-24 Thread Ulrich Mueller
> On Fri, 23 Sep 2022, Sheng Yu wrote:

> The hash does not need to be lowercase. It can be a quick fix in
> portage to accept any case.

That would break backwards compatibility. Also, having a uniquely
defined format helps when comparing hashes with the output of tools like
b2sum or sha512sum.

Ulrich


signature.asc
Description: PGP signature


Re: [gentoo-dev] [PATCH 0/3] glep-0074: clarifications about size and checksum fields

2022-09-23 Thread Sheng Yu
Hi,

The hash does not need to be lowercase. It can be a quick fix in portage to 
accept any case.

Thanks,
Sheng Yu


--- Original Message ---
On Friday, September 23rd, 2022 at 10:03, Michał Górny  
wrote:


> 
> 
> Hi,
> 
> Here's next part of GLEP 74 updates, this time I think it qualifiers
> as 100% editorial. Ulrich Müller noticed that we don't specify how
> to express sizes and checksums. I've partially solved the latter while
> adding the hash algorithm table and these patches should clarify
> the matters even further.
> 
> Notably, they:
> 
> 1) clarify that DIST entries are relevant to package managers only
> 
> 2) specify that size is an unsigned decimal, and checksums consist
> of name-value pairs
> 
> 3) clarify that 'Hex' encoding of hashes means lowercase hex without
> any prefix or suffix
> 
> Perhaps the most controversial part is requiring lowercase -- but
> (drumroll...) Portage reports hash mismatch if you use uppercase.
> 
> 
> Michał Górny (3):
> glep-0074: Clarify that DIST entries are specific to PMs
> glep-0074: Specify the format of size and checksum fields
> glep-0074: Clarify the hex encoding of hash values
> 
> glep-0074.rst | 26 +-
> 1 file changed, 25 insertions(+), 1 deletion(-)
> 
> --
> 2.37.3