All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1 v3] pyqtgraph: new package
@ 2015-10-13 16:54 Guillaume William Bres
  2016-07-03  9:30 ` Maxime Hadjinlian
  0 siblings, 1 reply; 3+ messages in thread
From: Guillaume William Bres @ 2015-10-13 16:54 UTC (permalink / raw)
  To: buildroot


Signed-off-by: Guillaume William Bres <guillaume.bressaix@gmail.com>
---
 PyQtGraph is an easy to use library using PyQt bindings, to develop
 scientific applications.

 The library requires python-numpy to run, which depends on
 certain types of toolchain. I know Gwen has submitted a new variable 
 BR2_PACKAGE_PYTHON_NUMPY_ARCH_DEPENDS so I added that one as a
 dependency: please correct me on that.

 All suggestions are appreciated.
 The package builds easily because it is only made of python scripts.

 Builds and runs successfully on ARM Cortex A9.

 package/Config.in                              |    1 +
 package/python-pyqtgraph/Config.in             |   14 ++++++++++++++
 package/python-pyqtgraph/python-pyqtgraph.hash |    5 +++++
 package/python-pyqtgraph/python-pyqtgraph.mk   |   15 +++++++++++++++
 4 files changed, 35 insertions(+)
 create mode 100644 package/python-pyqtgraph/Config.in
 create mode 100644 package/python-pyqtgraph/python-pyqtgraph.hash
 create mode 100644 package/python-pyqtgraph/python-pyqtgraph.mk

diff --git a/package/Config.in b/package/Config.in
index f686f13..7ef27b4 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -646,6 +646,7 @@ menu "External python modules"
 	source "package/python-pyparsing/Config.in"
 	source "package/python-pypcap/Config.in"
 	source "package/python-pyqt/Config.in"
+	source "package/python-pyqtgraph/Config.in"
 	source "package/python-pyratemp/Config.in"
 	source "package/python-pyro/Config.in"
 	source "package/python-pyroute2/Config.in"
diff --git a/package/python-pyqtgraph/Config.in b/package/python-pyqtgraph/Config.in
new file mode 100644
index 0000000..5e1a1e6
--- /dev/null
+++ b/package/python-pyqtgraph/Config.in
@@ -0,0 +1,14 @@
+config BR2_PACKAGE_PYTHON_PYQTGRAPH
+	bool "python-pyqtgraph"
+	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_DEPENDS
+	select BR2_PACKAGE_PYTHON_PYQT
+	select BR2_PACKAGE_PYTHON_NUMPY # runtime dependency
+	select BR2_PACKAGE_PYTHON_PYEXPAT if BR2_PACKAGE_PYTHON # runtime dependency
+	select BR2_PACKAGE_PYTHON3_PYEXPAT if BR2_PACKAGE_PYTHON3 #runtime dependency
+	select BR2_PACKAGE_QT_SVG # runtime dependency
+	help
+	  Pyqtgraph is a pure Python graphics and GUI library built on PyQt4, 
+	  PySide and Numpy. 
+	  
+	  Official website: 
+	  http://www.pyqtgraph.org/
diff --git a/package/python-pyqtgraph/python-pyqtgraph.hash b/package/python-pyqtgraph/python-pyqtgraph.hash
new file mode 100644
index 0000000..c20eea2
--- /dev/null
+++ b/package/python-pyqtgraph/python-pyqtgraph.hash
@@ -0,0 +1,5 @@
+# From http://www.pyqtgraph.org/downloads
+md5 6e2efa185b6b9227dfe16fefd921a8ec  pyqtgraph-0.9.10.tar.gz
+
+# Locally calculated:
+sha256 4c0589774e3c8b0c374931397cf6356b9cc99a790215d1917bb7f015c6f0729a  pyqtgraph-0.9.10.tar.gz
diff --git a/package/python-pyqtgraph/python-pyqtgraph.mk b/package/python-pyqtgraph/python-pyqtgraph.mk
new file mode 100644
index 0000000..575ea71
--- /dev/null
+++ b/package/python-pyqtgraph/python-pyqtgraph.mk
@@ -0,0 +1,15 @@
+################################################################################
+#
+# python-pyqtgraph
+#
+################################################################################
+
+PYTHON_PYQTGRAPH_VERSION = 0.9.10
+PYTHON_PYQTGRAPH_SOURCE = pyqtgraph-$(PYTHON_PYQTGRAPH_VERSION).tar.gz
+PYTHON_PYQTGRAPH_SITE = http://www.pyqtgraph.org/downloads
+PYTHON_PYQTGRAPH_LICENSE = MIT
+PYTHON_PYQTGRAPH_LICENSE_FILES = LICENSE 
+PYTHON_PYQTGRAPH_SETUP_TYPE = distutils
+PYTHON_PYQTGRAPH_DEPENDENCIES = python-pyqt
+
+$(eval $(python-package))
-- 
1.7.10.4

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

* [Buildroot] [PATCH 1/1 v3] pyqtgraph: new package
  2015-10-13 16:54 [Buildroot] [PATCH 1/1 v3] pyqtgraph: new package Guillaume William Bres
@ 2016-07-03  9:30 ` Maxime Hadjinlian
  2016-07-03 19:30   ` Arnout Vandecappelle
  0 siblings, 1 reply; 3+ messages in thread
From: Maxime Hadjinlian @ 2016-07-03  9:30 UTC (permalink / raw)
  To: buildroot

Hi Guillaume, all

Thank you for your patches, I have made a few comments inline, your
patch has been marked as Changes Requested in our patchwork, if you
can address theses and resend a new version of your patch.

On Tue, Oct 13, 2015 at 6:54 PM, Guillaume William Bres
<guillaume.bressaix@gmail.com> wrote:
>
> Signed-off-by: Guillaume William Bres <guillaume.bressaix@gmail.com>
> ---
>  PyQtGraph is an easy to use library using PyQt bindings, to develop
>  scientific applications.
>
>  The library requires python-numpy to run, which depends on
>  certain types of toolchain. I know Gwen has submitted a new variable
>  BR2_PACKAGE_PYTHON_NUMPY_ARCH_DEPENDS so I added that one as a
>  dependency: please correct me on that.
>
>  All suggestions are appreciated.
>  The package builds easily because it is only made of python scripts.
>
>  Builds and runs successfully on ARM Cortex A9.
>
>  package/Config.in                              |    1 +
>  package/python-pyqtgraph/Config.in             |   14 ++++++++++++++
>  package/python-pyqtgraph/python-pyqtgraph.hash |    5 +++++
>  package/python-pyqtgraph/python-pyqtgraph.mk   |   15 +++++++++++++++
>  4 files changed, 35 insertions(+)
>  create mode 100644 package/python-pyqtgraph/Config.in
>  create mode 100644 package/python-pyqtgraph/python-pyqtgraph.hash
>  create mode 100644 package/python-pyqtgraph/python-pyqtgraph.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index f686f13..7ef27b4 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -646,6 +646,7 @@ menu "External python modules"
>         source "package/python-pyparsing/Config.in"
>         source "package/python-pypcap/Config.in"
>         source "package/python-pyqt/Config.in"
> +       source "package/python-pyqtgraph/Config.in"
>         source "package/python-pyratemp/Config.in"
>         source "package/python-pyro/Config.in"
>         source "package/python-pyroute2/Config.in"
> diff --git a/package/python-pyqtgraph/Config.in b/package/python-pyqtgraph/Config.in
Your patch doesn't apply anymore, if you could rebase on a more recent
master and resend with the other comments it would be great.

> new file mode 100644
> index 0000000..5e1a1e6
> --- /dev/null
> +++ b/package/python-pyqtgraph/Config.in
> @@ -0,0 +1,14 @@
> +config BR2_PACKAGE_PYTHON_PYQTGRAPH
> +       bool "python-pyqtgraph"
> +       depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_DEPENDS
This symbol doesn't exists (anymore maybe), it's
BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS.
> +       select BR2_PACKAGE_PYTHON_PYQT
> +       select BR2_PACKAGE_PYTHON_NUMPY # runtime dependency
> +       select BR2_PACKAGE_PYTHON_PYEXPAT if BR2_PACKAGE_PYTHON # runtime dependency
> +       select BR2_PACKAGE_PYTHON3_PYEXPAT if BR2_PACKAGE_PYTHON3 #runtime dependency
> +       select BR2_PACKAGE_QT_SVG # runtime dependency
It doesn't make sense to select BR2_PACKAGE_QT_SVG without selecting
BR2_PACKAGE_QT. And it's fine to select it, because it's not a hidden
dependency, it's even in the name of the package.
> +       help
> +         Pyqtgraph is a pure Python graphics and GUI library built on PyQt4,
> +         PySide and Numpy.
> +
> +         Official website:
> +         http://www.pyqtgraph.org/
> diff --git a/package/python-pyqtgraph/python-pyqtgraph.hash b/package/python-pyqtgraph/python-pyqtgraph.hash
> new file mode 100644
> index 0000000..c20eea2
> --- /dev/null
> +++ b/package/python-pyqtgraph/python-pyqtgraph.hash
> @@ -0,0 +1,5 @@
> +# From http://www.pyqtgraph.org/downloads
> +md5 6e2efa185b6b9227dfe16fefd921a8ec  pyqtgraph-0.9.10.tar.gz
> +
> +# Locally calculated:
> +sha256 4c0589774e3c8b0c374931397cf6356b9cc99a790215d1917bb7f015c6f0729a  pyqtgraph-0.9.10.tar.gz
> diff --git a/package/python-pyqtgraph/python-pyqtgraph.mk b/package/python-pyqtgraph/python-pyqtgraph.mk
> new file mode 100644
> index 0000000..575ea71
> --- /dev/null
> +++ b/package/python-pyqtgraph/python-pyqtgraph.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# python-pyqtgraph
> +#
> +################################################################################
> +
> +PYTHON_PYQTGRAPH_VERSION = 0.9.10
> +PYTHON_PYQTGRAPH_SOURCE = pyqtgraph-$(PYTHON_PYQTGRAPH_VERSION).tar.gz
> +PYTHON_PYQTGRAPH_SITE = http://www.pyqtgraph.org/downloads
> +PYTHON_PYQTGRAPH_LICENSE = MIT
> +PYTHON_PYQTGRAPH_LICENSE_FILES = LICENSE
You have a trailing space in your patch
> +PYTHON_PYQTGRAPH_SETUP_TYPE = distutils
> +PYTHON_PYQTGRAPH_DEPENDENCIES = python-pyqt
> +
> +$(eval $(python-package))
> --
> 1.7.10.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

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

* [Buildroot] [PATCH 1/1 v3] pyqtgraph: new package
  2016-07-03  9:30 ` Maxime Hadjinlian
