Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Roland Hieber
On Thu, Nov 14, 2019 at 12:40:43PM +0100, Guillermo Rodriguez Garcia wrote:
> El jue., 14 nov. 2019 a las 12:18, Roland Hieber ()
> escribió:
> 
> > On Thu, Nov 14, 2019 at 11:07:16AM +0100, Guillermo Rodriguez Garcia wrote:
> > > El jue., 14 nov. 2019 a las 10:56, Roland Hieber ()
> > > escribió:
> > >
> > > > > > numpy/core/src/multiarray/dragon4.c seems to be a license found
> > nowhere
> > > > > > else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE
> > and
> > > > add
> > > > > > its verbatim license text with startline and endline parameters in
> > > > > > PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> > > > > > texts mentioned in that variable and adds them to the license
> > report,
> > > > > > so it doesn't get lost too :)
> > > >
> > >
> > > On re-reading this: The license in dragon4.c is just MIT. So while it is
> > OK
> > > to add it to LICENSE_FILES I think we should not add the "AND unknown"
> > bit.
> >
> > Oh. I just blindly believed LICENSE.txt. But now that you say it, my
> > license matcher identifies it as "Zlib", not as "MIT".
> >
> 
> Uhm, doesn't look like Zlib to me; here's the license text extracted from
> dragon4.c:
> 
> /*
>  * Copyright (c) 2014 Ryan Juckett
>  *
>  * Permission is hereby granted, free of charge, to any person obtaining a
> copy
>  * of this software and associated documentation files (the "Software"), to
>  * deal in the Software without restriction, including without limitation
> the
>  * rights to use, copy, modify, merge, publish, distribute, sublicense,
> and/or
>  * sell copies of the Software, and to permit persons to whom the Software
> is
>  * furnished to do so, subject to the following conditions:
>  *
>  * The above copyright notice and this permission notice shall be included
> in
>  * all copies or substantial portions of the Software.
>  *
>  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> OR
>  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
>  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> THE
>  * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
>  * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>  * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> DEALINGS
>  * IN THE SOFTWARE.
>  */
> 
> And here are the MIT and Zlib licenses:
> 
> https://spdx.org/licenses/MIT.html
> https://spdx.org/licenses/Zlib.html
> 
> I would say that the license is indeed MIT, which is what LICENSE.txt says:
> 
> Name: dragon4
> Files: numpy/core/src/multiarray/dragon4.c
> License: MIT
>   For license text, see numpy/core/src/multiarray/dragon4.c

We're looking at different versions of the code. Your make file
specifies 1.16.1, and indeed that license was changed in commit
2babaaa12 ("LICENSE: update dragon4 license to MIT.") [1], which landed
in numpy 1.17.4. The _LICENSE vars should describe the version in the
make file.

Do you want to send a version-bump to the new version
instead? ;-) 

[1]: 
https://github.com/numpy/numpy/commit/2babaaa123c64c9f8d47d5b8b05942ab2c79a4e3

  - Roland


-- 
Roland Hieber, Pengutronix e.K.  | r.hie...@pengutronix.de |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany| Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Guillermo Rodriguez Garcia
El jue., 14 nov. 2019 a las 12:18, Roland Hieber ()
escribió:

> On Thu, Nov 14, 2019 at 11:07:16AM +0100, Guillermo Rodriguez Garcia wrote:
> > El jue., 14 nov. 2019 a las 10:56, Roland Hieber ()
> > escribió:
> >
> > > > > numpy/core/src/multiarray/dragon4.c seems to be a license found
> nowhere
> > > > > else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE
> and
> > > add
> > > > > its verbatim license text with startline and endline parameters in
> > > > > PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> > > > > texts mentioned in that variable and adds them to the license
> report,
> > > > > so it doesn't get lost too :)
> > >
> >
> > On re-reading this: The license in dragon4.c is just MIT. So while it is
> OK
> > to add it to LICENSE_FILES I think we should not add the "AND unknown"
> bit.
>
> Oh. I just blindly believed LICENSE.txt. But now that you say it, my
> license matcher identifies it as "Zlib", not as "MIT".
>

