* [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers
@ 2012-09-17 10:51 Thiago A. Correa
2012-09-20 20:34 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Thiago A. Correa @ 2012-09-17 10:51 UTC (permalink / raw)
To: buildroot
Signed-off-by: Thiago A. Correa <thiago.correa@gmail.com>
---
package/Config.in | 1 +
package/python-protobuf/Config.in | 11 +++++++++++
package/python-protobuf/python-protobuf.mk | 28 ++++++++++++++++++++++++++++
3 files changed, 40 insertions(+), 0 deletions(-)
create mode 100644 package/python-protobuf/Config.in
create mode 100644 package/python-protobuf/python-protobuf.mk
diff --git a/package/Config.in b/package/Config.in
index e129522..3aff270 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -276,6 +276,7 @@ source "package/python-mad/Config.in"
source "package/python-meld3/Config.in"
source "package/python-netifaces/Config.in"
source "package/python-nfc/Config.in"
+source "package/python-protobuf/Config.in"
source "package/python-pygame/Config.in"
source "package/python-serial/Config.in"
source "package/python-setuptools/Config.in"
diff --git a/package/python-protobuf/Config.in b/package/python-protobuf/Config.in
new file mode 100644
index 0000000..d2bf70d
--- /dev/null
+++ b/package/python-protobuf/Config.in
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PYTHON_PROTOBUF
+ bool "python-protobuf"
+ depends on BR2_PACKAGE_PYTHON
+ select BR2_PACKAGE_PYTHON_SETUPTOOLS
+ help
+ Python implementation of the Google Protocol Buffers.
+
+ Protocol buffers are Google's language-neutral, platform-neutral,
+ extensible mechanism for serializing structured data.
+
+ http://code.google.com/p/protobuf/
diff --git a/package/python-protobuf/python-protobuf.mk b/package/python-protobuf/python-protobuf.mk
new file mode 100644
index 0000000..d632e00
--- /dev/null
+++ b/package/python-protobuf/python-protobuf.mk
@@ -0,0 +1,28 @@
+#############################################################
+#
+# python-protobuf
+#
+#############################################################
+
+PYTHON_PROTOBUF_VERSION = $(PROTOBUF_VERSION)
+PYTHON_PROTOBUF_SOURCE = $(PROTOBUF_SOURCE)
+PYTHON_PROTOBUF_SITE = $(PROTOBUF_SITE)
+PYTHON_PROTOBUF_LICENSE = BSD-3c
+PYTHON_PROTOBUF_LICENSE_FILES = COPYING.txt
+
+PYTHON_PROTOBUF_DEPENDENCIES = python host-python-setuptools host-python-distutilscross host-protobuf
+
+define PYTHON_PROTOBUF_BUILD_CMDS
+ (cd $(@D)/python; \
+ PYTHONXCPREFIX="$(STAGING_DIR)/usr/" \
+ LDFLAGS="-L$(STAGING_DIR)/lib -L$(STAGING_DIR)/usr/lib" \
+ PATH=$(HOST_PATH) \
+ $(HOST_DIR)/usr/bin/python setup.py build -x)
+endef
+
+define PYTHON_PROTOBUF_INSTALL_TARGET_CMDS
+ (cd $(@D)/python; PYTHONPATH=$(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR)/site-packages \
+ $(HOST_DIR)/usr/bin/python setup.py install --prefix=$(TARGET_DIR)/usr)
+endef
+
+$(eval $(generic-package))
--
1.7.3.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers
2012-09-17 10:51 [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers Thiago A. Correa
@ 2012-09-20 20:34 ` Thomas Petazzoni
2012-09-20 21:40 ` Arnout Vandecappelle
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2012-09-20 20:34 UTC (permalink / raw)
To: buildroot
Dear Thiago A. Correa,
On Mon, 17 Sep 2012 07:51:03 -0300, Thiago A. Correa wrote:
> Signed-off-by: Thiago A. Correa <thiago.correa@gmail.com>
> ---
> package/Config.in | 1 +
> package/python-protobuf/Config.in | 11 +++++++++++
> package/python-protobuf/python-protobuf.mk | 28 ++++++++++++++++++++++++++++
> 3 files changed, 40 insertions(+), 0 deletions(-)
> create mode 100644 package/python-protobuf/Config.in
> create mode 100644 package/python-protobuf/python-protobuf.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index e129522..3aff270 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -276,6 +276,7 @@ source "package/python-mad/Config.in"
> source "package/python-meld3/Config.in"
> source "package/python-netifaces/Config.in"
> source "package/python-nfc/Config.in"
> +source "package/python-protobuf/Config.in"
> source "package/python-pygame/Config.in"
> source "package/python-serial/Config.in"
> source "package/python-setuptools/Config.in"
> diff --git a/package/python-protobuf/Config.in b/package/python-protobuf/Config.in
> new file mode 100644
> index 0000000..d2bf70d
> --- /dev/null
> +++ b/package/python-protobuf/Config.in
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_PYTHON_PROTOBUF
> + bool "python-protobuf"
> + depends on BR2_PACKAGE_PYTHON
> + select BR2_PACKAGE_PYTHON_SETUPTOOLS
Is this select really needed? python-setuptools for the target is not
amongst the dependencies of your package. It seems you only need
host-python-setuptools, in which case this select is not necessary.
> + help
> + Python implementation of the Google Protocol Buffers.
> +
> + Protocol buffers are Google's language-neutral, platform-neutral,
> + extensible mechanism for serializing structured data.
> +
> + http://code.google.com/p/protobuf/
> diff --git a/package/python-protobuf/python-protobuf.mk b/package/python-protobuf/python-protobuf.mk
> new file mode 100644
> index 0000000..d632e00
> --- /dev/null
> +++ b/package/python-protobuf/python-protobuf.mk
> @@ -0,0 +1,28 @@
> +#############################################################
> +#
> +# python-protobuf
> +#
> +#############################################################
> +
> +PYTHON_PROTOBUF_VERSION = $(PROTOBUF_VERSION)
> +PYTHON_PROTOBUF_SOURCE = $(PROTOBUF_SOURCE)
> +PYTHON_PROTOBUF_SITE = $(PROTOBUF_SITE)
So the source tarball is the same? So what you're doing here in fact is
having two separate Buildroot packages for the same source tarball,
something that we generally don't do. But in this case, it seems like
logical to do it this way.
What do others think about this strategy?
> +PYTHON_PROTOBUF_LICENSE = BSD-3c
> +PYTHON_PROTOBUF_LICENSE_FILES = COPYING.txt
> +
> +PYTHON_PROTOBUF_DEPENDENCIES = python host-python-setuptools host-python-distutilscross host-protobuf
Using \ to cut the line width would be nice.
> +define PYTHON_PROTOBUF_BUILD_CMDS
> + (cd $(@D)/python; \
> + PYTHONXCPREFIX="$(STAGING_DIR)/usr/" \
> + LDFLAGS="-L$(STAGING_DIR)/lib -L$(STAGING_DIR)/usr/lib" \
And you sure those LDFLAGS are necessary? Our cross-compiler already
has its sysroot set so that it finds libraries in the staging directory.
> + PATH=$(HOST_PATH) \
> + $(HOST_DIR)/usr/bin/python setup.py build -x)
> +endef
> +
> +define PYTHON_PROTOBUF_INSTALL_TARGET_CMDS
> + (cd $(@D)/python; PYTHONPATH=$(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR)/site-packages \
> + $(HOST_DIR)/usr/bin/python setup.py install --prefix=$(TARGET_DIR)/usr)
> +endef
> +
> +$(eval $(generic-package))
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] 4+ messages in thread
* [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers
2012-09-20 20:34 ` Thomas Petazzoni
@ 2012-09-20 21:40 ` Arnout Vandecappelle
2012-10-05 17:55 ` Thiago A. Corrêa
0 siblings, 1 reply; 4+ messages in thread
From: Arnout Vandecappelle @ 2012-09-20 21:40 UTC (permalink / raw)
To: buildroot
On 09/20/12 22:34, Thomas Petazzoni wrote:
>> > +PYTHON_PROTOBUF_VERSION = $(PROTOBUF_VERSION)
>> > +PYTHON_PROTOBUF_SOURCE = $(PROTOBUF_SOURCE)
>> > +PYTHON_PROTOBUF_SITE = $(PROTOBUF_SITE)
> So the source tarball is the same? So what you're doing here in fact is
> having two separate Buildroot packages for the same source tarball,
> something that we generally don't do. But in this case, it seems like
> logical to do it this way.
>
> What do others think about this strategy?
>
It's a good start, but I still think we need more generic infrastructure
for this use case. For instance, this will fail if protobuf moves to a
hg version, because then PROTOBUF_SOURCE is empty.
For the time being, it's OK, however.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
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] 4+ messages in thread
* [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers
2012-09-20 21:40 ` Arnout Vandecappelle
@ 2012-10-05 17:55 ` Thiago A. Corrêa
0 siblings, 0 replies; 4+ messages in thread
From: Thiago A. Corrêa @ 2012-10-05 17:55 UTC (permalink / raw)
To: buildroot
Hi,
Somehow I missed your replies. I will resubmit with the changes
suggested. Thanks.
Kind Regards,
Thiago A. Correa
On Thu, Sep 20, 2012 at 6:40 PM, Arnout Vandecappelle <arnout@mind.be> wrote:
> On 09/20/12 22:34, Thomas Petazzoni wrote:
>>>
>>> > +PYTHON_PROTOBUF_VERSION = $(PROTOBUF_VERSION)
>>> > +PYTHON_PROTOBUF_SOURCE = $(PROTOBUF_SOURCE)
>>> > +PYTHON_PROTOBUF_SITE = $(PROTOBUF_SITE)
>>
>> So the source tarball is the same? So what you're doing here in fact is
>> having two separate Buildroot packages for the same source tarball,
>> something that we generally don't do. But in this case, it seems like
>> logical to do it this way.
>>
>> What do others think about this strategy?
>>
>
> It's a good start, but I still think we need more generic infrastructure
> for this use case. For instance, this will fail if protobuf moves to a
> hg version, because then PROTOBUF_SOURCE is empty.
>
> For the time being, it's OK, however.
>
> Regards,
> Arnout
> --
> Arnout Vandecappelle arnout at mind be
> Senior Embedded Software Architect +32-16-286540
> 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] 4+ messages in thread
end of thread, other threads:[~2012-10-05 17:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-17 10:51 [Buildroot] [PATCHv2][RESEND] python-protobuf: Add support for Python implementation of Google's Protocol Buffers Thiago A. Correa
2012-09-20 20:34 ` Thomas Petazzoni
2012-09-20 21:40 ` Arnout Vandecappelle
2012-10-05 17:55 ` Thiago A. Corrêa
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox