[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: |
Sun, 25 Oct 2015 22:38:12 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"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.)
OK to push with this change! Exciting stuff, thank you!
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 <=
- 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, 2015/10/31