On Mon, Oct 20, 2014 at 11:15 PM, Eric Bavier <ericbav...@gmail.com> wrote:
>
> It's not a common need when building packages.  There are a few other
> packages that need an existing, writable $HOME when building or testing.

OK. But if the directory would exist, then the build system would
cover more packages with no extra tweaking required.

> I don't see what prompts this new module.  Leftover from development?

Yes.

> Does package-with-python2 not work here?  You might be able to do::
>
> (define-public cython2
>   (package (inherit (package-with-python2 cython))
>     (name "cython2")))

Yes, that's nicer. Changed.

Thanks for the comments!
Fede
From 28ff33bea638e9d8182b013624acbc7dedd68b4a Mon Sep 17 00:00:00 2001
From: Federico Beffa <be...@fbengineering.ch>
Date: Mon, 20 Oct 2014 19:52:45 +0200
Subject: [PATCH] gnu: Add Cython

* gnu/packages/python.scm(cython,cython2): New variables.
---
 gnu/packages/python.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 897e248..63bc588 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -1832,3 +1832,39 @@ sources.")
 
 (define-public python2-sphinx
   (package-with-python2 python-sphinx))
+
+(define-public cython
+  (package
+    (name "cython")
+    (version "0.21.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "http://cython.org/release/Cython-";
+                           version ".tar.gz"))
+       (sha256
+        (base32
+         "0ddz2l2dvcy5hdkxx4xlfiwpccvwia7ixgcy4h0pdv46a4i4vxj3"))))
+    (build-system python-build-system)
+    (inputs
+     `(("python" ,python))) ; otherwise ld doesn't find libpython3.3m.so
+    (arguments
+     `(#:phases
+       (alist-cons-before
+        'check 'set-HOME
+        ;; some tests require access to "$HOME/.cython"
+        (lambda* _ (setenv "HOME" "/tmp"))
+        (alist-replace
+         'check
+         (lambda _ (zero? (system* "python" "runtests.py" "-vv")))
+         %standard-phases))))
+    (home-page "http://cython.org/";)
+    (synopsis "C extensions for Python")
+    (description "Cython is an optimising static compiler for both the Python
+programming language and the extended Cython programming language.  It makes
+writing C extensions for Python as easy as Python itself.")
+    (license asl2.0)))
+
+(define-public cython2
+  (package (inherit (package-with-python2 cython))
+    (name "cython2")))
-- 
1.8.4

Reply via email to