From ef8742ea69b8998320d4815e2d9820432852517d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ludovic=20Court=C3=A8s?= Date: Fri, 27 Nov 2015 10:59:56 +0100 Subject: [PATCH] gnu: eigen: Really run tests in verbose mode. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Suggested by Gaël Guennebaud . * gnu/packages/algebra.scm (eigen)[arguments]: Remove #:test-target. Rename 'build-tests' phase to 'check', and have it run 'ctest -V'. --- gnu/packages/algebra.scm | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/gnu/packages/algebra.scm b/gnu/packages/algebra.scm index e9edf36d1e..25e1af1750 100644 --- a/gnu/packages/algebra.scm +++ b/gnu/packages/algebra.scm @@ -409,26 +409,18 @@ (define-public eigen '(;; Turn off debugging symbols to save space. #:build-type "Release" - ;; Use 'make check', as per - ;; . - #:test-target "check" - #:phases (modify-phases %standard-phases - (add-before 'check 'build-tests + (replace 'check (lambda _ - ;; First build the tests, in parallel. - ;; See . (let* ((cores (parallel-job-count)) (dash-j (format #f "-j~a" cores))) - ;; These variables are supposed to be honored. - (setenv "EIGEN_MAKE_ARGS" dash-j) + ;; First build the tests, in parallel. See + ;; . + (and (zero? (system* "make" "buildtests" dash-j)) - ;; Use '-V' to get more details in case of test - ;; failures. - (setenv "EIGEN_CTEST_ARGS" - (string-append "-V " dash-j)) - - (zero? (system* "make" "buildtests" dash-j)))))))) + ;; Then run 'CTest' with -V so we get more + ;; details upon failure. + (zero? (system* "ctest" "-V" dash-j))))))))) (home-page "http://eigen.tuxfamily.org") (synopsis "C++ template library for linear algebra") (description