Re: [PATCH] gnu: Add python-netcdf4.

[Marius Bakke <mbakke@fastmail.com>]

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

Thomas Danckaert <thomas.danckaert@gmail.com> writes:

> From b4b54b695e286c19332600c38d5e07fabee409f8 Mon Sep 17 00:00:00 2001
> From: Thomas Danckaert <thomas.danckaert@gmail.com>
> Date: Fri, 23 Dec 2016 15:06:43 +0100
> Subject: [PATCH] gnu: Add python-netcdf4.
>
> * gnu/packages/python.scm (python-netcdf4): New variable.
> ---
>  gnu/packages/python.scm | 50 +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 50 insertions(+)

Thanks for this! The patch looks good, but the license seems to be ISC
with parts covered by the expat license according to the COPYING file.
Was there any particular reason you linked to the documentation?

> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index cc53f76..37ad3df 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -800,6 +800,56 @@ concepts.")
>  (define-public python2-h5py
>    (package-with-python2 python-h5py))
>  
> +(define-public python-netcdf4
> +  (package
> +    (name "python-netcdf4")
> +    (version "1.2.6")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (pypi-uri "netCDF4" version))
> +       (sha256
> +        (base32
> +         "1qcymsfxsdfr4sx0vl7ih5d14z66k6c9sjy4gb6rjaksk5387zvg"))))
> +    (build-system python-build-system)
> +    (native-inputs
> +     `(("python-cython" ,python-cython)))
> +    (propagated-inputs
> +     `(("python-numpy" ,python-numpy)))
> +    (inputs
> +     `(("netcdf" ,netcdf)
> +       ("hdf4" ,hdf4)
> +       ("hdf5" ,hdf5)))
> +    (arguments
> +     '(#:phases
> +       (modify-phases %standard-phases
> +         (replace 'check
> +           (lambda _
> +             (setenv "NO_NET" "1") ;; disable opendap tests
> +             (with-directory-excursion "test"
> +               (setenv "PYTHONPATH" ;; find and add the library we just built
> +                       (string-append
> +                        (car (find-files "../build" "lib.*"
> +                                         #:directories? #:t
> +                                         #:fail-on-error? #:t))
> +                        ":" (getenv "PYTHONPATH")))
> +               (zero? (system* "python" "run_all.py"))))))))
> +    (home-page
> +     "https://github.com/Unidata/netcdf4-python")
> +    (synopsis "Python/numpy interface to the netCDF library")
> +    (description "Netcdf4-python is a Python interface to the netCDF C
> +library.  netCDF version 4 has many features not found in earlier
> +versions of the library and is implemented on top of HDF5.  This module
> +can read and write files in both the new netCDF 4 and the old netCDF 3
> +format, and can create files that are readable by HDF5 clients.  The
> +API is modelled after Scientific.IO.NetCDF, and should be familiar to
> +users of that module.")
> +    (license
> +     (license:non-copyleft "https://unidata.github.io/netcdf4-python"))))
> +
> +(define-public python2-netcdf4
> +  (package-with-python2 python-netcdf4))
> +
>  (define-public python-lockfile
>    (package
>      (name "python-lockfile")
> -- 
> 2.7.4

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