summaryrefslogtreecommitdiff
path: root/gnu/tests.scm
diff options
context:
space:
mode:
authorLudovic Courtès <ludo@gnu.org>2023-04-21 15:38:06 +0200
committerLudovic Courtès <ludo@gnu.org>2023-04-21 16:16:38 +0200
commita09c7da8f8d8e732f969cf0a09aaa78f87032ab1 (patch)
treee4b41bde7ae01c29722f0265a3b44ef6a8933898 /gnu/tests.scm
parentfb32e226ce3d3cd9bf12989850b2dd719266d583 (diff)
tests: Fork and exec a new Guile for the marionette REPL.
By merely forking PID 1, details from PID 1 (shepherd) would leak into the marionette process, such as the set of modules in scope and state inherited from the shepherd process (<service> instances, fibers, etc.). Running a fresh Guile instance avoids that. * gnu/tests.scm (marionette-program): New procedure. (marionette-shepherd-service): Change 'start' to use 'make-forkexec-constructor', and run the result of 'marionette-program'.
Diffstat (limited to 'gnu/tests.scm')
-rw-r--r--gnu/tests.scm112
1 files changed, 60 insertions, 52 deletions
diff --git a/gnu/tests.scm b/gnu/tests.scm
index ca677d315b..96ecb40ea2 100644
--- a/gnu/tests.scm
+++ b/gnu/tests.scm
@@ -1,5 +1,5 @@
;;; GNU Guix --- Functional package management for GNU
-;;; Copyright © 2016-2020, 2022 Ludovic Courtès <ludo@gnu.org>
+;;; Copyright © 2016-2020, 2022-2023 Ludovic Courtès <ludo@gnu.org>
;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr>
;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
@@ -88,6 +88,61 @@
(with-extensions extensions
gexp)))
+(define (marionette-program device imported-modules extensions)
+ "Return the program that runs the marionette REPL on DEVICE. Ensure
+IMPORTED-MODULES and EXTENSIONS are accessible from the REPL."
+ (define code
+ (with-imported-modules-and-extensions
+ `((guix build utils)
+ (guix build syscalls)
+ ,@imported-modules)
+ extensions
+ #~(begin
+ (use-modules (ice-9 match)
+ (ice-9 binary-ports))
+
+ (define (self-quoting? x)
+ (letrec-syntax ((one-of (syntax-rules ()
+ ((_) #f)
+ ((_ pred rest ...)
+ (or (pred x)
+ (one-of rest ...))))))
+ (one-of symbol? string? keyword? pair? null? array?
+ number? boolean? char?)))
+
+ (let ((repl (open-file #$device "r+0"))
+ (console (open-file "/dev/console" "r+0")))
+ ;; Redirect output to the console.
+ (close-fdes 1)
+ (close-fdes 2)
+ (dup2 (fileno console) 1)
+ (dup2 (fileno console) 2)
+ (close-port console)
+
+ (display 'ready repl)
+ (let loop ()
+ (newline repl)
+
+ (match (read repl)
+ ((? eof-object?)
+ (primitive-exit 0))
+ (expr
+ (catch #t
+ (lambda ()
+ (let ((result (primitive-eval expr)))
+ (write (if (self-quoting? result)
+ result
+ (object->string result))
+ repl)))
+ (lambda (key . args)
+ (print-exception (current-error-port)
+ (stack-ref (make-stack #t) 1)
+ key args)
+ (write #f repl)))))
+ (loop))))))
+
+ (program-file "marionette-repl.scm" code))
+
(define (marionette-shepherd-service config)
"Return the Shepherd service for the marionette REPL"
(match config
@@ -101,57 +156,10 @@
(modules '((ice-9 match)
(srfi srfi-9 gnu)))
- (start
- (with-imported-modules-and-extensions imported-modules extensions
- #~(lambda ()
- (define (self-quoting? x)
- (letrec-syntax ((one-of (syntax-rules ()
- ((_) #f)
- ((_ pred rest ...)
- (or (pred x)
- (one-of rest ...))))))
- (one-of symbol? string? keyword? pair? null? array?
- number? boolean? char?)))
-
- (match (primitive-fork)
- (0
- (dynamic-wind
- (const #t)
- (lambda ()
- (let ((repl (open-file #$device "r+0"))
- (console (open-file "/dev/console" "r+0")))
- ;; Redirect output to the console.
- (close-fdes 1)
- (close-fdes 2)
- (dup2 (fileno console) 1)
- (dup2 (fileno console) 2)
- (close-port console)
-
- (display 'ready repl)
- (let loop ()
- (newline repl)
-
- (match (read repl)
- ((? eof-object?)
- (primitive-exit 0))
- (expr
- (catch #t
- (lambda ()
- (let ((result (primitive-eval expr)))
- (write (if (self-quoting? result)
- result
- (object->string result))
- repl)))
- (lambda (key . args)
- (print-exception (current-error-port)
- (stack-ref (make-stack #t) 1)
- key args)
- (write #f repl)))))
- (loop))))
- (lambda ()
- (primitive-exit 1))))
- (pid
- pid)))))
+ (start #~(make-forkexec-constructor
+ (list #$(marionette-program device
+ imported-modules
+ extensions))))
(stop #~(make-kill-destructor)))))))
(define marionette-service-type