[bug#35890] [PATCH 3/4] gnu: python-hdf4: Update to 0.9.2.

[bug#35890] [PATCH 3/4] gnu: python-hdf4: Update to 0.9.2.

[Brett Gilio]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: [PATCH 3/4] gnu: python-hdf4: Update to 0.9.2. --]
[-- Type: text/x-patch, Size: 1124 bytes --]

From fb102715cd2b809e4d963b8f71df54bbad06c369 Mon Sep 17 00:00:00 2001
From: Brett Gilio <brettg@posteo.net>
Date: Fri, 24 May 2019 19:10:33 -0500
Subject: [PATCH 3/4] gnu: python-hdf4: Update to 0.9.2.

* gnu/packages/python-xyz.scm (python-hdf4): Update to 0.9.2.
---
 gnu/packages/python-xyz.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index a58b788ca1..54cc0c2b27 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -370,14 +370,14 @@ and verifies that it matches the intended target hostname.")
 (define-public python-hdf4
   (package
    (name "python-hdf4")
-   (version "0.9")
+   (version "0.9.2")
    (source
     (origin
       (method url-fetch)
       (uri (pypi-uri name version))
       (sha256
        (base32
-        "1hjiyrxvxk9817qyqky3nar4y3fs4z8wxz0n884zzb5wi6skrjks"))))
+        "00sxppysk3w620g1jdskjzkybvpf8dkpzjfj3wlw5khpzw1g0hq5"))))
    (build-system python-build-system)
    (native-inputs `(("nose" ,python-nose)))
    (propagated-inputs `(("numpy" ,python-numpy)))
-- 
2.21.0

[bug#35890] [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.

[Brett Gilio]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument. --]
[-- Type: text/x-patch, Size: 1474 bytes --]

From 74c956284d5ff7b20e12ed4315404697320dc809 Mon Sep 17 00:00:00 2001
From: Brett Gilio <brettg@posteo.net>
Date: Fri, 24 May 2019 19:11:00 -0500
Subject: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.