Uhm, doesn't look like Zlib to me; here's the license text extracted from
dragon4.c:

/*
 * Copyright (c) 2014 Ryan Juckett
 *
 * Permission is hereby granted, free of charge, to any person obtaining a
copy
 * of this software and associated documentation files (the "Software"), to
 * deal in the Software without restriction, including without limitation
the
 * rights to use, copy, modify, merge, publish, distribute, sublicense,
and/or
 * sell copies of the Software, and to permit persons to whom the Software
is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included
in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
DEALINGS
 * IN THE SOFTWARE.
 */

And here are the MIT and Zlib licenses:

https://spdx.org/licenses/MIT.html
https://spdx.org/licenses/Zlib.html

I would say that the license is indeed MIT, which is what LICENSE.txt says:

Name: dragon4
Files: numpy/core/src/multiarray/dragon4.c
License: MIT
  For license text, see numpy/core/src/multiarray/dragon4.c


BR,

Guillermo Rodriguez Garcia
guille.rodrig...@gmail.com
___
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Roland Hieber
On Thu, Nov 14, 2019 at 11:07:16AM +0100, Guillermo Rodriguez Garcia wrote:
> El jue., 14 nov. 2019 a las 10:56, Roland Hieber ()
> escribió:
> 
> > > > numpy/core/src/multiarray/dragon4.c seems to be a license found nowhere
> > > > else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE and
> > add
> > > > its verbatim license text with startline and endline parameters in
> > > > PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> > > > texts mentioned in that variable and adds them to the license report,
> > > > so it doesn't get lost too :)
> >
> 
> On re-reading this: The license in dragon4.c is just MIT. So while it is OK
> to add it to LICENSE_FILES I think we should not add the "AND unknown" bit.

Oh. I just blindly believed LICENSE.txt. But now that you say it, my
license matcher identifies it as "Zlib", not as "MIT".

> I am sending v5, sorry for the noise.

No problem, that's what review is for, and thanks for your patience  :)

 - Roland

-- 
Roland Hieber, Pengutronix e.K.  | r.hie...@pengutronix.de |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany| Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Guillermo Rodriguez Garcia
El jue., 14 nov. 2019 a las 10:56, Roland Hieber ()
escribió:

> On Thu, Nov 14, 2019 at 10:36:56AM +0100, Guillermo Rodriguez Garcia wrote:
> > Hi Roland,
> >
> > El mié., 13 nov. 2019 a las 10:36, Roland Hieber ()
> > escribió:
> >
> > > On Tue, Nov 12, 2019 at 12:51:03PM +0100, Guillermo Rodríguez wrote:
> > > > NumPy is the fundamental package for scientific computing with
> Python.
> > > >
> > > > Signed-off-by: Guillermo Rodriguez 
> > > > ---
> > > > v2: Switch to PyPi URL; this removes the dependency on Cython.
> > > > v3: Update LICENSE, add LICENSE_FILES
> > > >
> > > >  rules/python3-numpy.in   | 10 
> > > >  rules/python3-numpy.make | 54
> 
> > > >  2 files changed, 64 insertions(+)
> > > >  create mode 100644 rules/python3-numpy.in
> > > >  create mode 100644 rules/python3-numpy.make
> > > >
> > > > diff --git a/rules/python3-numpy.in b/rules/python3-numpy.in
> > > > new file mode 100644
> > > > index 0..1440e409a
> > > > --- /dev/null
> > > > +++ b/rules/python3-numpy.in
> > > > @@ -0,0 +1,10 @@
> > > > +## SECTION=python3
> > > > +
> > > > +config PYTHON3_NUMPY
> > > > + tristate
> > > > + select PYTHON3
> > > > + select HOST_PYTHON3_SETUPTOOLS
> > > > + prompt "python3-numpy"
> > > > + help
> > > > +   NumPy is the fundamental package for scientific computing
> with
> > > > +   Python.
> > > > diff --git a/rules/python3-numpy.make b/rules/python3-numpy.make
> > > > new file mode 100644
> > > > index 0..40cc351d3
> > > > --- /dev/null
> > > > +++ b/rules/python3-numpy.make
> > > > @@ -0,0 +1,54 @@
> > > > +# -*-makefile-*-
> > > > +#
> > > > +# Copyright (C) 2019 by Guillermo Rodriguez <
> guille.rodrig...@gmail.com
> > > >
> > > > +#
> > > > +# For further information about the PTXdist project and license
> > > conditions
> > > > +# see the README file.
> > > > +#
> > > > +
> > > > +#
> > > > +# We provide this package
> > > > +#
> > > > +PACKAGES-$(PTXCONF_PYTHON3_NUMPY) += python3-numpy
> > > > +
> > > > +#
> > > > +# Paths and names
> > > > +#
> > > > +PYTHON3_NUMPY_VERSION:= 1.16.1
> > > > +PYTHON3_NUMPY_MD5:= dafda51934f645d66f98424521ae
> > > > +PYTHON3_NUMPY:= numpy-$(PYTHON3_NUMPY_VERSION)
> > > > +PYTHON3_NUMPY_SUFFIX := zip
> > > > +PYTHON3_NUMPY_URL:=
> > >
> https://pypi.python.org/packages/source/n/numpy/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > > > +PYTHON3_NUMPY_SOURCE :=
> > > $(SRCDIR)/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > > > +PYTHON3_NUMPY_DIR:= $(BUILDDIR)/$(PYTHON3_NUMPY)
> > > > +PYTHON3_NUMPY_LICENSE:= BSD AND BSD-3-Clause AND MIT
> > >
> > > AND Apache-2.0, according to LICENSE.txt.
> >
> >
> > I assume you mean LICENSES_bundled.txt and not LICENSE.txt
>
> I don't have a LICENSES_bundled.txt in the extracted zip file with that
> MD5sum from that URL...
>

Right, sorry. The tarball combines the LICENSE.txt and LICENSES_bundled.txt
files from the Github repo into one single LICENSE.txt file. I mixed them
up.

Guillermo


