GNU bug report logs - #58050
[INSTALLED] rm: fix diagnostics on I/O error

Previous Next

Package: coreutils;

Reported by: Paul Eggert <eggert <at> cs.ucla.edu>

Date: Sat, 24 Sep 2022 23:39:02 UTC

Severity: normal

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log

Message #10 received at 58050 <at> debbugs.gnu.org (full text, mbox):

From: Pádraig Brady <P <at> draigBrady.com>
To: Paul Eggert <eggert <at> cs.ucla.edu>, 58050 <at> debbugs.gnu.org
Subject: Re: bug#58050: [INSTALLED] rm: fix diagnostics on I/O error
Date: Sun, 25 Sep 2022 15:25:25 +0100

[Message part 1 (text/plain, inline)]
On 25/09/2022 00:37, Paul Eggert wrote:
> I ran into this problem when attempting to recursively
> remove a directory in a filesystem on flaky hardware.
> Although the underlying readdir syscall failed with errno == EIO,
> rm issued no diagnostic about the I/O error.

This looks good.
How about the attached to add a NEWS entry,
and add DS_EMPTY, DS_NONEMPTY enums to make the code easier to read?

thanks,
Pádraig

[rm-use-constants.patch (text/x-patch, attachment)]
This bug report was last modified 2 years and 238 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.