[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 14/15] scripts: environment: Add --container option.
From: |
Thompson, David |
Subject: |
Re: [PATCH 14/15] scripts: environment: Add --container option. |
Date: |
Sun, 25 Oct 2015 20:35:02 -0400 |
On Sun, Oct 25, 2015 at 5:38 PM, Ludovic Courtès <address@hidden> wrote:
> "Thompson, David" <address@hidden> skribis:
>
>> From a61c65357174263790a55e8785fc625209e11324 Mon Sep 17 00:00:00 2001
>> From: David Thompson <address@hidden>
>> Date: Fri, 19 Jun 2015 08:57:44 -0400
>> Subject: [PATCH] scripts: environment: Add --container option.
>>
>> * guix/scripts/system.scm (specification->file-system-mapping): Move from
>> here...
>> * guix/ui.scm (specification->file-system-mapping): ... to here.
>> * guix/scripts/enviroment.scm (show-help): Show help for new options.
>> (%options): Add --container --network, --expose, and --share options.
>> (%network-configuration-files): New variable.
>> (launch-environment, launch-environment/container, requisites*,
>> inputs->requisites): New procedures.
>> (guix-environment): Spawn new process in a container when requested.
>> * doc/guix.texi (Invoking guix environment): Document it.
>> * tests/guix-environment-container.sh: New file.
>> * Makefile.am (SH_TESTS): Add it.
>
> [...]
>
>> --- a/tests/guix-environment.sh
>> +++ b/tests/guix-environment.sh
>> @@ -55,6 +55,15 @@ else
>> test $? = 42
>> fi
>>
>> +# Make sure the exit value is preserved for containers, too.
>> +if guix environment --container --ad-hoc --bootstrap guile-bootstrap \
>> + -- guile -c '(exit 42)'
>> +then
>> + false
>> +else
>> + test $? = 42
>> +fi
>
> I think this bit is a leftover from a previous attempt that can now be
> removed (guix-environment-container.sh contains the same test.)
Thanks for catching this. I could've sworn I had deduplicated that!
> OK to push with this change! Exciting stuff, thank you!
Thanks for all of your help. Pushed! Woooooooooooooooooo!!!
- Dave
- 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 <=
- 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, 2015/10/31