* gnu/packages/python-xyz.scm (python-hdf4): Update to .
---
 gnu/packages/python-xyz.scm | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 54cc0c2b27..43287d212e 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -385,21 +385,6 @@ and verifies that it matches the intended target hostname.")
     `(("hdf4" ,hdf4)
       ("libjpeg" ,libjpeg)
       ("zlib" ,zlib)))
-   (arguments
-    `(#:phases
-      (modify-phases %standard-phases
-        (replace 'check
-          (lambda _
-            ;; The 'runexamples' script sets PYTHONPATH to CWD, then goes
-            ;; on to import numpy. Somehow this works on their CI system.
-            ;; Let's just manage PYTHONPATH here instead.
-            (substitute* "runexamples.sh"
-              (("export PYTHONPATH=.*") ""))
-            (setenv "PYTHONPATH"
-                    (string-append (getcwd) ":"
-                                   (getenv "PYTHONPATH")))
-            (invoke "./runexamples.sh")
-            (invoke "nosetests" "-v"))))))
    (home-page "https://github.com/fhs/python-hdf4")
    (synopsis "Python interface to the NCSA HDF4 library")
    (description
-- 
2.21.0

[bug#35890] [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.

[Marius Bakke]

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

Brett Gilio <brettg@posteo.net> writes:

> From 74c956284d5ff7b20e12ed4315404697320dc809 Mon Sep 17 00:00:00 2001
> From: Brett Gilio <brettg@posteo.net>
> Date: Fri, 24 May 2019 19:11:00 -0500
> Subject: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.
>
> * gnu/packages/python-xyz.scm (python-hdf4): Update to .
> ---
>  gnu/packages/python-xyz.scm | 15 ---------------
>  1 file changed, 15 deletions(-)
>
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 54cc0c2b27..43287d212e 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -385,21 +385,6 @@ and verifies that it matches the intended target hostname.")
>      `(("hdf4" ,hdf4)
>        ("libjpeg" ,libjpeg)
>        ("zlib" ,zlib)))
> -   (arguments
> -    `(#:phases
> -      (modify-phases %standard-phases
> -        (replace 'check
> -          (lambda _
> -            ;; The 'runexamples' script sets PYTHONPATH to CWD, then goes
> -            ;; on to import numpy. Somehow this works on their CI system.
> -            ;; Let's just manage PYTHONPATH here instead.
> -            (substitute* "runexamples.sh"
> -              (("export PYTHONPATH=.*") ""))
> -            (setenv "PYTHONPATH"
> -                    (string-append (getcwd) ":"
> -                                   (getenv "PYTHONPATH")))
> -            (invoke "./runexamples.sh")
> -            (invoke "nosetests" "-v"))))))
>     (home-page "https://github.com/fhs/python-hdf4")
>     (synopsis "Python interface to the NCSA HDF4 library")
>     (description

While this makes the build succeed, the 'check' phase is now moot.  It
appears to simply build the library for a second time, and then prints
'Ran 0 tests in N seconds...'.

Can you look into what happened to the tests between version 0.9.0 and
0.9.2?

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

[bug#35890] [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.

[Ludovic Courtès]

Hello Brett,

Marius Bakke <mbakke@fastmail.com> skribis:

> Brett Gilio <brettg@posteo.net> writes:
>
>> From 74c956284d5ff7b20e12ed4315404697320dc809 Mon Sep 17 00:00:00 2001
>> From: Brett Gilio <brettg@posteo.net>
>> Date: Fri, 24 May 2019 19:11:00 -0500
>> Subject: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.
>>
>> * gnu/packages/python-xyz.scm (python-hdf4): Update to .
>> ---
>>  gnu/packages/python-xyz.scm | 15 ---------------
>>  1 file changed, 15 deletions(-)
>>
>> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
>> index 54cc0c2b27..43287d212e 100644
>> --- a/gnu/packages/python-xyz.scm
>> +++ b/gnu/packages/python-xyz.scm
>> @@ -385,21 +385,6 @@ and verifies that it matches the intended target hostname.")
>>      `(("hdf4" ,hdf4)
>>        ("libjpeg" ,libjpeg)
>>        ("zlib" ,zlib)))
>> -   (arguments
>> -    `(#:phases
>> -      (modify-phases %standard-phases
>> -        (replace 'check
>> -          (lambda _
>> -            ;; The 'runexamples' script sets PYTHONPATH to CWD, then goes
>> -            ;; on to import numpy. Somehow this works on their CI system.
>> -            ;; Let's just manage PYTHONPATH here instead.
>> -            (substitute* "runexamples.sh"
>> -              (("export PYTHONPATH=.*") ""))
>> -            (setenv "PYTHONPATH"
>> -                    (string-append (getcwd) ":"
>> -                                   (getenv "PYTHONPATH")))
>> -            (invoke "./runexamples.sh")
>> -            (invoke "nosetests" "-v"))))))
>>     (home-page "https://github.com/fhs/python-hdf4")
>>     (synopsis "Python interface to the NCSA HDF4 library")
>>     (description
>
> While this makes the build succeed, the 'check' phase is now moot.  It
> appears to simply build the library for a second time, and then prints
> 'Ran 0 tests in N seconds...'.
>
> Can you look into what happened to the tests between version 0.9.0 and
> 0.9.2?

Any update on this?

Ludo’.

[bug#35890] [PATCH 3/4] gnu: python-hdf4: Update to 0.9.2.

[Maxim Cournoyer]

Hi,

Ludovic Courtès <ludo@gnu.org> writes:

> Hello Brett,
>
> Marius Bakke <mbakke@fastmail.com> skribis:
>
>> Brett Gilio <brettg@posteo.net> writes:
>>
>>> From 74c956284d5ff7b20e12ed4315404697320dc809 Mon Sep 17 00:00:00 2001
>>> From: Brett Gilio <brettg@posteo.net>
>>> Date: Fri, 24 May 2019 19:11:00 -0500
>>> Subject: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.
>>>
>>> * gnu/packages/python-xyz.scm (python-hdf4): Update to .
>>> ---
>>>  gnu/packages/python-xyz.scm | 15 ---------------
>>>  1 file changed, 15 deletions(-)
>>>
>>> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
>>> index 54cc0c2b27..43287d212e 100644
>>> --- a/gnu/packages/python-xyz.scm
>>> +++ b/gnu/packages/python-xyz.scm
>>> @@ -385,21 +385,6 @@ and verifies that it matches the intended target hostname.")
>>>      `(("hdf4" ,hdf4)
>>>        ("libjpeg" ,libjpeg)
>>>        ("zlib" ,zlib)))
>>> -   (arguments
>>> -    `(#:phases
>>> -      (modify-phases %standard-phases
>>> -        (replace 'check
>>> -          (lambda _
>>> -            ;; The 'runexamples' script sets PYTHONPATH to CWD, then goes
>>> -            ;; on to import numpy. Somehow this works on their CI system.
>>> -            ;; Let's just manage PYTHONPATH here instead.
>>> -            (substitute* "runexamples.sh"
>>> -              (("export PYTHONPATH=.*") ""))
>>> -            (setenv "PYTHONPATH"
>>> -                    (string-append (getcwd) ":"
>>> -                                   (getenv "PYTHONPATH")))
>>> -            (invoke "./runexamples.sh")
>>> -            (invoke "nosetests" "-v"))))))
>>>     (home-page "https://github.com/fhs/python-hdf4")
>>>     (synopsis "Python interface to the NCSA HDF4 library")
>>>     (description
>>
>> While this makes the build succeed, the 'check' phase is now moot.  It
>> appears to simply build the library for a second time, and then prints
>> 'Ran 0 tests in N seconds...'.
>>
>> Can you look into what happened to the tests between version 0.9.0 and
>> 0.9.2?
>
> Any update on this?

No update in 3 years; closing.

Maxim

[bug#35890] [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.

[Ricardo Wurmus]

Brett Gilio <brettg@posteo.net> writes:

> From 74c956284d5ff7b20e12ed4315404697320dc809 Mon Sep 17 00:00:00 2001
> From: Brett Gilio <brettg@posteo.net>
> Date: Fri, 24 May 2019 19:11:00 -0500
> Subject: [PATCH 4/4] gnu: python-hdf4: Remove check-phase argument.
>
> * gnu/packages/python-xyz.scm (python-hdf4): Update to .

I guess you’re using the yasnippets for Emacs :)

It’s fine to do this in the same commit as the upgrade, but as Marius
wrote the tests should actually be run.

-- 
Ricardo