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: Stefan Monnier
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:03:15 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)

> +  (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.
So I think that instead of removing any `foo' from the environment, we
should on the contrary add a `foo' binding in the process-environment to
make sure that the local binding takes precedence.


        Stefan



reply via email to

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