@ 2016-07-03 19:30   ` Arnout Vandecappelle
  0 siblings, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2016-07-03 19:30 UTC (permalink / raw)
  To: buildroot

On 03-07-16 11:30, Maxime Hadjinlian wrote:
> Hi Guillaume, all
> 
> Thank you for your patches, I have made a few comments inline, your
> patch has been marked as Changes Requested in our patchwork, if you
> can address theses and resend a new version of your patch.
> 
> On Tue, Oct 13, 2015 at 6:54 PM, Guillaume William Bres
> <guillaume.bressaix@gmail.com> wrote:
[snip]
>> +       select BR2_PACKAGE_PYTHON_PYQT
>> +       select BR2_PACKAGE_PYTHON_NUMPY # runtime dependency
>> +       select BR2_PACKAGE_PYTHON_PYEXPAT if BR2_PACKAGE_PYTHON # runtime dependency
>> +       select BR2_PACKAGE_PYTHON3_PYEXPAT if BR2_PACKAGE_PYTHON3 #runtime dependency
>> +       select BR2_PACKAGE_QT_SVG # runtime dependency
> It doesn't make sense to select BR2_PACKAGE_QT_SVG without selecting
> BR2_PACKAGE_QT. And it's fine to select it, because it's not a hidden
> dependency, it's even in the name of the package.

 BR2_PACKAGE_PYTHON_PYQT selects BR2_PACKAGE_QT already, and pyqtgraph doesn't
use Qt directly, so the way it's done now is OK IMO.


 Regards,
 Arnout

[snip]

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

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

end of thread, other threads:[~2016-07-03 19:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-13 16:54 [Buildroot] [PATCH 1/1 v3] pyqtgraph: new package Guillaume William Bres
2016-07-03  9:30 ` Maxime Hadjinlian
2016-07-03 19:30   ` Arnout Vandecappelle

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.