[Top][All Lists]

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

Re: [PATCH 14/15] scripts: environment: Add --container option.

From: Ludovic Courtès
Subject: Re: [PATCH 14/15] scripts: environment: Add --container option.
Date: Sat, 31 Oct 2015 11:28:25 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

"Thompson, David" <address@hidden> skribis:

> Hello, sorry for the delay.
> On Tue, Oct 27, 2015 at 6:13 AM, Ludovic Courtès <address@hidden> wrote:


>> What about automatically mapping $SHELL in the container when it is set?
> The issue is that the default command to evaluate is $SHELL or
> "/bin/sh."  You can be almost certain that $SHELL is not going to be
> available in the container.  What to do?  Check if the command is
> 'equal?' to (list (getenv "SHELL")) and make it '("/bin/sh") instead?

What I meant above is to to the equivalent --expose=$SHELL by default
when ‘SHELL’ is set.  Anything wrong with that?

If we did that, we could honor $SHELL even for --container (it’s already
honored in the other case.)


reply via email to

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