[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.)
Thanks,
Ludo’.
- Re: [PATCH 14/15] scripts: environment: Add --container option., Thompson, David, 2015/10/10
- Re: [PATCH 14/15] scripts: environment: Add --container option., Ludovic Courtès, 2015/10/11
- Re: [PATCH 14/15] scripts: environment: Add --container option., Ludovic Courtès, 2015/10/17
- Re: [PATCH 14/15] scripts: environment: Add --container option., Thompson, David, 2015/10/21
- Re: [PATCH 14/15] scripts: environment: Add --container option., Ludovic Courtès, 2015/10/25
- Re: [PATCH 14/15] scripts: environment: Add --container option., Thompson, David, 2015/10/25
- Re: [PATCH 14/15] scripts: environment: Add --container option., Ludovic Courtès, 2015/10/27
- Re: [PATCH 14/15] scripts: environment: Add --container option., Thompson, David, 2015/10/30
- Re: [PATCH 14/15] scripts: environment: Add --container option.,
Ludovic Courtès <=