TakeV/guix
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

gnu: python-cython: The i686 python2 workaround is needed for 32-bit ARM too.

Browse source 
* gnu/packages/python-xyz.scm (python-cython)[arguments]: In the check phase,
disable parallel test execution for all 32-bit systems.
* gnu/packages/python-xyz.scm (python2-cython)[arguments]: Remove similar i686 workaround.
This commit is contained in:
Marius Bakke 2020-04-09 00:53:28 +02:00
parent c7eec67a93
commit d523eb5c9c
No known key found for this signature in database
GPG key ID: A2A06DF2A33A54FA
1 changed files with 7 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
gnu/packages/python-xyz.scm
View file

@ -3894,7 +3894,13 @@ (define-public python-cython
(setenv "CFLAGS" "-O0")
(invoke "python" "runtests.py" "-vv"
"-j" (number->string (parallel-job-count))))))))
"-j" (number->string (parallel-job-count))
;; XXX: On 32-bit architectures, running the parallel tests
;; fails on many-core systems, see
;; <https://github.com/cython/cython/issues/2807>.
,@(if (not (target-64bit?))
'("-x" "run.parallel")
'())))))))
(home-page "https://cython.org/")
(synopsis "C extensions for Python")
(description "Cython is an optimising static compiler for both the Python
@ -3914,18 +3920,6 @@ (define-public python2-cython
(substitute-keyword-arguments (package-arguments base)
((#:phases phases)
`(modify-phases ,phases
;; XXX: On i686-linux, running the parallel tests fails on many-core
;; systems, see <https://github.com/cython/cython/issues/2807>.
;; TODO: Move this logic to the regular check phase in a future
;; rebuild cycle.
,@(if (string-prefix? "i686" (%current-system))
'((replace 'check
(lambda _
(setenv "CFLAGS" "-O0")
(invoke "python" "runtests.py" "-vv"
"-j" (number->string (parallel-job-count))
"-x" "run.parallel"))))
'())
(add-before 'check 'adjust-test_embed
(lambda _
(substitute* "runtests.py"