guix-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

04/19: build: julia-build-system: Fix corner-case for parallel tests.


From: guix-commits
Subject: 04/19: build: julia-build-system: Fix corner-case for parallel tests.
Date: Thu, 24 Feb 2022 05:23:43 -0500 (EST)

efraim pushed a commit to branch master
in repository guix.

commit 1f6ae9a8e02c457ee499e61f78c8a282ab9d4971
Author: zimoun <zimon.toutoune@gmail.com>
AuthorDate: Wed Feb 23 14:47:07 2022 +0100

    build: julia-build-system: Fix corner-case for parallel tests.
    
    * guix/build/julia-build-system.scm (check): Do not run parallel tests when
    '--cores=1'.
    
    Signed-off-by: Efraim Flashner <efraim@flashner.co.il>
---
 guix/build/julia-build-system.scm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/guix/build/julia-build-system.scm 
b/guix/build/julia-build-system.scm
index c5ad65d029..b0dac154e9 100644
--- a/guix/build/julia-build-system.scm
+++ b/guix/build/julia-build-system.scm
@@ -112,9 +112,9 @@ Project.toml)."
            (job-count (if parallel-tests?
                           (parallel-job-count)
                           1))
-           ;; The --proc argument of Julia *adds* extra processors rather than
-           ;; specify the exact count to use, so zero must be specified to
-           ;; disable parallel processing...
+           ;; The --procs argument of Julia *adds* extra processors rather
+           ;; than specify the exact count to use, so zero must be specified
+           ;; to disable parallel processing...
            (additional-procs (max 0 (1- job-count))))
       ;; With a patch, SOURCE_DATE_EPOCH is honored
       (setenv "SOURCE_DATE_EPOCH" "1")
@@ -127,7 +127,7 @@ Project.toml)."
       (setenv "HOME" "/tmp")
       (apply invoke "julia"
              `("--depwarn=yes"
-               ,@(if parallel-tests?
+               ,@(if (and parallel-tests? (< 0 additional-procs))
                      ;; XXX: ... but '--procs' doesn't accept 0 as a valid
                      ;; value, so just omit the argument entirely.
                      (list (string-append  "--procs="



reply via email to

[Prev in Thread] Current Thread [Next in Thread]