[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bug#15604: [coreutils] [PATCH] md5sum: Add option to ignore non-exis
From: |
Pádraig Brady |
Subject: |
Re: bug#15604: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files |
Date: |
Mon, 23 Nov 2015 17:24:42 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 23/11/15 16:41, Jim Meyering wrote:
> I think a common expected usage of --ignore-missing would be
> the case of an SHA1SUM file listing all possibly-verified files for
> which it is common to verify only the one or two downloaded files.
> In any invocation that ends up ignoring *all* file names, I would
> want a loud warning and failure, to be sure that my eyes (and/or
> any tool) notice something is wrong.
>
> The absence of an "OK" is far easier to miss than a diagnostic.
> At least a few are often expected to be missing, so I see little
> value in emitting "MISSING" diagnostics.
Yes I agree. Thinking more, one could have a syntactically correct
checksum file which is adjusted to comment out certain entries, and
currently sha1sum etc. (with or without --ignore-missing)
will error out unless something is verified:
$ echo '#'|sha1sum --check --ignore-missing
sha1sum: standard input: no properly formatted SHA1 checksum lines found
So given that the existing functionality is to ensure something is verified,
then --ignore-missing should be consistent.
I'll push the attached a bit later.
thanks!
Pádraig
ignore-missing-none.patch
Description: Text Data
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Pádraig Brady, 2015/11/01
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Pádraig Brady, 2015/11/22
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Bernhard Voelker, 2015/11/23
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Pádraig Brady, 2015/11/23
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Pádraig Brady, 2015/11/23
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Jim Meyering, 2015/11/23
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Pádraig Brady, 2015/11/23
- Re: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Jim Meyering, 2015/11/23
- Re: bug#15604: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files,
Pádraig Brady <=
- Re: bug#15604: [coreutils] [PATCH] md5sum: Add option to ignore non-existant files, Jim Meyering, 2015/11/23