Re: [OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-10 Thread Ricardo Ribalda Delgado
Hello On Sun, May 10, 2015 at 12:00 AM, Richard Purdie wrote: > > GSTREAMER_CONFIG = "" = or ?= > PACKAGECONFIG ??= "${GSTREAMER_CONFIG}" > > then setting GSTREAMER_CONFIG = "debug" could do what you describe if > there really is the demand for it. Shall I send a patch to fix it also on plugi

Re: [OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-09 Thread Richard Purdie
On Fri, 2015-05-08 at 13:35 -0700, Andre McCurdy wrote: > On Fri, May 8, 2015 at 12:12 PM, Otavio Salvador > wrote: > > On Fri, May 8, 2015 at 4:07 PM, Andre McCurdy wrote: > > ... > >> Or perhaps even use the "GSTREAMER_DEBUG" variable in gstreamer 1.x > >> recipes? > > > > I think the PACKAGEC

Re: [OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-08 Thread Andre McCurdy
On Fri, May 8, 2015 at 12:12 PM, Otavio Salvador wrote: > On Fri, May 8, 2015 at 4:07 PM, Andre McCurdy wrote: > ... >> Or perhaps even use the "GSTREAMER_DEBUG" variable in gstreamer 1.x recipes? > > I think the PACKAGECONFIG is a nice compromise as it allows for > granular control over what to

Re: [OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-08 Thread Otavio Salvador
On Fri, May 8, 2015 at 4:07 PM, Andre McCurdy wrote: ... > Or perhaps even use the "GSTREAMER_DEBUG" variable in gstreamer 1.x recipes? I think the PACKAGECONFIG is a nice compromise as it allows for granular control over what to enable or not. It is easy to enable it for recipes in a local.conf

Re: [OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-08 Thread Andre McCurdy
On Fri, May 8, 2015 at 8:04 AM, Ricardo Ribalda Delgado wrote: > It is more elegant For gstreamer 0.10, it was possible to enable debug for gstreamer and most gstreamer plug-ins by over-riding a single variable, GSTREAMER_DEBUG. It might be nice if gstreamer 1.x allowed a similar approach. For t

[OE-core] [PATCH 1/2] gstreamer1.0: convert GSTREAMER_1_DEBUG to PACKAGECONFIG

2015-05-08 Thread Ricardo Ribalda Delgado
It is more elegant Suggested-by: Ross Burton Signed-off-by: Ricardo Ribalda Delgado --- meta/recipes-multimedia/gstreamer/gstreamer1.0.inc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0.inc b/meta/recipes-multimedia/gstre