all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] build: container: Fix call-with-clean-exit.
@ 2015-10-09 20:26 David Thompson
  2015-10-09 21:59 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: David Thompson @ 2015-10-09 20:26 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 92 bytes --]

An oversight on my part that I noticed while doing 'guix environment
--container' testing.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-build-container-Fix-call-with-clean-exit.patch --]
[-- Type: text/x-patch, Size: 1429 bytes --]

From dc72ee831d6c6b93cf404a7fd1abc4b8fb27b7f2 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Fri, 9 Oct 2015 12:33:40 -0400
Subject: [PATCH] build: container: Fix call-with-clean-exit.

Before, call-with-clean-exit would *always* return an exit code of 1.

* gnu/build/linux-container.scm (call-with-clean-exit): Exit with status
  code of 0 if thunk does not throw an exception.
* tests/containers.scm: Add test.
---
 gnu/build/linux-container.scm | 4 +++-
 tests/containers.scm          | 4 ++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/build/linux-container.scm b/gnu/build/linux-container.scm
index 95220d0..e911494 100644
--- a/gnu/build/linux-container.scm
+++ b/gnu/build/linux-container.scm
@@ -36,7 +36,9 @@
   "Apply THUNK, but exit with a status code of 1 if it fails."
   (dynamic-wind
     (const #t)
-    thunk
+    (lambda ()
+      (thunk)
+      (primitive-exit 0))
     (lambda ()
       (primitive-exit 1))))
 
diff --git a/tests/containers.scm b/tests/containers.scm
index 4783f8e..0ba8149 100644
--- a/tests/containers.scm
+++ b/tests/containers.scm
@@ -34,6 +34,10 @@
 
 (test-begin "containers")
 
+(test-assert "call-with-container, exit with 0 when there is no error"
+  (zero?
+   (call-with-container '() (const #t) #:namespaces '(user))))
+
 (test-assert "call-with-container, user namespace"
   (zero?
    (call-with-container '()
-- 
2.5.0


[-- Attachment #3: Type: text/plain, Size: 38 bytes --]


-- 
David Thompson
GPG Key: 0FF1D807

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] build: container: Fix call-with-clean-exit.
  2015-10-09 20:26 [PATCH] build: container: Fix call-with-clean-exit David Thompson
@ 2015-10-09 21:59 ` Ludovic Courtès
  2015-10-10 13:01   ` Thompson, David
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2015-10-09 21:59 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

David Thompson <dthompson2@worcester.edu> skribis:

> From dc72ee831d6c6b93cf404a7fd1abc4b8fb27b7f2 Mon Sep 17 00:00:00 2001
> From: David Thompson <dthompson2@worcester.edu>
> Date: Fri, 9 Oct 2015 12:33:40 -0400
> Subject: [PATCH] build: container: Fix call-with-clean-exit.
>
> Before, call-with-clean-exit would *always* return an exit code of 1.
>
> * gnu/build/linux-container.scm (call-with-clean-exit): Exit with status
>   code of 0 if thunk does not throw an exception.
> * tests/containers.scm: Add test.

LGTM, thanks.

Ludo’.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] build: container: Fix call-with-clean-exit.
  2015-10-09 21:59 ` Ludovic Courtès
@ 2015-10-10 13:01   ` Thompson, David
  0 siblings, 0 replies; 3+ messages in thread
From: Thompson, David @ 2015-10-10 13:01 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

On Fri, Oct 9, 2015 at 5:59 PM, Ludovic Courtès <ludo@gnu.org> wrote:
> David Thompson <dthompson2@worcester.edu> skribis:
>
>> From dc72ee831d6c6b93cf404a7fd1abc4b8fb27b7f2 Mon Sep 17 00:00:00 2001
>> From: David Thompson <dthompson2@worcester.edu>
>> Date: Fri, 9 Oct 2015 12:33:40 -0400
>> Subject: [PATCH] build: container: Fix call-with-clean-exit.
>>
>> Before, call-with-clean-exit would *always* return an exit code of 1.
>>
>> * gnu/build/linux-container.scm (call-with-clean-exit): Exit with status
>>   code of 0 if thunk does not throw an exception.
>> * tests/containers.scm: Add test.
>
> LGTM, thanks.

Pushed, thanks.

- Dave

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-10-10 13:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-09 20:26 [PATCH] build: container: Fix call-with-clean-exit David Thompson
2015-10-09 21:59 ` Ludovic Courtès
2015-10-10 13:01   ` Thompson, David

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.