>
> > If I am reading that correctly, the Apache license only applies to the
> > Sphinx theme. Since we are not bundling any documentation, I assume we
> can
> > ignore this one.
>
> OK. That was not clear to me.
>
>  - Roland
>
> >
> >
> > > I would also add all the other
> > > license files mentioned therein so we notice when they change or
> vanish.
> > >
> > > The Python-2.0 license in doc/scipy-sphinx-theme/LICENSE.txt is
> > > currently not available as an SPDX identifier, so I would leave this as
> > > "AND UNKNOWN", and wait until the respective SPDX issue is resolved...
> > > https://github.com/spdx/license-list-XML/issues/919
> >
> >
> > Same as above, this only applies to the Sphinx theme, so I assume we can
> > ignore this.
> >
> >
> > >
> > >
> > > numpy/core/src/multiarray/dragon4.c seems to be a license found nowhere
> > > else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE and
> add
> > > its verbatim license text with startline and endline parameters in
> > > PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> > > texts mentioned in that variable and adds them to the license report,
> > > so it doesn't get lost too :)
> > >
> >
> > OK.
> >
> >
> > >
> > > > +PYTHON3_NUMPY_LICENSE_FILES := \
> > > > + file://LICENSE.txt;md5=d26bde5432613cce2334b93985576231
> > >
> > >
> > > file://doc/sphinxext/LICENSE.txt;md5=dc37e8b18377b83250218fc557984e1a \
> > >
> > >
> file://doc/scipy-sphinx-theme/LICENSE.txt;md5=ea17c9a65c9ae0ccdf3b0a7fd1ee4616
> > > \
> > >
> > >
> file://tools/npy_tempita/license.txt;md5=c66b85ddcd09296abff87601467724fd \
> > >
> > >
> file://numpy/core/src/multiarray/dragon4.c;startline=2;endline=22;md5=19537439573c5696a922ed7957c5b37e
> > > \
> > >
> > > (numpy/linalg/lapack_lite/LICENSE.txt is currently missing in the
> > > tarball, see https://github.com/numpy/numpy/issues/13295)
> > >
> > > For reference, that last one can be generated with a

Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Roland Hieber
On Thu, Nov 14, 2019 at 10:36:56AM +0100, Guillermo Rodriguez Garcia wrote:
> Hi Roland,
> 
> El mié., 13 nov. 2019 a las 10:36, Roland Hieber ()
> escribió:
> 
> > On Tue, Nov 12, 2019 at 12:51:03PM +0100, Guillermo Rodríguez wrote:
> > > NumPy is the fundamental package for scientific computing with Python.
> > >
> > > Signed-off-by: Guillermo Rodriguez 
> > > ---
> > > v2: Switch to PyPi URL; this removes the dependency on Cython.
> > > v3: Update LICENSE, add LICENSE_FILES
> > >
> > >  rules/python3-numpy.in   | 10 
> > >  rules/python3-numpy.make | 54 
> > >  2 files changed, 64 insertions(+)
> > >  create mode 100644 rules/python3-numpy.in
> > >  create mode 100644 rules/python3-numpy.make
> > >
> > > diff --git a/rules/python3-numpy.in b/rules/python3-numpy.in
> > > new file mode 100644
> > > index 0..1440e409a
> > > --- /dev/null
> > > +++ b/rules/python3-numpy.in
> > > @@ -0,0 +1,10 @@
> > > +## SECTION=python3
> > > +
> > > +config PYTHON3_NUMPY
> > > + tristate
> > > + select PYTHON3
> > > + select HOST_PYTHON3_SETUPTOOLS
> > > + prompt "python3-numpy"
> > > + help
> > > +   NumPy is the fundamental package for scientific computing with
> > > +   Python.
> > > diff --git a/rules/python3-numpy.make b/rules/python3-numpy.make
> > > new file mode 100644
> > > index 0..40cc351d3
> > > --- /dev/null
> > > +++ b/rules/python3-numpy.make
> > > @@ -0,0 +1,54 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2019 by Guillermo Rodriguez  > >
> > > +#
> > > +# For further information about the PTXdist project and license
> > conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +#
> > > +# We provide this package
> > > +#
> > > +PACKAGES-$(PTXCONF_PYTHON3_NUMPY) += python3-numpy
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +PYTHON3_NUMPY_VERSION:= 1.16.1
> > > +PYTHON3_NUMPY_MD5:= dafda51934f645d66f98424521ae
> > > +PYTHON3_NUMPY:= numpy-$(PYTHON3_NUMPY_VERSION)
> > > +PYTHON3_NUMPY_SUFFIX := zip
> > > +PYTHON3_NUMPY_URL:=
> > https://pypi.python.org/packages/source/n/numpy/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > > +PYTHON3_NUMPY_SOURCE :=
> > $(SRCDIR)/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > > +PYTHON3_NUMPY_DIR:= $(BUILDDIR)/$(PYTHON3_NUMPY)
> > > +PYTHON3_NUMPY_LICENSE:= BSD AND BSD-3-Clause AND MIT
> >
> > AND Apache-2.0, according to LICENSE.txt.
> 
> 
> I assume you mean LICENSES_bundled.txt and not LICENSE.txt

I don't have a LICENSES_bundled.txt in the extracted zip file with that
MD5sum from that URL...

> If I am reading that correctly, the Apache license only applies to the
> Sphinx theme. Since we are not bundling any documentation, I assume we can
> ignore this one.

OK. That was not clear to me.

 - Roland

> 
> 
> > I would also add all the other
> > license files mentioned therein so we notice when they change or vanish.
> >
> > The Python-2.0 license in doc/scipy-sphinx-theme/LICENSE.txt is
> > currently not available as an SPDX identifier, so I would leave this as
> > "AND UNKNOWN", and wait until the respective SPDX issue is resolved...
> > https://github.com/spdx/license-list-XML/issues/919
> 
> 
> Same as above, this only applies to the Sphinx theme, so I assume we can
> ignore this.
> 
> 
> >
> >
> > numpy/core/src/multiarray/dragon4.c seems to be a license found nowhere
> > else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE and add
> > its verbatim license text with startline and endline parameters in
> > PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> > texts mentioned in that variable and adds them to the license report,
> > so it doesn't get lost too :)
> >
> 
> OK.
> 
> 
> >
> > > +PYTHON3_NUMPY_LICENSE_FILES := \
> > > + file://LICENSE.txt;md5=d26bde5432613cce2334b93985576231
> >
> >
> > file://doc/sphinxext/LICENSE.txt;md5=dc37e8b18377b83250218fc557984e1a \
> >
> > file://doc/scipy-sphinx-theme/LICENSE.txt;md5=ea17c9a65c9ae0ccdf3b0a7fd1ee4616
> > \
> >
> > file://tools/npy_tempita/license.txt;md5=c66b85ddcd09296abff87601467724fd \
> >
> > file://numpy/core/src/multiarray/dragon4.c;startline=2;endline=22;md5=19537439573c5696a922ed7957c5b37e
> > \
> >
> > (numpy/linalg/lapack_lite/LICENSE.txt is currently missing in the
> > tarball, see https://github.com/numpy/numpy/issues/13295)
> >
> > For reference, that last one can be generated with a
> >
> > sed -n 2,22p < inputfilename | md5sum
> >
> > Please check if all of this (especially the MD5s) make sense to you :)
> >
> 
> Yes. Will double check and submit a new version of the patch.
> 
> Thank you!
> 
> Guillermo Rodriguez Garcia
> guille.rodrig...@gmail.com

> ___
> ptxdist mailing list
> ptxdist@pengutronix.de


-- 
Roland Hieber, Pengutronix e.K.  | r.hie...@pengutronix.de |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 

Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-14 Thread Guillermo Rodriguez Garcia
Hi Roland,

El mié., 13 nov. 2019 a las 10:36, Roland Hieber ()
escribió:

> On Tue, Nov 12, 2019 at 12:51:03PM +0100, Guillermo Rodríguez wrote:
> > NumPy is the fundamental package for scientific computing with Python.
> >
> > Signed-off-by: Guillermo Rodriguez 
> > ---
> > v2: Switch to PyPi URL; this removes the dependency on Cython.
> > v3: Update LICENSE, add LICENSE_FILES
> >
> >  rules/python3-numpy.in   | 10 
> >  rules/python3-numpy.make | 54 
> >  2 files changed, 64 insertions(+)
> >  create mode 100644 rules/python3-numpy.in
> >  create mode 100644 rules/python3-numpy.make
> >
> > diff --git a/rules/python3-numpy.in b/rules/python3-numpy.in
> > new file mode 100644
> > index 0..1440e409a
> > --- /dev/null
> > +++ b/rules/python3-numpy.in
> > @@ -0,0 +1,10 @@
> > +## SECTION=python3
> > +
> > +config PYTHON3_NUMPY
> > + tristate
> > + select PYTHON3
> > + select HOST_PYTHON3_SETUPTOOLS
> > + prompt "python3-numpy"
> > + help
> > +   NumPy is the fundamental package for scientific computing with
> > +   Python.
> > diff --git a/rules/python3-numpy.make b/rules/python3-numpy.make
> > new file mode 100644
> > index 0..40cc351d3
> > --- /dev/null
> > +++ b/rules/python3-numpy.make
> > @@ -0,0 +1,54 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2019 by Guillermo Rodriguez  >
> > +#
> > +# For further information about the PTXdist project and license
> conditions
> > +# see the README file.
> > +#
> > +
> > +#
> > +# We provide this package
> > +#
> > +PACKAGES-$(PTXCONF_PYTHON3_NUMPY) += python3-numpy
> > +
> > +#
> > +# Paths and names
> > +#
> > +PYTHON3_NUMPY_VERSION:= 1.16.1
> > +PYTHON3_NUMPY_MD5:= dafda51934f645d66f98424521ae
> > +PYTHON3_NUMPY:= numpy-$(PYTHON3_NUMPY_VERSION)
> > +PYTHON3_NUMPY_SUFFIX := zip
> > +PYTHON3_NUMPY_URL:=
> https://pypi.python.org/packages/source/n/numpy/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > +PYTHON3_NUMPY_SOURCE :=
> $(SRCDIR)/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> > +PYTHON3_NUMPY_DIR:= $(BUILDDIR)/$(PYTHON3_NUMPY)
> > +PYTHON3_NUMPY_LICENSE:= BSD AND BSD-3-Clause AND MIT
>
> AND Apache-2.0, according to LICENSE.txt.


I assume you mean LICENSES_bundled.txt and not LICENSE.txt

If I am reading that correctly, the Apache license only applies to the
Sphinx theme. Since we are not bundling any documentation, I assume we can
ignore this one.


> I would also add all the other
> license files mentioned therein so we notice when they change or vanish.
>
> The Python-2.0 license in doc/scipy-sphinx-theme/LICENSE.txt is
> currently not available as an SPDX identifier, so I would leave this as
> "AND UNKNOWN", and wait until the respective SPDX issue is resolved...
> https://github.com/spdx/license-list-XML/issues/919


Same as above, this only applies to the Sphinx theme, so I assume we can
ignore this.


>
>
> numpy/core/src/multiarray/dragon4.c seems to be a license found nowhere
> else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE and add
> its verbatim license text with startline and endline parameters in
> PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
> texts mentioned in that variable and adds them to the license report,
> so it doesn't get lost too :)
>

OK.


>
> > +PYTHON3_NUMPY_LICENSE_FILES := \
> > + file://LICENSE.txt;md5=d26bde5432613cce2334b93985576231
>
>
> file://doc/sphinxext/LICENSE.txt;md5=dc37e8b18377b83250218fc557984e1a \
>
> file://doc/scipy-sphinx-theme/LICENSE.txt;md5=ea17c9a65c9ae0ccdf3b0a7fd1ee4616
> \
>
> file://tools/npy_tempita/license.txt;md5=c66b85ddcd09296abff87601467724fd \
>
> file://numpy/core/src/multiarray/dragon4.c;startline=2;endline=22;md5=19537439573c5696a922ed7957c5b37e
> \
>
> (numpy/linalg/lapack_lite/LICENSE.txt is currently missing in the
> tarball, see https://github.com/numpy/numpy/issues/13295)
>
> For reference, that last one can be generated with a
>
> sed -n 2,22p < inputfilename | md5sum
>
> Please check if all of this (especially the MD5s) make sense to you :)
>

Yes. Will double check and submit a new version of the patch.

Thank you!

Guillermo Rodriguez Garcia
guille.rodrig...@gmail.com
___
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH v3] python3-numpy: New package

