emacs-devel
[Top][All Lists]
Advanced

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

Re: [Emacs-diffs] trunk r114276: * test/automated/eshell.el (eshell-test


From: Glenn Morris
Subject: Re: [Emacs-diffs] trunk r114276: * test/automated/eshell.el (eshell-test/for-loop, eshell-test/for-name-loop):
Date: Fri, 13 Sep 2013 15:15:15 -0400
User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)

Stefan Monnier wrote:

>> +  (let ((process-environment (cons "foo" process-environment)))
>> +    (should (equal (eshell-test-command-result
>> +                    "for foo in 5 { echo $foo }") 5))))
>
> The $foo above should never be able to refer to an env-var (the local
> "for binding" should always take precedence).  If not, it's a bug.

(Oh, now I see what you meant by "I saw this email too late".
Seems we read mailing lists in different order of precedence. :) )

Yeah, it's a different, long-standing eshell bug.
http://debbugs.gnu.org/15372

I just papered over it in the tests for now.
When it gets fixed, it can have its own test.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]