[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/15] gnu: build: Add Linux container module.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 08/15] gnu: build: Add Linux container module. |
Date: |
Wed, 08 Jul 2015 23:57:24 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"Thompson, David" <address@hidden> skribis:
>>> +(test-assert "call-with-container, pid namespace"
>>> + (zero?
>>> + (call-with-container '()
>>> + (lambda ()
>>> + (match (primitive-fork)
>>> + (0
>>> + ;; The first forked process in the new pid namespace is pid 2.
>>> + (assert-exit (= 2 (getpid))))
>>
>> But its parent doesn’t sees itself as PID 1?
>
> Only if it were to 'exec'. The reason being that PID namespaces are
> special in how they treat the process that created the new namespace.
> It's somewhat confusing.
Hmm, indeed. :-)
> From 83943ab47145180f13d3c08490a9ae09fccf3b92 Mon Sep 17 00:00:00 2001
> From: David Thompson <address@hidden>
> Date: Tue, 7 Jul 2015 21:58:15 -0400
> Subject: [PATCH 1/2] build: file-systems: Import (guix build syscalls) for
> non-static Guiles.
>
> * gnu/build/file-systems.scm: Import (guix build syscalls) when 'mount' is not
> defined.
> * gnu/system.scm (operating-system-activation-script): Include (guix build
> syscalls) module in derivation.
LGTM.
> From 72705fd6a8cd7b60bd727221897dc8bb79e3e4d7 Mon Sep 17 00:00:00 2001
> From: David Thompson <address@hidden>
> Date: Tue, 2 Jun 2015 08:48:16 -0400
> Subject: [PATCH 2/2] gnu: build: Add Linux container module.
>
> * gnu/build/linux-container.scm: New file.
> * gnu-system.am (GNU_SYSTEM_MODULES): Add it.
> * .dir-locals.el: Add Scheme indent rules for 'call-with-clone', 'with-clone',
> 'call-with-container', and 'container-excursion'.
> * tests/containers.scm: New file.
> * Makefile.am (SCM_TESTS): Add it.
OK!
Thanks,
Ludo'.
- Re: [PATCH 10/15] gnu: system: Move file-system->spec to (gnu system file-systems)., (continued)
[PATCH 12/15] gnu: system: Add Linux container file systems., David Thompson, 2015/07/06
[PATCH 14/15] scripts: environment: Add --container option., David Thompson, 2015/07/06
[PATCH 11/15] gnu: system: Add Linux container module., David Thompson, 2015/07/06