2019-11-13 Thread Roland Hieber
On Tue, Nov 12, 2019 at 12:51:03PM +0100, Guillermo Rodríguez wrote:
> NumPy is the fundamental package for scientific computing with Python.
> 
> Signed-off-by: Guillermo Rodriguez 
> ---
> v2: Switch to PyPi URL; this removes the dependency on Cython.
> v3: Update LICENSE, add LICENSE_FILES
> 
>  rules/python3-numpy.in   | 10 
>  rules/python3-numpy.make | 54 
>  2 files changed, 64 insertions(+)
>  create mode 100644 rules/python3-numpy.in
>  create mode 100644 rules/python3-numpy.make
> 
> diff --git a/rules/python3-numpy.in b/rules/python3-numpy.in
> new file mode 100644
> index 0..1440e409a
> --- /dev/null
> +++ b/rules/python3-numpy.in
> @@ -0,0 +1,10 @@
> +## SECTION=python3
> +
> +config PYTHON3_NUMPY
> + tristate
> + select PYTHON3
> + select HOST_PYTHON3_SETUPTOOLS
> + prompt "python3-numpy"
> + help
> +   NumPy is the fundamental package for scientific computing with
> +   Python.
> diff --git a/rules/python3-numpy.make b/rules/python3-numpy.make
> new file mode 100644
> index 0..40cc351d3
> --- /dev/null
> +++ b/rules/python3-numpy.make
> @@ -0,0 +1,54 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2019 by Guillermo Rodriguez 
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_PYTHON3_NUMPY) += python3-numpy
> +
> +#
> +# Paths and names
> +#
> +PYTHON3_NUMPY_VERSION:= 1.16.1
> +PYTHON3_NUMPY_MD5:= dafda51934f645d66f98424521ae
> +PYTHON3_NUMPY:= numpy-$(PYTHON3_NUMPY_VERSION)
> +PYTHON3_NUMPY_SUFFIX := zip
> +PYTHON3_NUMPY_URL:= 
> https://pypi.python.org/packages/source/n/numpy/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> +PYTHON3_NUMPY_SOURCE := $(SRCDIR)/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
> +PYTHON3_NUMPY_DIR:= $(BUILDDIR)/$(PYTHON3_NUMPY)
> +PYTHON3_NUMPY_LICENSE:= BSD AND BSD-3-Clause AND MIT

