Re: new module

[Zefram <zefram@xxxxxxxxxxxxxxxxx>]

Zoltan Hidvegi wrote:
>There are two problems with this code.  First it cannot remove a directory
>hierarchy if it has too deep subdirectories.  That's quite bad when you use
>it to clean up directories.  A user who wants to save his files from
>automatic deletion can hide it deeply in a subdirectory.

True.  This ought to be avoided.

>Btw. I've just noticed that rm -r in files.c uses stat() instead of lstat()
>hence it follows symlinks happily :-(.

Oops.  That was not how I intended it.  I did consider that particular
problem, and other related cases, and deliberately chose lstat() over
stat() in several places.  This I think needs to be fixed immediately.

I'll fix the other problems after the next beta, unless of course you
get to them first.

-zefram