* [Buildroot] [PATCH 1/1] pyzmq: new package
@ 2013-08-30 21:58 Michael Rommel
2013-08-31 8:55 ` Thomas De Schampheleire
2013-09-02 6:05 ` Arnout Vandecappelle
0 siblings, 2 replies; 5+ messages in thread
From: Michael Rommel @ 2013-08-30 21:58 UTC (permalink / raw
To: buildroot
Hi,
as promised, here is the separate patch for pyzmq.
Michael.
PS: I noticed, that I missed in the previous e-mail to change the Config.in and remove the pyzmq line from it. Do I need to re-submit the patch for knock?
The python language bindings for zeromq.
Please note that the reason for the hardcoded version number in the
python-pyzmq package is due to the following dilemma:
- the setup.py script tries to compile a test C program and runs it, to
retrieve a version string
- if the cross compiler links it together, the result cannot be run on
the host, due to different architectures
- if the host compiler would compile/link it, it would not link with the
library version inside buildroot but with the library from the host
Signed-off-by: Michael Rommel <rommel@layer-7.net>
---
package/Config.in | 1 +
package/python-pyzmq/Config.in | 13 ++++++++++
package/python-pyzmq/pyzmq.mk | 50 ++++++++++++++++++++++++++++++++++++++
package/python-pyzmq/pyzmq.patch | 28 +++++++++++++++++++++
4 files changed, 92 insertions(+)
create mode 100644 package/python-pyzmq/Config.in
create mode 100644 package/python-pyzmq/pyzmq.mk
create mode 100644 package/python-pyzmq/pyzmq.patch
diff --git a/package/Config.in b/package/Config.in
index 97cd7da..0289fd1 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -363,6 +363,7 @@ source "package/python-protobuf/Config.in"
source "package/python-pygame/Config.in"
source "package/python-pyparsing/Config.in"
source "package/python-pyro/Config.in"
+source "package/python-pyzmq/Config.in"
source "package/python-serial/Config.in"
source "package/python-setuptools/Config.in"
source "package/python-thrift/Config.in"
diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in
new file mode 100644
index 0000000..d9791c1
--- /dev/null
+++ b/package/python-pyzmq/Config.in
@@ -0,0 +1,13 @@
+config BR2_PACKAGE_PYZMQ
+ bool "pyzmq"
+ select BR2_PACKAGE_ZEROMQ
+ select BR2_PACKAGE_PYTHON
+ depends on BR2_USE_WCHAR
+ depends on BR2_USE_MMU
+ help
+ This package contains the python language binding for zeromq.
+ Due to issues with cross-compiling this package is hardcoded
+ to ZeroMQ version 3.2.3
+
+ See also: http://zeromq.org/bindings:python
+
diff --git a/package/python-pyzmq/pyzmq.mk b/package/python-pyzmq/pyzmq.mk
new file mode 100644
index 0000000..a7efcad
--- /dev/null
+++ b/package/python-pyzmq/pyzmq.mk
@@ -0,0 +1,50 @@
+##########################################
+#
+# pyzmq
+#
+##########################################
+
+PYZMQ_VERSION = 13.0.2
+PYZMQ_ZMQ_VERSION = 3.2.3
+PYZMQ_SOURCE = pyzmq-$(PYZMQ_VERSION).tar.gz
+PYZMQ_SITE = https://pypi.python.org/packages/source/p/pyzmq/
+PYZMQ_LICENSE = LGPLv3 BSD-3c
+PYZMQ_LICENSE_FILES = COPYING.LESSER COPYING.BSD
+PYZMQ_INSTALL_TARGET = YES
+
+define PYZMQ_POST_PATCH_FIXUP
+ (cd $(@D); \
+ cat buildutils/detect.py | \
+ sed -e "s/##PYZMQ_ZMQ_VERSION##/$(PYZMQ_ZMQ_VERSION)/" \
+ >buildutils/detect.py.new; \
+ cp buildutils/detect.py.new buildutils/detect.py)
+endef
+
+PYZMQ_POST_PATCH_HOOKS += PYZMQ_POST_PATCH_FIXUP
+
+define PYZMQ_CONFIGURE_CMDS
+ (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
+ LDSHARED="$(TARGET_CROSS)gcc -shared" \
+ CROSS_COMPILING=yes \
+ _python_sysroot=$(STAGING_DIR) \
+ _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
+ _python_prefix=/usr \
+ _python_exec_prefix=/usr \
+ $(HOST_DIR)/usr/bin/python setup.py configure \
+ --zmq=$(@D)/../../staging/usr)
+endef
+
+define PYZMQ_INSTALL_TARGET_CMDS
+ (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
+ LDSHARED="$(TARGET_CROSS)gcc -shared" \
+ CROSS_COMPILING=yes \
+ _python_sysroot=$(STAGING_DIR) \
+ _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
+ _python_prefix=/usr \
+ _python_exec_prefix=/usr \
+ $(HOST_DIR)/usr/bin/python setup.py install \
+ --prefix=$(TARGET_DIR)/usr)
+endef
+
+$(eval $(generic-package))
+
diff --git a/package/python-pyzmq/pyzmq.patch b/package/python-pyzmq/pyzmq.patch
new file mode 100644
index 0000000..c660457
--- /dev/null
+++ b/package/python-pyzmq/pyzmq.patch
@@ -0,0 +1,28 @@
+--- pyzmq-13.0.2/buildutils/detect.py.orig 2013-08-10 00:49:28.242557978 +0200
++++ pyzmq-13.0.2/buildutils/detect.py 2013-08-10 00:44:35.197572704 +0200
+@@ -118,15 +118,17 @@ def detect_zmq(basedir, compiler=None, *
+
+ efile = test_compilation(cfile, compiler=compiler, **compiler_attrs)
+
+- result = Popen(efile, stdout=PIPE, stderr=PIPE)
+- so, se = result.communicate()
++ # result = Popen(efile, stdout=PIPE, stderr=PIPE)
++ # so, se = result.communicate()
+ # for py3k:
+- so = so.decode()
+- se = se.decode()
+- if result.returncode:
+- msg = "Error running version detection script:\n%s\n%s" % (so,se)
+- logging.error(msg)
+- raise IOError(msg)
++ #so = so.decode()
++ #se = se.decode()
++ #if result.returncode:
++ # msg = "Error running version detection script:\n%s\n%s" % (so,se)
++ # logging.error(msg)
++ # raise IOError(msg)
++
++ so = "vers: ##PYZMQ_ZMQ_VERSION##"
+
+ handlers = {'vers': lambda val: tuple(int(v) for v in val.split('.'))}
+
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] pyzmq: new package
2013-08-30 21:58 [Buildroot] [PATCH 1/1] pyzmq: new package Michael Rommel
@ 2013-08-31 8:55 ` Thomas De Schampheleire
2013-09-01 7:11 ` Thomas Petazzoni
2013-09-02 6:05 ` Arnout Vandecappelle
1 sibling, 1 reply; 5+ messages in thread
From: Thomas De Schampheleire @ 2013-08-31 8:55 UTC (permalink / raw
To: buildroot
On Fri, Aug 30, 2013 at 11:58 PM, Michael Rommel <rommel@layer-7.net> wrote:
> Hi,
>
> as promised, here is the separate patch for pyzmq.
>
> Michael.
>
> PS: I noticed, that I missed in the previous e-mail to change the Config.in and remove the pyzmq line from it. Do I need to re-submit the patch for knock?
Yes, please send an updated patch taking into account the comments you received.
As I said in the knock package, this explanation should be below '---'.
>
>
>
>
> The python language bindings for zeromq.
>
> Please note that the reason for the hardcoded version number in the
> python-pyzmq package is due to the following dilemma:
> - the setup.py script tries to compile a test C program and runs it, to
> retrieve a version string
> - if the cross compiler links it together, the result cannot be run on
> the host, due to different architectures
> - if the host compiler would compile/link it, it would not link with the
> library version inside buildroot but with the library from the host
>
> Signed-off-by: Michael Rommel <rommel@layer-7.net>
> ---
> package/Config.in | 1 +
> package/python-pyzmq/Config.in | 13 ++++++++++
> package/python-pyzmq/pyzmq.mk | 50 ++++++++++++++++++++++++++++++++++++++
> package/python-pyzmq/pyzmq.patch | 28 +++++++++++++++++++++
> 4 files changed, 92 insertions(+)
> create mode 100644 package/python-pyzmq/Config.in
> create mode 100644 package/python-pyzmq/pyzmq.mk
> create mode 100644 package/python-pyzmq/pyzmq.patch
Patches should be named according to this format:
<packagename>-<number>-<description>.patch, see
http://buildroot.uclibc.org/downloads/manual/manual.html#patch-policy
>
> diff --git a/package/Config.in b/package/Config.in
> index 97cd7da..0289fd1 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -363,6 +363,7 @@ source "package/python-protobuf/Config.in"
> source "package/python-pygame/Config.in"
> source "package/python-pyparsing/Config.in"
> source "package/python-pyro/Config.in"
> +source "package/python-pyzmq/Config.in"
> source "package/python-serial/Config.in"
> source "package/python-setuptools/Config.in"
> source "package/python-thrift/Config.in"
> diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in
> new file mode 100644
> index 0000000..d9791c1
> --- /dev/null
> +++ b/package/python-pyzmq/Config.in
> @@ -0,0 +1,13 @@
> +config BR2_PACKAGE_PYZMQ
> + bool "pyzmq"
> + select BR2_PACKAGE_ZEROMQ
> + select BR2_PACKAGE_PYTHON
This line is indented with spaces instead of one tab.
You will need to add some other lines here, because both zeromq as
python depend on other packages, and as explained here:
http://buildroot.uclibc.org/downloads/manual/manual.html#depends-on-vs-select,
you will need to replicate these dependencies in this package.
> + depends on BR2_USE_WCHAR
> + depends on BR2_USE_MMU
For this type of dependencies, we generally add a comment in case the
dependencies are not met, warning the user about this. Have a look at
some other packages to get an idea.
> + help
> + This package contains the python language binding for zeromq.
> + Due to issues with cross-compiling this package is hardcoded
> + to ZeroMQ version 3.2.3
> +
> + See also: http://zeromq.org/bindings:python
> +
> diff --git a/package/python-pyzmq/pyzmq.mk b/package/python-pyzmq/pyzmq.mk
> new file mode 100644
> index 0000000..a7efcad
> --- /dev/null
> +++ b/package/python-pyzmq/pyzmq.mk
> @@ -0,0 +1,50 @@
> +##########################################
> +#
> +# pyzmq
> +#
> +##########################################
> +
> +PYZMQ_VERSION = 13.0.2
> +PYZMQ_ZMQ_VERSION = 3.2.3
I haven't thought about better ways to do this, so I'll take it as a
given that you need to have a fixed version.
But have you considered using $(ZEROMQ_VERSION) instead of the
hardcoded 3.2.3. This way, pyzmq will not have to be updated when the
zeromq version is bumped.
> +PYZMQ_SOURCE = pyzmq-$(PYZMQ_VERSION).tar.gz
This format is the default value for FOO_SOURCE, so this line can be ommitted.
> +PYZMQ_SITE = https://pypi.python.org/packages/source/p/pyzmq/
> +PYZMQ_LICENSE = LGPLv3 BSD-3c
> +PYZMQ_LICENSE_FILES = COPYING.LESSER COPYING.BSD
> +PYZMQ_INSTALL_TARGET = YES
FOO_INSTALL_TARGET = YES is the default, so line can be ommitted.
> +
> +define PYZMQ_POST_PATCH_FIXUP
> + (cd $(@D); \
> + cat buildutils/detect.py | \
> + sed -e "s/##PYZMQ_ZMQ_VERSION##/$(PYZMQ_ZMQ_VERSION)/" \
> + >buildutils/detect.py.new; \
> + cp buildutils/detect.py.new buildutils/detect.py)
> +endef
> +
> +PYZMQ_POST_PATCH_HOOKS += PYZMQ_POST_PATCH_FIXUP
> +
> +define PYZMQ_CONFIGURE_CMDS
> + (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
> + LDSHARED="$(TARGET_CROSS)gcc -shared" \
> + CROSS_COMPILING=yes \
> + _python_sysroot=$(STAGING_DIR) \
> + _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
> + _python_prefix=/usr \
> + _python_exec_prefix=/usr \
> + $(HOST_DIR)/usr/bin/python setup.py configure \
> + --zmq=$(@D)/../../staging/usr)
> +endef
Python and zeromq will already have to be installed before pyzmq is
built, so you will have to add a line like:
PYZMQ_DEPENDENCIES = python zeromq
> +
> +define PYZMQ_INSTALL_TARGET_CMDS
> + (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
> + LDSHARED="$(TARGET_CROSS)gcc -shared" \
> + CROSS_COMPILING=yes \
> + _python_sysroot=$(STAGING_DIR) \
> + _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
> + _python_prefix=/usr \
> + _python_exec_prefix=/usr \
> + $(HOST_DIR)/usr/bin/python setup.py install \
> + --prefix=$(TARGET_DIR)/usr)
> +endef
> +
> +$(eval $(generic-package))
> +
This empty line can be removed.
> diff --git a/package/python-pyzmq/pyzmq.patch b/package/python-pyzmq/pyzmq.patch
> new file mode 100644
> index 0000000..c660457
> --- /dev/null
> +++ b/package/python-pyzmq/pyzmq.patch
Also check the manual about patches here: the patch should have a
different name, and should contain an explanation and signed-off-by
line.
> @@ -0,0 +1,28 @@
> +--- pyzmq-13.0.2/buildutils/detect.py.orig 2013-08-10 00:49:28.242557978 +0200
> ++++ pyzmq-13.0.2/buildutils/detect.py 2013-08-10 00:44:35.197572704 +0200
> +@@ -118,15 +118,17 @@ def detect_zmq(basedir, compiler=None, *
> +
> + efile = test_compilation(cfile, compiler=compiler, **compiler_attrs)
> +
> +- result = Popen(efile, stdout=PIPE, stderr=PIPE)
> +- so, se = result.communicate()
> ++ # result = Popen(efile, stdout=PIPE, stderr=PIPE)
> ++ # so, se = result.communicate()
> + # for py3k:
> +- so = so.decode()
> +- se = se.decode()
> +- if result.returncode:
> +- msg = "Error running version detection script:\n%s\n%s" % (so,se)
> +- logging.error(msg)
> +- raise IOError(msg)
> ++ #so = so.decode()
> ++ #se = se.decode()
> ++ #if result.returncode:
> ++ # msg = "Error running version detection script:\n%s\n%s" % (so,se)
> ++ # logging.error(msg)
> ++ # raise IOError(msg)
> ++
> ++ so = "vers: ##PYZMQ_ZMQ_VERSION##"
> +
> + handlers = {'vers': lambda val: tuple(int(v) for v in val.split('.'))}
> +
Thanks for both of your contributions!
Best regards,
Thomas
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] pyzmq: new package
2013-08-31 8:55 ` Thomas De Schampheleire
@ 2013-09-01 7:11 ` Thomas Petazzoni
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2013-09-01 7:11 UTC (permalink / raw
To: buildroot
Michael, Thomas,
A few comments below.
On Sat, 31 Aug 2013 10:55:23 +0200, Thomas De Schampheleire wrote:
> > Please note that the reason for the hardcoded version number in the
> > python-pyzmq package is due to the following dilemma:
> > - the setup.py script tries to compile a test C program and runs it, to
> > retrieve a version string
> > - if the cross compiler links it together, the result cannot be run on
> > the host, due to different architectures
> > - if the host compiler would compile/link it, it would not link with the
> > library version inside buildroot but with the library from the host
I believe such explanations fall inside the .mk file itself, so that
people looking at your code later can understand this 'bizarre' aspect
of your package.
> > + depends on BR2_USE_WCHAR
> > + depends on BR2_USE_MMU
>
> For this type of dependencies, we generally add a comment in case the
> dependencies are not met, warning the user about this. Have a look at
> some other packages to get an idea.
That's true for the WCHAR dependency, but not for the MMU dependency.
So the comment should be:
comment "pyzmq requires wchar support in toolchain"
depends on !BR2_USE_WCHAR
> > + help
> > + This package contains the python language binding for zeromq.
> > + Due to issues with cross-compiling this package is hardcoded
> > + to ZeroMQ version 3.2.3
I wouldn't mention the version of ZeroMQ here, as it will most likely
not be updated when ZeroMQ is upgraded in Buildroot.
> > + See also: http://zeromq.org/bindings:python
No need for 'See also:', just put the upstream URL.
> > diff --git a/package/python-pyzmq/pyzmq.mk b/package/python-pyzmq/pyzmq.mk
> > new file mode 100644
> > index 0000000..a7efcad
> > --- /dev/null
> > +++ b/package/python-pyzmq/pyzmq.mk
> > @@ -0,0 +1,50 @@
> > +##########################################
> > +#
> > +# pyzmq
> > +#
> > +##########################################
80 '#' should be used.
> > +PYZMQ_VERSION = 13.0.2
> > +PYZMQ_ZMQ_VERSION = 3.2.3
>
> I haven't thought about better ways to do this, so I'll take it as a
> given that you need to have a fixed version.
> But have you considered using $(ZEROMQ_VERSION) instead of the
> hardcoded 3.2.3. This way, pyzmq will not have to be updated when the
> zeromq version is bumped.
I agree.
>
> > +PYZMQ_SOURCE = pyzmq-$(PYZMQ_VERSION).tar.gz
>
> This format is the default value for FOO_SOURCE, so this line can be ommitted.
>
> > +PYZMQ_SITE = https://pypi.python.org/packages/source/p/pyzmq/
> > +PYZMQ_LICENSE = LGPLv3 BSD-3c
> > +PYZMQ_LICENSE_FILES = COPYING.LESSER COPYING.BSD
Check whether it's really LGPLv3 or LGPLv3+ (i.e, whether it's LGPL
version 3 only, or LGPL version 3 or later).
> > +PYZMQ_INSTALL_TARGET = YES
>
> FOO_INSTALL_TARGET = YES is the default, so line can be ommitted.
>
>
> > +
> > +define PYZMQ_POST_PATCH_FIXUP
> > + (cd $(@D); \
> > + cat buildutils/detect.py | \
> > + sed -e "s/##PYZMQ_ZMQ_VERSION##/$(PYZMQ_ZMQ_VERSION)/" \
> > + >buildutils/detect.py.new; \
> > + cp buildutils/detect.py.new buildutils/detect.py)
I think you should use $(SED) to do an in-place replacement. See for
example what package/dmalloc/dmalloc.mk is doing. Something like:
$(SED) 's/##PYZMQ_ZMQ_VERSION##/$(PYZMQ_ZMQ_VERSION)/' $(@D)/buildutils/detect.py
> > +endef
> > +
> > +PYZMQ_POST_PATCH_HOOKS += PYZMQ_POST_PATCH_FIXUP
> > +
> > +define PYZMQ_CONFIGURE_CMDS
> > + (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
> > + LDSHARED="$(TARGET_CROSS)gcc -shared" \
LDSHARED="$(TARGET_CC) -shared" would be preferable I believe.
> > + CROSS_COMPILING=yes \
> > + _python_sysroot=$(STAGING_DIR) \
> > + _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
> > + _python_prefix=/usr \
> > + _python_exec_prefix=/usr \
> > + $(HOST_DIR)/usr/bin/python setup.py configure \
> > + --zmq=$(@D)/../../staging/usr)
No, this should be --zmq=$(STAGING_DIR)/usr
> > +endef
>
> Python and zeromq will already have to be installed before pyzmq is
> built, so you will have to add a line like:
> PYZMQ_DEPENDENCIES = python zeromq
>
> > +
> > +define PYZMQ_INSTALL_TARGET_CMDS
> > + (cd $(@D); CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \
> > + LDSHARED="$(TARGET_CROSS)gcc -shared" \
> > + CROSS_COMPILING=yes \
> > + _python_sysroot=$(STAGING_DIR) \
> > + _python_srcdir=$(BUILD_DIR)/python$(PYTHON_VERSION) \
> > + _python_prefix=/usr \
> > + _python_exec_prefix=/usr \
> > + $(HOST_DIR)/usr/bin/python setup.py install \
> > + --prefix=$(TARGET_DIR)/usr)
This prefix looks wrong. The prefix should normally always be '/usr'.
You also have lots of duplicated variables between the CONFIGURE_CMDS
and INSTALL_TARGET_CMDS, it'd be great to factorize them.
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] pyzmq: new package
2013-08-30 21:58 [Buildroot] [PATCH 1/1] pyzmq: new package Michael Rommel
2013-08-31 8:55 ` Thomas De Schampheleire
@ 2013-09-02 6:05 ` Arnout Vandecappelle
2013-09-02 7:34 ` Michael Rommel
1 sibling, 1 reply; 5+ messages in thread
From: Arnout Vandecappelle @ 2013-09-02 6:05 UTC (permalink / raw
To: buildroot
On 08/30/13 23:58, Michael Rommel wrote:
[snip]
> diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in
> new file mode 100644
> index 0000000..d9791c1
> --- /dev/null
> +++ b/package/python-pyzmq/Config.in
> @@ -0,0 +1,13 @@
> +config BR2_PACKAGE_PYZMQ
> + bool "pyzmq"
> + select BR2_PACKAGE_ZEROMQ
> + select BR2_PACKAGE_PYTHON
For python packages, we use a 'depends on' here rather than a select.
Regards,
Arnout
> + depends on BR2_USE_WCHAR
> + depends on BR2_USE_MMU
[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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] pyzmq: new package
2013-09-02 6:05 ` Arnout Vandecappelle
@ 2013-09-02 7:34 ` Michael Rommel
0 siblings, 0 replies; 5+ messages in thread
From: Michael Rommel @ 2013-09-02 7:34 UTC (permalink / raw
To: buildroot
Hi Arnout,
I'll test that again today in the evening. IIRC "make savedefconfig"
complained
about a cyclic dependency, but I need to check that again. Thanks for
pointing
that out!
Michael.
On Mon, 02 Sep 2013 08:05:47 +0200, Arnout Vandecappelle
<arnout@mind.be> wrote:
> On 08/30/13 23:58, Michael Rommel wrote:
> [snip]
>> diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in
>> new file mode 100644
>> index 0000000..d9791c1
>> --- /dev/null
>> +++ b/package/python-pyzmq/Config.in
>> @@ -0,0 +1,13 @@
>> +config BR2_PACKAGE_PYZMQ
>> + bool "pyzmq"
>> + select BR2_PACKAGE_ZEROMQ
>> + select BR2_PACKAGE_PYTHON
>
> For python packages, we use a 'depends on' here rather than a select.
>
>
> Regards,
> Arnout
>
>> + depends on BR2_USE_WCHAR
>> + depends on BR2_USE_MMU
> [snip]
--
Michael Rommel, Erlangen, Germany
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-09-02 7:34 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-30 21:58 [Buildroot] [PATCH 1/1] pyzmq: new package Michael Rommel
2013-08-31 8:55 ` Thomas De Schampheleire
2013-09-01 7:11 ` Thomas Petazzoni
2013-09-02 6:05 ` Arnout Vandecappelle
2013-09-02 7:34 ` Michael Rommel
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.