[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: cksum: add tests to verify the presence of "*" (and question about u
|
From: |
Pádraig Brady |
|
Subject: |
Re: cksum: add tests to verify the presence of "*" (and question about usability of cksum) |
|
Date: |
Sat, 4 May 2024 14:22:28 +0100 |
|
User-agent: |
Mozilla Thunderbird Beta |
On 03/05/2024 18:53, Sylvestre Ledru wrote:
Hello,
The attached patch checks for a code path that wasn't covered before
(afaik).
Pushed.
As a side note, the various combinations of --tag / --untagged and
--binary / --text is producing some unexpected results at time :)
Maybe that ship sailed 10 or 20 years ago but would it make sense to
reject --tag & --untagged (and binary/text) passed together?
Well the binary mode indicator wasn't really well designed IMHO,
so current efforts are mainly to keep compat.
Note we support --tag with --untagged to support overriding.
For example one might have md5sum implemented like:
md5sum() { cksum -a md5 --untagged "$@"; }
In that case it's best to support `md5sum --tag`
thanks,
Pádraig