AND Apache-2.0, according to LICENSE.txt. I would also add all the other
license files mentioned therein so we notice when they change or vanish.

The Python-2.0 license in doc/scipy-sphinx-theme/LICENSE.txt is
currently not available as an SPDX identifier, so I would leave this as
"AND UNKNOWN", and wait until the respective SPDX issue is resolved...
https://github.com/spdx/license-list-XML/issues/919

numpy/core/src/multiarray/dragon4.c seems to be a license found nowhere
else, so I would also add "AND UNKNOWN" to PYTHON3_NUMPY_LICENSE and add
its verbatim license text with startline and endline parameters in
PYTHON3_NUMPY_LICENSE_FILES. PTXdist extracts all those license
texts mentioned in that variable and adds them to the license report,
so it doesn't get lost too :)

> +PYTHON3_NUMPY_LICENSE_FILES := \
> + file://LICENSE.txt;md5=d26bde5432613cce2334b93985576231

file://doc/sphinxext/LICENSE.txt;md5=dc37e8b18377b83250218fc557984e1a \

file://doc/scipy-sphinx-theme/LICENSE.txt;md5=ea17c9a65c9ae0ccdf3b0a7fd1ee4616 \

file://tools/npy_tempita/license.txt;md5=c66b85ddcd09296abff87601467724fd \

