* [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally
@ 2012-02-17 22:56 Andreas Oberritter
2012-02-17 22:56 ` [PATCH 2/2] mpeg2dec: depend on libx11 conditionally Andreas Oberritter
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Andreas Oberritter @ 2012-02-17 22:56 UTC (permalink / raw)
To: openembedded-core
* gst-plugins-base currently doesn't build if x11 is
unavailable, due to unconditional dependencies on libx11
and libxv. However, they are not required to build
gst-plugins-base.
Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
.../gstreamer/gst-plugins-base_0.10.35.bb | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
index e49a00c..0b287bc 100644
--- a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
@@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
file://COPYING.LIB;md5=55ca817ccb7d5b5b66355690e9abc605 \
file://gst/ffmpegcolorspace/utils.c;beginline=1;endline=20;md5=9c83a200b8e597b26ca29df20fc6ecd0"
-DEPENDS += "virtual/libx11 alsa-lib freetype liboil libogg libvorbis libxv libtheora avahi util-linux tremor"
+DEPENDS += "${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxv', '', d)}"
+DEPENDS += "alsa-lib freetype liboil libogg libvorbis libtheora avahi util-linux tremor"
SRC_URI += " file://gst-plugins-base-tremor.patch"
--
1.7.5.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 2/2] mpeg2dec: depend on libx11 conditionally
2012-02-17 22:56 [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Andreas Oberritter
@ 2012-02-17 22:56 ` Andreas Oberritter
2012-02-23 0:58 ` [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Saul Wold
2012-02-24 4:15 ` Saul Wold
2 siblings, 0 replies; 5+ messages in thread
From: Andreas Oberritter @ 2012-02-17 22:56 UTC (permalink / raw)
To: openembedded-core
* mpeg2dec currently doesn't build if x11 is unavailable,
due to an unconditional dependency on libx11. However,
libx11 is not required to build mpeg2dec.
Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
meta/recipes-multimedia/mpeg2dec/mpeg2dec_0.4.1.bb | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/meta/recipes-multimedia/mpeg2dec/mpeg2dec_0.4.1.bb b/meta/recipes-multimedia/mpeg2dec/mpeg2dec_0.4.1.bb
index f2865f6..f38066a 100644
--- a/meta/recipes-multimedia/mpeg2dec/mpeg2dec_0.4.1.bb
+++ b/meta/recipes-multimedia/mpeg2dec/mpeg2dec_0.4.1.bb
@@ -6,7 +6,7 @@ LICENSE_FLAGS = "commercial"
LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \
file://include/mpeg2.h;beginline=1;endline=22;md5=ead62602d4638329d3b5b86a55803154"
-DEPENDS = "virtual/libx11"
+DEPENDS = "${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11', '', d)}"
PR = "r2"
--
1.7.5.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally
2012-02-17 22:56 [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Andreas Oberritter
2012-02-17 22:56 ` [PATCH 2/2] mpeg2dec: depend on libx11 conditionally Andreas Oberritter
@ 2012-02-23 0:58 ` Saul Wold
2012-02-23 1:49 ` Andreas Oberritter
2012-02-24 4:15 ` Saul Wold
2 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2012-02-23 0:58 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On 02/17/2012 02:56 PM, Andreas Oberritter wrote:
> * gst-plugins-base currently doesn't build if x11 is
> unavailable, due to unconditional dependencies on libx11
> and libxv. However, they are not required to build
> gst-plugins-base.
>
> Signed-off-by: Andreas Oberritter<obi@opendreambox.org>
> ---
> .../gstreamer/gst-plugins-base_0.10.35.bb | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> index e49a00c..0b287bc 100644
> --- a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> @@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
> file://COPYING.LIB;md5=55ca817ccb7d5b5b66355690e9abc605 \
> file://gst/ffmpegcolorspace/utils.c;beginline=1;endline=20;md5=9c83a200b8e597b26ca29df20fc6ecd0"
>
> -DEPENDS += "virtual/libx11 alsa-lib freetype liboil libogg libvorbis libxv libtheora avahi util-linux tremor"
> +DEPENDS += "${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxv', '', d)}"
> +DEPENDS += "alsa-lib freetype liboil libogg libvorbis libtheora avahi util-linux tremor"
>
> SRC_URI += " file://gst-plugins-base-tremor.patch"
>
As with the other patches do you need to have some configure option to
ensure that these will be exculded if perhaps the build already contains
these libraries?
This also pertains to your mpeg2dev patch
Sau!
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally
2012-02-23 0:58 ` [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Saul Wold
@ 2012-02-23 1:49 ` Andreas Oberritter
0 siblings, 0 replies; 5+ messages in thread
From: Andreas Oberritter @ 2012-02-23 1:49 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On 23.02.2012 01:58, Saul Wold wrote:
> On 02/17/2012 02:56 PM, Andreas Oberritter wrote:
>> * gst-plugins-base currently doesn't build if x11 is
>> unavailable, due to unconditional dependencies on libx11
>> and libxv. However, they are not required to build
>> gst-plugins-base.
>>
>> Signed-off-by: Andreas Oberritter<obi@opendreambox.org>
>> ---
>> .../gstreamer/gst-plugins-base_0.10.35.bb | 3 ++-
>> 1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git
>> a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
>> b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
>> index e49a00c..0b287bc 100644
>> --- a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
>> +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
>> @@ -6,7 +6,8 @@ LIC_FILES_CHKSUM =
>> "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
>>
>> file://COPYING.LIB;md5=55ca817ccb7d5b5b66355690e9abc605 \
>>
>> file://gst/ffmpegcolorspace/utils.c;beginline=1;endline=20;md5=9c83a200b8e597b26ca29df20fc6ecd0"
>>
>>
>> -DEPENDS += "virtual/libx11 alsa-lib freetype liboil libogg libvorbis
>> libxv libtheora avahi util-linux tremor"
>> +DEPENDS += "${@base_contains('DISTRO_FEATURES', 'x11',
>> 'virtual/libx11 libxv', '', d)}"
>> +DEPENDS += "alsa-lib freetype liboil libogg libvorbis libtheora avahi
>> util-linux tremor"
>>
>> SRC_URI += " file://gst-plugins-base-tremor.patch"
>>
> As with the other patches do you need to have some configure option to
> ensure that these will be exculded if perhaps the build already contains
> these libraries?
No, unless it should be supported to first build with
DISTRO_FEATURES="x11" and later on with DISTRO_FEATURES="" without
cleaning tmp in between. I don't think it's possible to get a consistent
build and feed repository that way.
If x11 is unset, libx11 can't be built and therefore gst-plugins-base
can't be built either (as noted in the commit message), currently.
Regards,
Andreas
> This also pertains to your mpeg2dev patch
>
> Sau!
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally
2012-02-17 22:56 [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Andreas Oberritter
2012-02-17 22:56 ` [PATCH 2/2] mpeg2dec: depend on libx11 conditionally Andreas Oberritter
2012-02-23 0:58 ` [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Saul Wold
@ 2012-02-24 4:15 ` Saul Wold
2 siblings, 0 replies; 5+ messages in thread
From: Saul Wold @ 2012-02-24 4:15 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On 02/17/2012 02:56 PM, Andreas Oberritter wrote:
> * gst-plugins-base currently doesn't build if x11 is
> unavailable, due to unconditional dependencies on libx11
> and libxv. However, they are not required to build
> gst-plugins-base.
>
> Signed-off-by: Andreas Oberritter<obi@opendreambox.org>
> ---
> .../gstreamer/gst-plugins-base_0.10.35.bb | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> index e49a00c..0b287bc 100644
> --- a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.35.bb
> @@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
> file://COPYING.LIB;md5=55ca817ccb7d5b5b66355690e9abc605 \
> file://gst/ffmpegcolorspace/utils.c;beginline=1;endline=20;md5=9c83a200b8e597b26ca29df20fc6ecd0"
>
> -DEPENDS += "virtual/libx11 alsa-lib freetype liboil libogg libvorbis libxv libtheora avahi util-linux tremor"
> +DEPENDS += "${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxv', '', d)}"
> +DEPENDS += "alsa-lib freetype liboil libogg libvorbis libtheora avahi util-linux tremor"
>
> SRC_URI += " file://gst-plugins-base-tremor.patch"
>
Merged into OE-core
Thanks
Sau!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-02-24 4:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-17 22:56 [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Andreas Oberritter
2012-02-17 22:56 ` [PATCH 2/2] mpeg2dec: depend on libx11 conditionally Andreas Oberritter
2012-02-23 0:58 ` [PATCH 1/2] gst-plugins-base: depend on libx11 and libxv conditionally Saul Wold
2012-02-23 1:49 ` Andreas Oberritter
2012-02-24 4:15 ` Saul Wold
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox