[PATCH] gstreamer: Provide easy way to enable runtime debugging

[PATCH] gstreamer: Provide easy way to enable runtime debugging

[Gary Thomas]

The gstreamer framework has a very useful debugging setup which is
essential for debugging pipelines and plugins.  This patch makes
it simple to enable this (disabled by default).  To enable debugging,
just add this line to local.conf
  GSTREAMER_DEBUG = "--enable-debug"

Signed-off-by: Gary Thomas <gary@mlbassoc.com>
---
 .../gstreamer/gst-ffmpeg_0.10.13.bb                |    3 ++-
 meta/recipes-multimedia/gstreamer/gst-fluendo.inc  |    3 ++-
 meta/recipes-multimedia/gstreamer/gst-plugins.inc  |    3 ++-
 .../gstreamer/gstreamer_0.10.36.bb                 |    3 ++-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
index 5ee5066..92cd349 100644
--- a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
@@ -25,7 +25,8 @@ SRC_URI[sha256sum] = "76fca05b08e00134e3cb92fa347507f42cbd48ddb08ed3343a912def18
 
 PR = "r1"
 
-EXTRA_OECONF = "--with-ffmpeg-extra-configure=\"--target-os=linux\" "
+GSTREAMER_DEBUG ?= "--disable-debug"
+EXTRA_OECONF = "--with-ffmpeg-extra-configure=\"--target-os=linux\" ${GSTREAMER_DEBUG}"
 
 # yasm not found, use --disable-yasm for a crippled build for libav
 EXTRA_OECONF_append_x86-64 = " --disable-yasm "
diff --git a/meta/recipes-multimedia/gstreamer/gst-fluendo.inc b/meta/recipes-multimedia/gstreamer/gst-fluendo.inc
index 8b24cf7..b2c7eea 100644
--- a/meta/recipes-multimedia/gstreamer/gst-fluendo.inc
+++ b/meta/recipes-multimedia/gstreamer/gst-fluendo.inc
@@ -11,4 +11,5 @@ FILES_${PN} += "${libdir}/gstreamer-0.10/*.so"
 FILES_${PN}-dbg += "${libdir}/gstreamer-0.10/.debug"
 FILES_${PN}-dev += "${libdir}/gstreamer-0.10/*.la ${libdir}/gstreamer-0.10/*.a"
 
-EXTRA_OECONF = "--disable-debug --disable-valgrind"
+GSTREAMER_DEBUG ?= "--disable-debug"
+EXTRA_OECONF = "${GSTREAMER_DEBUG} --disable-valgrind"
diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins.inc b/meta/recipes-multimedia/gstreamer/gst-plugins.inc
index f11a4af..ccb81b3 100644
--- a/meta/recipes-multimedia/gstreamer/gst-plugins.inc
+++ b/meta/recipes-multimedia/gstreamer/gst-plugins.inc
@@ -10,7 +10,8 @@ FILESPATH =. "${FILE_DIRNAME}/gst-plugins:"
 
 SRC_URI = "http://gstreamer.freedesktop.org/src/${BPN}/${BPN}-${PV}.tar.bz2"
 
-EXTRA_OECONF = "--disable-valgrind --disable-debug --disable-examples "
+GSTREAMER_DEBUG ?= "--disable-debug"
+EXTRA_OECONF = "--disable-valgrind ${GSTREAMER_DEBUG} --disable-examples "
 
 acpaths = "-I ${S}/common/m4 -I ${S}/m4"
 
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer_0.10.36.bb b/meta/recipes-multimedia/gstreamer/gstreamer_0.10.36.bb
index 3c03c0d..278c20e 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer_0.10.36.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer_0.10.36.bb
@@ -20,7 +20,8 @@ SRC_URI[sha256sum] = "e556a529e0a8cf1cd0afd0cab2af5488c9524e7c3f409de29b5d82bb41
 
 inherit autotools pkgconfig gettext
 
-EXTRA_OECONF = "--disable-docs-build --disable-dependency-tracking --with-check=no --disable-examples --disable-tests --disable-valgrind --disable-debug"
+GSTREAMER_DEBUG ?= "--disable-debug"
+EXTRA_OECONF = "--disable-docs-build --disable-dependency-tracking --with-check=no --disable-examples --disable-tests --disable-valgrind ${GSTREAMER_DEBUG}"
 
 #do_compile_prepend () {
 #	mv ${WORKDIR}/gstregistrybinary.[ch] ${S}/gst/
-- 
1.7.7.6

Re: [PATCH] gstreamer: Provide easy way to enable runtime debugging

[Richard Purdie]

On Thu, 2012-03-29 at 08:40 -0600, Gary Thomas wrote:
> The gstreamer framework has a very useful debugging setup which is
> essential for debugging pipelines and plugins.  This patch makes
> it simple to enable this (disabled by default).  To enable debugging,
> just add this line to local.conf
>   GSTREAMER_DEBUG = "--enable-debug"
> 
> Signed-off-by: Gary Thomas <gary@mlbassoc.com>
> ---
>  .../gstreamer/gst-ffmpeg_0.10.13.bb                |    3 ++-
>  meta/recipes-multimedia/gstreamer/gst-fluendo.inc  |    3 ++-
>  meta/recipes-multimedia/gstreamer/gst-plugins.inc  |    3 ++-
>  .../gstreamer/gstreamer_0.10.36.bb                 |    3 ++-
>  4 files changed, 8 insertions(+), 4 deletions(-)

Merged to master, thanks.

Richard

Re: [PATCH] gstreamer: Provide easy way to enable runtime debugging

[Koen Kooi]

Op 29 mrt. 2012, om 14:19 heeft Richard Purdie het volgende geschreven:

> On Thu, 2012-03-29 at 08:40 -0600, Gary Thomas wrote:
>> The gstreamer framework has a very useful debugging setup which is
>> essential for debugging pipelines and plugins.  This patch makes
>> it simple to enable this (disabled by default).  To enable debugging,
>> just add this line to local.conf
>>  GSTREAMER_DEBUG = "--enable-debug"
>> 
>> Signed-off-by: Gary Thomas <gary@mlbassoc.com>
>> ---
>> .../gstreamer/gst-ffmpeg_0.10.13.bb                |    3 ++-
>> meta/recipes-multimedia/gstreamer/gst-fluendo.inc  |    3 ++-
>> meta/recipes-multimedia/gstreamer/gst-plugins.inc  |    3 ++-
>> .../gstreamer/gstreamer_0.10.36.bb                 |    3 ++-
>> 4 files changed, 8 insertions(+), 4 deletions(-)
> 
> Merged to master, thanks.

Awesome, more magic variables. Where's the matching documentation for this one? And why isn't this using PACKAGE_CONFIG?