Re: Shred: Add recursion operations [PATCH] (original) (raw)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

From: Eric Blake
Subject: Re: Shred: Add recursion operations [PATCH]
Date: Fri, 09 Dec 2011 10:34:41 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111115 Thunderbird/8.0

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 {} +

-- Eric Blake address@hidden +1-919-301-3266 Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature