all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] Fix scipy
@ 2015-10-13 11:37 Ricardo Wurmus
  2015-10-13 11:41 ` Cyril Roelandt
  2015-10-13 14:32 ` Ludovic Courtès
  0 siblings, 2 replies; 3+ messages in thread
From: Ricardo Wurmus @ 2015-10-13 11:37 UTC (permalink / raw
  To: guix-devel@gnu.org

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]

The attached patch fixes python{,2}-scipy.  Currently, it does not
propagate Python modules it depends on at runtime, such as numpy.

I suspect that we need to fix many more Python packages in a similar
manner.

~~ Ricardo


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-python-scipy-python2-scipy-Propagate-Python-inpu.patch --]
[-- Type: text/x-patch, Size: 2586 bytes --]

From 73199a178d34eb5202906ea7ef6921c09d42ea28 Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Tue, 13 Oct 2015 13:29:49 +0200
Subject: [PATCH] gnu: python-scipy, python2-scipy: Propagate Python inputs.

* gnu/packages/python.scm (python-scipy)[inputs]: Move "python-numpy",
  "python-matplotlib", and "python-pyparsing" to propagated-inputs; move
  "python-nose" and "python-sphinx" to native-inputs.
  [propagated-inputs]: New field.
  [native-inputs]: Move "python-nose" and "python-sphinx" from inputs
  field here.
  (python2-scipy): Modify propagated-inputs instead of inputs field.
---
 gnu/packages/python.scm | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 72d353d..5c44a92 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -3244,16 +3244,17 @@ functions.")
         (base32
          "0fsqi05s035d7p6s8h3h2pvk1axias16chy17rw9l1bxvrfhmncf"))))
     (build-system python-build-system)
-    (inputs
+    (propagated-inputs
      `(("python-numpy" ,python-numpy)
        ("python-matplotlib" ,python-matplotlib)
-       ("python-pyparsing" ,python-pyparsing)
-       ("python-nose" ,python-nose)
-       ("python-sphinx" ,python-sphinx)
-       ("lapack" ,lapack)
+       ("python-pyparsing" ,python-pyparsing)))
+    (inputs
+     `(("lapack" ,lapack)
        ("openblas" ,openblas)))
     (native-inputs
-     `(("gfortran" ,gfortran)
+     `(("python-nose" ,python-nose)
+       ("python-sphinx" ,python-sphinx)
+       ("gfortran" ,gfortran)
        ("texlive" ,texlive)
        ("perl" ,perl)))
     (outputs '("out" "doc"))
@@ -3322,11 +3323,12 @@ routines such as routines for numerical integration and optimization.")
   (let ((scipy (package-with-python2 python-scipy)))
     (package (inherit scipy)
       ;; Use packages customized for python-2.
-      (inputs `(("python2-matplotlib" ,python2-matplotlib)
-                ("python2-numpy" ,python2-numpy)
-                ,@(alist-delete "python-matplotlib"
-                                (alist-delete "python-numpy"
-                                              (package-inputs scipy))))))))
+      (propagated-inputs
+       `(("python2-matplotlib" ,python2-matplotlib)
+         ("python2-numpy" ,python2-numpy)
+         ,@(alist-delete "python-matplotlib"
+                         (alist-delete "python-numpy"
+                                       (package-propagated-inputs scipy))))))))
 
 (define-public python-sqlalchemy
   (package
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Fix scipy
  2015-10-13 11:37 [PATCH] Fix scipy Ricardo Wurmus
@ 2015-10-13 11:41 ` Cyril Roelandt
  2015-10-13 14:32 ` Ludovic Courtès
  1 sibling, 0 replies; 3+ messages in thread
From: Cyril Roelandt @ 2015-10-13 11:41 UTC (permalink / raw
  To: guix-devel

On 10/13/2015 01:37 PM, Ricardo Wurmus wrote:
> I suspect that we need to fix many more Python packages in a similar
> manner.
> 

Yes, probably. I think one has to grep a lot when adding a new package,
in order to see whether the inputs are actually imported in the code.

Cyril.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Fix scipy
  2015-10-13 11:37 [PATCH] Fix scipy Ricardo Wurmus
  2015-10-13 11:41 ` Cyril Roelandt
@ 2015-10-13 14:32 ` Ludovic Courtès
  1 sibling, 0 replies; 3+ messages in thread
From: Ludovic Courtès @ 2015-10-13 14:32 UTC (permalink / raw
  To: Ricardo Wurmus; +Cc: guix-devel@gnu.org

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:

> From 73199a178d34eb5202906ea7ef6921c09d42ea28 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Tue, 13 Oct 2015 13:29:49 +0200
> Subject: [PATCH] gnu: python-scipy, python2-scipy: Propagate Python inputs.
>
> * gnu/packages/python.scm (python-scipy)[inputs]: Move "python-numpy",
>   "python-matplotlib", and "python-pyparsing" to propagated-inputs; move
>   "python-nose" and "python-sphinx" to native-inputs.
>   [propagated-inputs]: New field.
>   [native-inputs]: Move "python-nose" and "python-sphinx" from inputs
>   field here.
>   (python2-scipy): Modify propagated-inputs instead of inputs field.

Sure, thanks.

Ludo’.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-10-13 14:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-13 11:37 [PATCH] Fix scipy Ricardo Wurmus
2015-10-13 11:41 ` Cyril Roelandt
2015-10-13 14:32 ` Ludovic Courtès

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.