[PATCH]: gnu: python-jsonschema: Correct inputs.

[PATCH]: gnu: python-jsonschema: Correct inputs.

[Marius Bakke]

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


Hello Guix,

This fixes a problem building python2-tempest-lib due to missing
propagation of python2-functools32.

Note that the native-inputs are still referenced due to a bug with how
the automatic wrapper of $out/bin works. Should they still be added as
regular inputs, or can we ignore it for now? I *think* this is fixed in
wip-python-build-system, but haven't checked yet (still building
dependents).


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-python-jsonschema-Correct-inputs.patch --]
[-- Type: text/x-patch, Size: 1862 bytes --]

From 929b9caa995cc0b75a8a5cd8d4b85102c240d2f6 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Tue, 8 Nov 2016 15:50:13 +0000
Subject: [PATCH] gnu: python-jsonschema: Correct inputs.

* gnu/packages/python.scm (python-jsonschema)[inputs]: Remove
python-setuptools. Move python-vcversioner to ...
[native-inputs]: ... here.
* gnu/packages/python.scm (python2-jsonschema)[inputs]: Move
python2-functools32 to ...
[propagated-inputs]: ... here.
[native-inputs]: Add python2-setuptools.
---
 gnu/packages/python.scm | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 5b5287e..8407b2d 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -2474,9 +2474,8 @@ version numbers.")
               (base32
                "0hddbqjm4jq63y8jf44nswina1crjs16l9snb6m3vvgyg31klrrn"))))
     (build-system python-build-system)
-    (inputs
-     `(("python-setuptools" ,python-setuptools)
-       ("python-vcversioner" ,python-vcversioner)))
+    (native-inputs
+     `(("python-vcversioner" ,python-vcversioner)))
     (home-page "http://github.com/Julian/jsonschema")
     (synopsis "Implementation of JSON Schema for Python")
     (description
@@ -2488,9 +2487,11 @@ version numbers.")
   (let ((jsonschema (package-with-python2
                      (strip-python2-variant python-jsonschema))))
     (package (inherit jsonschema)
-      (inputs
-       `(("python2-functools32" ,python2-functools32)
-         ,@(package-inputs jsonschema))))))
+             (native-inputs
+              `(("python2-setuptools" ,python2-setuptools)
+                ,@(package-native-inputs jsonschema)))
+             (propagated-inputs
+              `(("python2-functools32" ,python2-functools32))))))
 
 (define-public python-unidecode
   (package
-- 
2.10.2

Re: [PATCH]: gnu: python-jsonschema: Correct inputs.

[Leo Famulari]

On Tue, Nov 08, 2016 at 04:05:53PM +0000, Marius Bakke wrote:
> This fixes a problem building python2-tempest-lib due to missing
> propagation of python2-functools32.
> 
> Note that the native-inputs are still referenced due to a bug with how
> the automatic wrapper of $out/bin works. Should they still be added as
> regular inputs, or can we ignore it for now? I *think* this is fixed in
> wip-python-build-system, but haven't checked yet (still building
> dependents).
> 

> From 929b9caa995cc0b75a8a5cd8d4b85102c240d2f6 Mon Sep 17 00:00:00 2001
> From: Marius Bakke <mbakke@fastmail.com>
> Date: Tue, 8 Nov 2016 15:50:13 +0000
> Subject: [PATCH] gnu: python-jsonschema: Correct inputs.
> 
> * gnu/packages/python.scm (python-jsonschema)[inputs]: Remove
> python-setuptools. Move python-vcversioner to ...
> [native-inputs]: ... here.
> * gnu/packages/python.scm (python2-jsonschema)[inputs]: Move
> python2-functools32 to ...
> [propagated-inputs]: ... here.
> [native-inputs]: Add python2-setuptools.

It works for me, thanks!

Hartmut, what do you think?

Re: [PATCH]: gnu: python-jsonschema: Correct inputs.

[Hartmut Goebel]

Am 08.11.2016 um 18:30 schrieb Leo Famulari:
>> > * gnu/packages/python.scm (python-jsonschema)[inputs]: Remove
>> > python-setuptools. Move python-vcversioner to ...
>> > [native-inputs]: ... here.
>> > * gnu/packages/python.scm (python2-jsonschema)[inputs]: Move
>> > python2-functools32 to ...
>> > [propagated-inputs]: ... here.
>> > [native-inputs]: Add python2-setuptools.
> It works for me, thanks!
>
> Hartmut, what do you think?

LGTM

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH]: gnu: python-jsonschema: Correct inputs.

[Marius Bakke]

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

Hartmut Goebel <h.goebel@crazy-compilers.com> writes:

> Am 08.11.2016 um 18:30 schrieb Leo Famulari:
>>> > * gnu/packages/python.scm (python-jsonschema)[inputs]: Remove
>>> > python-setuptools. Move python-vcversioner to ...
>>> > [native-inputs]: ... here.
>>> > * gnu/packages/python.scm (python2-jsonschema)[inputs]: Move
>>> > python2-functools32 to ...
>>> > [propagated-inputs]: ... here.
>>> > [native-inputs]: Add python2-setuptools.
>> It works for me, thanks!
>>
>> Hartmut, what do you think?
>
> LGTM

OK! Pushed along with updates to python-tempest-lib and python-os-testr.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 454 bytes --]