Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v3] gst1-validate: needs python
@ 2014-11-12 22:06 Peter Seiderer
  2014-11-13 10:20 ` Vicente Olivert Riera
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Seiderer @ 2014-11-12 22:06 UTC (permalink / raw)
  To: buildroot

Fixes:
 http://autobuild.buildroot.net/results/e8c/e8c27db2a0935c2daef173d5650f6a2b3a219493/

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Changes v2 -> v3:
  - add select BR2_PACKAGE_PYTHON_PYEXPAT (enables python xml package
    needed by gst-validate-launcher)

Changes v1 -> v2:
  - changed from python/python3 to python only (python3 is not usable because
    gst-validate-launcher script is python2 only, see [1] from
    Vicente Olivert Riera)

[1] http://lists.busybox.net/pipermail/buildroot/2014-November/111907.html
---
 package/gstreamer1/gst1-validate/Config.in        | 7 ++++++-
 package/gstreamer1/gst1-validate/gst1-validate.mk | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/package/gstreamer1/gst1-validate/Config.in b/package/gstreamer1/gst1-validate/Config.in
index ffcdd27..34ec85f 100644
--- a/package/gstreamer1/gst1-validate/Config.in
+++ b/package/gstreamer1/gst1-validate/Config.in
@@ -1,9 +1,14 @@
-menuconfig BR2_PACKAGE_GST1_VALIDATE
+config BR2_PACKAGE_GST1_VALIDATE
 	bool "gst1-validate"
+	depends on BR2_PACKAGE_PYTHON
 	select BR2_PACKAGE_GST1_PLUGINS_BASE
+	select BR2_PACKAGE_PYTHON_PYEXPAT
 	help
 	  GstValidate is a tool that allows GStreamer developers to
 	  check that the GstElements they write behave the way they
 	  are supposed to.
 
 	  http://gstreamer.freedesktop.org/
+
+comment "gst1-validate depends on python"
+	depends on !BR2_PACKAGE_PYTHON
diff --git a/package/gstreamer1/gst1-validate/gst1-validate.mk b/package/gstreamer1/gst1-validate/gst1-validate.mk
index 4972cac..9a9e796 100644
--- a/package/gstreamer1/gst1-validate/gst1-validate.mk
+++ b/package/gstreamer1/gst1-validate/gst1-validate.mk
@@ -12,6 +12,6 @@ GST1_VALIDATE_LICENSE_FILES = COPYING
 
 GST1_VALIDATE_CONF_OPTS = --disable-sphinx-doc
 
-GST1_VALIDATE_DEPENDENCIES = gstreamer1 gst1-plugins-base
+GST1_VALIDATE_DEPENDENCIES = gstreamer1 gst1-plugins-base host-python python
 
 $(eval $(autotools-package))
-- 
1.8.1.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* [Buildroot] [PATCH v3] gst1-validate: needs python
@ 2014-11-13 19:38 Peter Seiderer
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Seiderer @ 2014-11-13 19:38 UTC (permalink / raw)
  To: buildroot

Hello Vicente,

On Thu, Nov 13, 2014 at 10:20:22AM +0000, Vicente Olivert Riera wrote:
> Dear Peter Seiderer,
> 
> On 11/12/2014 10:06 PM, Peter Seiderer wrote:
> > Fixes:
> >  http://autobuild.buildroot.net/results/e8c/e8c27db2a0935c2daef173d5650f6a2b3a219493/
> > 
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > ---
> > Changes v2 -> v3:
> >   - add select BR2_PACKAGE_PYTHON_PYEXPAT (enables python xml package
> >     needed by gst-validate-launcher)
> > 
> > Changes v1 -> v2:
> >   - changed from python/python3 to python only (python3 is not usable because
> >     gst-validate-launcher script is python2 only, see [1] from
> >     Vicente Olivert Riera)
> > 
> > [1] http://lists.busybox.net/pipermail/buildroot/2014-November/111907.html
> > ---
> >  package/gstreamer1/gst1-validate/Config.in        | 7 ++++++-
> >  package/gstreamer1/gst1-validate/gst1-validate.mk | 2 +-
> >  2 files changed, 7 insertions(+), 2 deletions(-)
> > 
> > diff --git a/package/gstreamer1/gst1-validate/Config.in b/package/gstreamer1/gst1-validate/Config.in
> > index ffcdd27..34ec85f 100644
> > --- a/package/gstreamer1/gst1-validate/Config.in
> > +++ b/package/gstreamer1/gst1-validate/Config.in
> > @@ -1,9 +1,14 @@
> > -menuconfig BR2_PACKAGE_GST1_VALIDATE
> > +config BR2_PACKAGE_GST1_VALIDATE
> >     bool "gst1-validate"
> > +   depends on BR2_PACKAGE_PYTHON
> >     select BR2_PACKAGE_GST1_PLUGINS_BASE
> > +   select BR2_PACKAGE_PYTHON_PYEXPAT
> >     help
> >       GstValidate is a tool that allows GStreamer developers to
> >       check that the GstElements they write behave the way they
> >       are supposed to.
> >  
> >       http://gstreamer.freedesktop.org/
> > +
> > +comment "gst1-validate depends on python"
> > +   depends on !BR2_PACKAGE_PYTHON
> > diff --git a/package/gstreamer1/gst1-validate/gst1-validate.mk b/package/gstreamer1/gst1-validate/gst1-validate.mk
> > index 4972cac..9a9e796 100644
> > --- a/package/gstreamer1/gst1-validate/gst1-validate.mk
> > +++ b/package/gstreamer1/gst1-validate/gst1-validate.mk
> > @@ -12,6 +12,6 @@ GST1_VALIDATE_LICENSE_FILES = COPYING
> >  
> >  GST1_VALIDATE_CONF_OPTS = --disable-sphinx-doc
> >  
> > -GST1_VALIDATE_DEPENDENCIES = gstreamer1 gst1-plugins-base
> > +GST1_VALIDATE_DEPENDENCIES = gstreamer1 gst1-plugins-base host-python python
> >  
> >  $(eval $(autotools-package))
> 
> I have marked my patch as superseded, so I let you deal with this
> problem :-)

Thanks for the work ;-) ...

> Could you please mark as superseded the older versions of your patch?

Will try, did not work much with patchwork yet...

> 
> Also, could you please write an explanation in the commit message? You
> can take my description and modify it if you want:
> 
> http://patchwork.ozlabs.org/patch/409992/
> 

O.k. will enhance the changelog and send an update patch version...

Regards,
Peter


> Best regards,
> -- 
> Vicente Olivert Riera
> Graduate Software Engineer, MIPS Processor IP
> Imagination Technologies Limited
> t: +44 (0)113 2429814
> www.imgtec.com
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-13 19:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-12 22:06 [Buildroot] [PATCH v3] gst1-validate: needs python Peter Seiderer
2014-11-13 10:20 ` Vicente Olivert Riera
  -- strict thread matches above, loose matches on Subject: below --
2014-11-13 19:38 Peter Seiderer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox