Re: [PATCH] dd: sparse conv flag

[Pádraig Brady]

On 25/07/11 21:45, Eric Blake wrote:
> On 07/23/2011 10:08 AM, Roman Rybalko wrote:
>> ---
>>   doc/coreutils.texi |    4 +++
>>   src/dd.c           |   22 +++++++++++++++-
>>   tests/Makefile.am  |    1 +
>>   tests/dd/sparse    |   70
>> ++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   4 files changed, 96 insertions(+), 1 deletions(-)
>>   create mode 100755 tests/dd/sparse
> 
> Thanks for taking the time to do this.  I agree that since BSD dd
> understands a sparse option, that it makes sense to teach coreutils dd
> this option as well.  However, the size of your patch is non-trivial;
> would you be willing to start the copyright assignment process to give
> rights to your patch to the FSF?
> 

I agree that this would fit well.
I wonder should be try to use the efficient
sparse copy code done for cp?
I guess that could be done after this in any case.

cheers,
Pádraig