GNU bug report logs - #9157
[PATCH] dd: sparse conv flag

Previous Next

Package: coreutils;

Reported by: Roman Rybalko <devel <at> romanr.info>

Date: Sat, 23 Jul 2011 22:42:04 UTC

Severity: normal

Tags: patch

Done: Pádraig Brady <P <at> draigBrady.com>

Bug is archived. No further changes may be made.

Full log


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

From: Pádraig Brady <P <at> draigBrady.com>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 9157 <at> debbugs.gnu.org
Subject: Re: bug#9157: [PATCH] dd: sparse conv flag
Date: Mon, 27 Feb 2012 18:06:12 +0000
On 02/27/2012 05:14 PM, Paul Eggert wrote:
> On 02/27/2012 08:10 AM, Pádraig Brady wrote:
>> I'm also thinking I should warn once when
>> the new lseek fails, but only after the fall back
>> write() succeeds.
> 
> I wouldn't bother.  Either diagnostic should suffice,
> and warning about the lseek would slow the app down
> a bit and make it more complicated.

Yep, I meant defer to the write() diagnostic.

What I was wondering was whether to warn about
the case where the seek fails but the write succeeds like here:

  dd conv=sparse bs=1 count=10 | ...

I'll leave it as advisory for now, and not have
a warning for the above case.

cheers,
Pádraig.




This bug report was last modified 13 years and 182 days ago.

Previous Next


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