Re: rm patch

[Ian Lynagh]

On Wed, Feb 07, 2001 at 10:28:36AM +0100, Jim Meyering wrote:
> Thanks for the report and patch.
> 
> However, both standards and historical practice agree that
> rm should not do as you suggest.
> I suggest you change your backup script to adjust
> the permissions, or write a small wrapper script around rm to
> do what you want.

Hmmmm, I guess I could run chmod -R u+rx first, but this seems less
efficient and a less nice solution. If I altered the patch to use a
different command line option then would this be acceptable?


Thanks
Ian

> Ian Lynagh <address@hidden> wrote:
> | The attached patch takes care of the following situation:
> |
> | mkdir foo
> | touch foo/bar (optional)
> | chmod 0 foo
> | rm -rf foo
> |
> | not working as it can't chdir to foo.
> |
> | I need this as my backup scripts back up directories like this (out of
> | necessity, they're part of a test-suite) but they then don't get
> | rotated as rm -rf fails on the oldest backup.