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

2015-10-31 Thread Ludovic Courtès
"Thompson, David" skribis: > Hello, sorry for the delay. > > On Tue, Oct 27, 2015 at 6:13 AM, Ludovic Courtès 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 alm

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

2015-10-30 Thread Thompson, David
Hello, sorry for the delay. On Tue, Oct 27, 2015 at 6:13 AM, Ludovic Courtès wrote: > Hello! > > One thing I noticed is that ‘guix environment --container’ behaves > sub-optimally when ‘SHELL’ is set or set to the empty string: > > --8<---cut here---start->8---

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

2015-10-27 Thread Ludovic Courtès
Hello! One thing I noticed is that ‘guix environment --container’ behaves sub-optimally when ‘SHELL’ is set or set to the empty string: --8<---cut here---start->8--- $ guix environment alta --pure --container [...] In execvp of /home/ludo/.guix-profile/bin/ba

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

2015-10-25 Thread Thompson, David
On Sun, Oct 25, 2015 at 5:38 PM, Ludovic Courtès wrote: > "Thompson, David" skribis: > >> From a61c65357174263790a55e8785fc625209e11324 Mon Sep 17 00:00:00 2001 >> From: David Thompson >> Date: Fri, 19 Jun 2015 08:57:44 -0400 >> Subject: [PATCH] scripts: environment: Add --container option. >> >

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

2015-10-25 Thread Ludovic Courtès
"Thompson, David" skribis: > From a61c65357174263790a55e8785fc625209e11324 Mon Sep 17 00:00:00 2001 > From: David Thompson > Date: Fri, 19 Jun 2015 08:57:44 -0400 > Subject: [PATCH] scripts: environment: Add --container option. > > * guix/scripts/system.scm (specification->file-system-mapping):

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

2015-10-21 Thread Thompson, David
On Sat, Oct 17, 2015 at 6:05 AM, Ludovic Courtès wrote: > l...@gnu.org (Ludovic Courtès) skribis: > > As discussed on IRC, this turned out to be due to the fact that the Bash > in Guile’s shebang was zero-sized. > > This happened because the ‘mount-file-system’ procedure would truncate > that file

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

2015-10-17 Thread Ludovic Courtès
l...@gnu.org (Ludovic Courtès) skribis: > I think I got it. First, note that: > > $ guix build guile-bootstrap > /gnu/store/aiz8db2gni401wc9fgidmcggxyb1czis-guile-bootstrap-2.0 > $ cat > /gnu/store/aiz8db2gni401wc9fgidmcggxyb1czis-guile-bootstrap-2.0/bin/guile > #!/gnu/store/gvwf71vddp8c1d7ydq

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

2015-10-11 Thread Ludovic Courtès
Hi! I think I got it. First, note that: --8<---cut here---start->8--- $ guix build guile-bootstrap /gnu/store/aiz8db2gni401wc9fgidmcggxyb1czis-guile-bootstrap-2.0 $ cat /gnu/store/aiz8db2gni401wc9fgidmcggxyb1czis-guile-bootstrap-2.0/bin/guile #!/gnu/store/gv

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

2015-09-11 Thread Ludovic Courtès
"Thompson, David" skribis: > On Tue, Jul 7, 2015 at 10:35 AM, Ludovic Courtès wrote: [...] > I have a shiny new patch that adds --network, --share, and --expose > options. Neat! > Also, rather than bind-mounting the entire store, I figured out how to > bind-mount only the union of the closur

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

2015-09-05 Thread Thompson, David
On Tue, Jul 7, 2015 at 10:35 AM, Ludovic Courtès wrote: > David Thompson skribis: > >> * guix/scripts/enviroment.scm (show-help): Show help for new option. >> (%options): Add --container option. >> (launch-environment, launch-environment/container): New procedures. >> (guix-environment): Sp

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

2015-07-10 Thread Ludovic Courtès
"Thompson, David" skribis: > On Tue, Jul 7, 2015 at 10:35 AM, Ludovic Courtès wrote: >> Since this works without root privileges, what about adding a test in >> tests/guix-environment.sh? >> >> Basically something similar to one of the existing tests, but >> additionally checking from within th

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

2015-07-09 Thread Thompson, David
On Tue, Jul 7, 2015 at 10:35 AM, Ludovic Courtès wrote: > David Thompson skribis: > >> * guix/scripts/enviroment.scm (show-help): Show help for new option. >> (%options): Add --container option. >> (launch-environment, launch-environment/container): New procedures. >> (guix-environment): Sp

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

2015-07-07 Thread Ludovic Courtès
David Thompson skribis: > * guix/scripts/enviroment.scm (show-help): Show help for new option. > (%options): Add --container option. > (launch-environment, launch-environment/container): New procedures. > (guix-environment): Spawn new process in a container when requested. > * doc/guix.texi

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

2015-07-06 Thread David Thompson
From: David Thompson * guix/scripts/enviroment.scm (show-help): Show help for new option. (%options): Add --container option. (launch-environment, launch-environment/container): New procedures. (guix-environment): Spawn new process in a container when requested. * doc/guix.texi (Invoking gu