On 6/2/24 11:34 AM, Ulrich Mueller wrote:
>>>>>> On Sun, 02 Jun 2024, Florian Schmaus wrote:
> 
>> +    (
>> +            insinto "${_GREADME_DOC_DIR}"
>> +
>> +            doins "${_GREADME_TMP_FILE}"
>> +            cksum --raw "${_GREADME_TMP_FILE}" | newins - 
>> "${_GREADME_HASH_FILENAME}"
>> +            assert
>> +    )
> 
> Why do you need that hash file? The old README file exists on the
> system, so couldn't you just compare the new one with it?


Per the commit message, the old readme and the new readme can have the
same contents, but be compressed by different compressors on the live
system vs the image, and/or a compressor with unstable algorithms,
and/or one that isn't installed at the time of merging.

Given you've explicitly rejected disabling compression, I don't quite
see how you can have your cake and eat it too.

-- 
Eli Schwartz

Attachment: OpenPGP_0x84818A6819AF4A9B.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to