[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Shred: Add recursion operations [PATCH]
From: |
Amr Ali |
Subject: |
Re: Shred: Add recursion operations [PATCH] |
Date: |
Fri, 09 Dec 2011 19:52:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111124 Thunderbird/8.0 |
Hi Eric,
I completely agree about the philosophy. But ...
1) I've seen a comment at shred.c:26 that indicates adding -R/-r/--recursive as
a TODO.
2) Adding recursion to shred is not bloat, it is in fact enabling shred to do
it's job, and to do it *well*.
3) On the command line, the overhead of first, having to type down all of the
find parameters, and second, the overhead of executing shred, quite literally,
on every single file is a waste of both time and resources. Adding a simple `-r`
flag, however, enhances usability dramatically compared to the find(1) solution.
4) It became more than obvious that a lot of users *do* want such ability if you
will, given the amount of postings online that talks/addresses the shred
recursion issue.
5) If `find` truly is made to enable recursion for all programs, why recursion
is available for chmod or chgrp, or the plethora of other tools?
Best regards,
On 12/09/2011 07:34 PM, Eric Blake wrote:
> On 12/09/2011 10:22 AM, Amr Ali wrote:
>> Hi,
>>
>> I'm not sure if this was discussed before, but I've got tired from having to
>> work around the lack of native recursion within `shred`. So, attached is a
>> patch
>> to add recursion, including documentation.
>
> Thanks for taking the effort to write a patch, but we probably won't
> accept this upstream.
>
> The philosophy of the Unix toolset is for each tool to do one thing, and
> do it well, because then you can piece tools together to do larger
> things. find(1) already exists to add recursion to ANY program, so
> rather than bloat shred(1) with recursion, you should instead couple
> shred and find together:
>
> find dir -depth -exec shred --options {} +
>
--
Amr Ali
signature.asc
Description: OpenPGP digital signature
- Shred: Add recursion operations [PATCH], Amr Ali, 2011/12/09
- Re: Shred: Add recursion operations [PATCH], Eric Blake, 2011/12/09
- Re: Shred: Add recursion operations [PATCH],
Amr Ali <=
- Re: Shred: Add recursion operations [PATCH], Eric Blake, 2011/12/09
- Re: Shred: Add recursion operations [PATCH], Jim Meyering, 2011/12/09
- Re: Shred: Add recursion operations [PATCH], Amr Ali, 2011/12/09
- Re: Shred: Add recursion operations [PATCH], Eric Blake, 2011/12/09
- Re: Shred: Add recursion operations [PATCH], Amr Ali, 2011/12/09
Re: Shred: Add recursion operations [PATCH], Pádraig Brady, 2011/12/09