Re: bugfix eshell/rm

[Thierry Volpiatto]

Glenn Morris <address@hidden> writes:

> Thierry Volpiatto wrote:
>
>> `eshell-remove-entries' use a not bound 'verbose' variable, and fail.
>
> It's dynamically bound in eshell/rm via eshell-eval-using-options.
>
>> To reproduce bug, try "rm -rf /some/directory" in eshell.
>
> Works fine for me.
Sorry to insist, but something is wrong here.
On emacs-23.3, if you turn on `delete-by-moving-to-trash',
rm -rf hang forever.
This is because in `eshell-remove-entries', `delete-directory' and
`delete-file' take respectively 3 and 2 arguments.
This is correct for emacs24 but not for emacs23.

Use a consequent directory to test, i.e not a too small directory.

So of course in emacs24 this is working fine.

Maybe some changes in em-unix.el for emacs24 have been merged in
emacs-23 branch without appropriate modifications?


-- 
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997