[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bugfix eshell/rm
From: |
Thierry Volpiatto |
Subject: |
Re: bugfix eshell/rm |
Date: |
Sat, 12 Mar 2011 09:45:00 +0100 |
User-agent: |
Gnus/5.110014 (No Gnus v0.14) Emacs/23.3 (gnu/linux) |
Thierry Volpiatto <address@hidden> writes:
> 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?
So now back in emacs23, if i remove the 3 and 2 arguments to
delete-directory and delete-file, reeval eshell-remove-entries and
eshell/rm, all work fine.
--8<---------------cut here---------------start------------->8---
(eshell-funcalln 'delete-directory (car files) t t)
^ removed
(eshell-funcalln 'delete-file (car files) t)
^ removed
--8<---------------cut here---------------end--------------->8---
--
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
- Re: bugfix eshell/rm, (continued)
- Re: bugfix eshell/rm, Thierry Volpiatto, 2011/03/11
- Re: bugfix eshell/rm, Eli Zaretskii, 2011/03/11
- Re: bugfix eshell/rm, Thierry Volpiatto, 2011/03/11
- Re: bugfix eshell/rm, Eli Zaretskii, 2011/03/11
- Re: bugfix eshell/rm, Thierry Volpiatto, 2011/03/11
- Re: bugfix eshell/rm, Glenn Morris, 2011/03/12
- bug and dev mailing lists [was: bugfix eshell/rm], Drew Adams, 2011/03/12
- Re: bug and dev mailing lists [was: bugfix eshell/rm], Eli Zaretskii, 2011/03/12
- RE: bug and dev mailing lists [was: bugfix eshell/rm], Drew Adams, 2011/03/12
Re: bugfix eshell/rm, Thierry Volpiatto, 2011/03/12
- Re: bugfix eshell/rm,
Thierry Volpiatto <=