file://numpy/core/src/multiarray/dragon4.c;startline=2;endline=22;md5=19537439573c5696a922ed7957c5b37e
 \

(numpy/linalg/lapack_lite/LICENSE.txt is currently missing in the
tarball, see https://github.com/numpy/numpy/issues/13295)

For reference, that last one can be generated with a 

sed -n 2,22p < inputfilename | md5sum

Please check if all of this (especially the MD5s) make sense to you :)

 - Roland


> +
> +# 
> 
> +# Prepare
> +# 
> 
> +
> +PYTHON3_NUMPY_CONF_TOOL  := python3
> +
> +# 
> 
> +# Target-Install
> +# 
> 
> +
> +$(STATEDIR)/python3-numpy.targetinstall:
> + @$(call targetinfo)
> +
> + @$(call install_init, python3-numpy)
> + @$(call install_fixup, python3-numpy, PRIORITY, optional)
> + @$(call install_fixup, python3-numpy, SECTION, base)
> + @$(call install_fixup, python3-numpy, AUTHOR, "Guillermo Rodriguez 
> ")
> + @$(call install_fixup, python3-numpy, DESCRIPTION, missing)
> +
> + @$(call install_glob, python3-numpy, 0, 0, -, \
> + /usr/lib/python$(PYTHON3_MAJORMINOR)/site-packages/numpy,,  
> *.py)
> +
> + @$(call install_finish, python3-numpy)
> +
> + @$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.21.0
> 
> 
> ___
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Roland Hieber, Pengutronix e.K.  | r.hie...@pengutronix.de |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany| Phone: +49-5121-206917-0

[ptxdist] [PATCH v3] python3-numpy: New package

2019-11-12 Thread Guillermo Rodríguez
NumPy is the fundamental package for scientific computing with Python.

Signed-off-by: Guillermo Rodriguez 
---
v2: Switch to PyPi URL; this removes the dependency on Cython.
v3: Update LICENSE, add LICENSE_FILES

 rules/python3-numpy.in   | 10 
 rules/python3-numpy.make | 54 
 2 files changed, 64 insertions(+)
 create mode 100644 rules/python3-numpy.in
 create mode 100644 rules/python3-numpy.make

diff --git a/rules/python3-numpy.in b/rules/python3-numpy.in
new file mode 100644
index 0..1440e409a
--- /dev/null
+++ b/rules/python3-numpy.in
@@ -0,0 +1,10 @@
+## SECTION=python3
+
+config PYTHON3_NUMPY
+   tristate
+   select PYTHON3
+   select HOST_PYTHON3_SETUPTOOLS
+   prompt "python3-numpy"
+   help
+ NumPy is the fundamental package for scientific computing with
+ Python.
diff --git a/rules/python3-numpy.make b/rules/python3-numpy.make
new file mode 100644
index 0..40cc351d3
--- /dev/null
+++ b/rules/python3-numpy.make
@@ -0,0 +1,54 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2019 by Guillermo Rodriguez 
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_PYTHON3_NUMPY) += python3-numpy
+
+#
+# Paths and names
+#
+PYTHON3_NUMPY_VERSION  := 1.16.1
+PYTHON3_NUMPY_MD5  := dafda51934f645d66f98424521ae
+PYTHON3_NUMPY  := numpy-$(PYTHON3_NUMPY_VERSION)
+PYTHON3_NUMPY_SUFFIX   := zip
+PYTHON3_NUMPY_URL  := 
https://pypi.python.org/packages/source/n/numpy/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
+PYTHON3_NUMPY_SOURCE   := $(SRCDIR)/$(PYTHON3_NUMPY).$(PYTHON3_NUMPY_SUFFIX)
+PYTHON3_NUMPY_DIR  := $(BUILDDIR)/$(PYTHON3_NUMPY)
+PYTHON3_NUMPY_LICENSE  := BSD AND BSD-3-Clause AND MIT
+PYTHON3_NUMPY_LICENSE_FILES := \
+   file://LICENSE.txt;md5=d26bde5432613cce2334b93985576231
+
+# 
+# Prepare
+# 
+
+PYTHON3_NUMPY_CONF_TOOL:= python3
+
+# 
+# Target-Install
+# 
+
+$(STATEDIR)/python3-numpy.targetinstall:
+   @$(call targetinfo)
+
+   @$(call install_init, python3-numpy)
+   @$(call install_fixup, python3-numpy, PRIORITY, optional)
+   @$(call install_fixup, python3-numpy, SECTION, base)
+   @$(call install_fixup, python3-numpy, AUTHOR, "Guillermo Rodriguez 
")
+   @$(call install_fixup, python3-numpy, DESCRIPTION, missing)
+
+   @$(call install_glob, python3-numpy, 0, 0, -, \
+   /usr/lib/python$(PYTHON3_MAJORMINOR)/site-packages/numpy,,  
*.py)
+
+   @$(call install_finish, python3-numpy)
+
+   @$(call touch)
+
+# vim: syntax=make
-- 
2.21.0


___
ptxdist mailing list
ptxdist@pengutronix.de