public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
       [not found] <189E1B5A8FDE995D.2964266@lists.openembedded.org>
@ 2026-04-13  3:17 ` Changqing Li
  2026-04-13  6:05   ` Mathieu Dubois-Briand
  0 siblings, 1 reply; 5+ messages in thread
From: Changqing Li @ 2026-04-13  3:17 UTC (permalink / raw)
  To: openembedded-core

kindly ping

On 3/19/26 09:54, Changqing Li via lists.openembedded.org wrote:
> When vulkan is enabled in DISTRO_FEATURES without x11 or wayland,
> do_configure will failed with error:
> gst-libs/gst/vulkan/meson.build:311:4: ERROR: Problem encountered: No Windowing system found. vulkansink will not work
>
> set ANY_OF_DISTRO_FEATURES to make gstreamer1.0-plugins-bad build only
> when x11 or wayland is enabled.
>
> Signed-off-by: Changqing Li <changqing.li@windriver.com>
> ---
>   .../gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb              | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> index 4a9cb37dd8..2cb00f180f 100644
> --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> @@ -19,7 +19,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c"
>   
>   DEPENDS += "gstreamer1.0-plugins-base"
>   
> -inherit gobject-introspection
> +inherit gobject-introspection features_check
> +
> +ANY_OF_DISTRO_FEATURES = "${@bb.utils.contains('DISTRO_FEATURES', 'vulkan', 'x11 wayland', '', d)}"
>   
>   PACKAGECONFIG ??= " \
>       ${GSTREAMER_ORC} \
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#233455): https://lists.openembedded.org/g/openembedded-core/message/233455
> Mute This Topic: https://lists.openembedded.org/mt/118394443/3616873
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [changqing.li@windriver.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
  2026-04-13  3:17 ` [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES Changqing Li
@ 2026-04-13  6:05   ` Mathieu Dubois-Briand
  2026-04-13  6:55     ` Changqing Li
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Dubois-Briand @ 2026-04-13  6:05 UTC (permalink / raw)
  To: changqing.li, openembedded-core

On Mon Apr 13, 2026 at 5:17 AM CEST, Changqing Li via lists.openembedded.org wrote:
> kindly ping
>

Hi Changqing,

Your patch is still under review, but we have a pretty long backlog. It
should be merged or you should get some feedback soon.

Thanks,
Mathieu

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
  2026-04-13  6:05   ` Mathieu Dubois-Briand
@ 2026-04-13  6:55     ` Changqing Li
  0 siblings, 0 replies; 5+ messages in thread
From: Changqing Li @ 2026-04-13  6:55 UTC (permalink / raw)
  To: Mathieu Dubois-Briand, openembedded-core


On 4/13/26 14:05, Mathieu Dubois-Briand wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> On Mon Apr 13, 2026 at 5:17 AM CEST, Changqing Li via lists.openembedded.org wrote:
>> kindly ping
>>
> Hi Changqing,
>
> Your patch is still under review, but we have a pretty long backlog. It
> should be merged or you should get some feedback soon.
>
> Thanks,
> Mathieu

Thanks for your feedback!

//Changqing

> --
> Mathieu Dubois-Briand, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
  2026-03-19  1:54 Changqing Li
@ 2026-04-13 15:42 ` Mathieu Dubois-Briand
  2026-04-14  3:24   ` Changqing Li
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Dubois-Briand @ 2026-04-13 15:42 UTC (permalink / raw)
  To: changqing.li, openembedded-core

On Thu Mar 19, 2026 at 2:54 AM CET, Changqing Li via lists.openembedded.org wrote:
> When vulkan is enabled in DISTRO_FEATURES without x11 or wayland,
> do_configure will failed with error:
> gst-libs/gst/vulkan/meson.build:311:4: ERROR: Problem encountered: No Windowing system found. vulkansink will not work
>
> set ANY_OF_DISTRO_FEATURES to make gstreamer1.0-plugins-bad build only
> when x11 or wayland is enabled.
>
> Signed-off-by: Changqing Li <changqing.li@windriver.com>
> ---

Hi Changqing,

Thanks for your patch.

>  .../gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb              | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> index 4a9cb37dd8..2cb00f180f 100644
> --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
> @@ -19,7 +19,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c"
>  
>  DEPENDS += "gstreamer1.0-plugins-base"
>  
> -inherit gobject-introspection
> +inherit gobject-introspection features_check
> +
> +ANY_OF_DISTRO_FEATURES = "${@bb.utils.contains('DISTRO_FEATURES', 'vulkan', 'x11 wayland', '', d)}"

So the error we get if we try to build with "vulkan" in DISTRO_FEATURES
but not "x11" not "wayland" is the following:

ERROR: Nothing PROVIDES 'gstreamer1.0-plugins-bad'
gstreamer1.0-plugins-bad was skipped: one of 'wayland x11' needs to be in DISTRO_FEATURES

To be honest, I believe this is a bit misleading, as we are not
mentioning the vulkan DISTRO_FEATURES, while this is the main issue:
gstreamer1.0-plugins-bad can be built without wayland or x11, just not
with vulkan.

I would really prefer a better error message, but otherwise we should at
least add some comment in the recipe explaining why we have this
ANY_OF_DISTRO_FEATURES line.

Thanks,
Mathieu

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
  2026-04-13 15:42 ` [OE-core] " Mathieu Dubois-Briand
@ 2026-04-14  3:24   ` Changqing Li
  0 siblings, 0 replies; 5+ messages in thread
From: Changqing Li @ 2026-04-14  3:24 UTC (permalink / raw)
  To: Mathieu Dubois-Briand, openembedded-core

[-- Attachment #1: Type: text/plain, Size: 2812 bytes --]


On 4/13/26 23:42, Mathieu Dubois-Briand wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> On Thu Mar 19, 2026 at 2:54 AM CET, Changqing Li via lists.openembedded.org wrote:
>> When vulkan is enabled in DISTRO_FEATURES without x11 or wayland,
>> do_configure will failed with error:
>> gst-libs/gst/vulkan/meson.build:311:4: ERROR: Problem encountered: No Windowing system found. vulkansink will not work
>>
>> set ANY_OF_DISTRO_FEATURES to make gstreamer1.0-plugins-bad build only
>> when x11 or wayland is enabled.
>>
>> Signed-off-by: Changqing Li<changqing.li@windriver.com>
>> ---
> Hi Changqing,
>
> Thanks for your patch.
>
>>   .../gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb              | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
>> index 4a9cb37dd8..2cb00f180f 100644
>> --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
>> +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.26.9.bb
>> @@ -19,7 +19,9 @@ LIC_FILES_CHKSUM ="file://COPYING;md5=4fbd65380cdd255951079008b364516c"
>>
>>   DEPENDS += "gstreamer1.0-plugins-base"
>>
>> -inherit gobject-introspection
>> +inherit gobject-introspection features_check
>> +
>> +ANY_OF_DISTRO_FEATURES ="${@bb.utils.contains('DISTRO_FEATURES', 'vulkan', 'x11 wayland', '', 
>> d)}"
> So the error we get if we try to build with "vulkan" in DISTRO_FEATURES
> but not "x11" not "wayland" is the following:
>
> ERROR: Nothing PROVIDES 'gstreamer1.0-plugins-bad'
> gstreamer1.0-plugins-bad was skipped: one of 'wayland x11' needs to be in DISTRO_FEATURES

yes, if user bitbake gstreamer1.0-plugins-bad, this will be reported.

And this fix will unblock "bitbake world" since gstreamer1.0-plugins-bad 
is skipped

>
> To be honest, I believe this is a bit misleading, as we are not
> mentioning the vulkan DISTRO_FEATURES, while this is the main issue:
> gstreamer1.0-plugins-bad can be built without wayland or x11, just not
> with vulkan.
user only can get why by checking recipe settings
>
> I would really prefer a better error message, but otherwise we should at
> least add some comment in the recipe explaining why we have this
> ANY_OF_DISTRO_FEATURES line.

I send a V2 to add a comment,  user need to check the comments and 
commit message

to know why ANY_OF_DISTRO_FEATURES is added.

Regards

//changqing

>
> Thanks,
> Mathieu
>
> --
> Mathieu Dubois-Briand, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>

[-- Attachment #2: Type: text/html, Size: 4521 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-04-14  3:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <189E1B5A8FDE995D.2964266@lists.openembedded.org>
2026-04-13  3:17 ` [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES Changqing Li
2026-04-13  6:05   ` Mathieu Dubois-Briand
2026-04-13  6:55     ` Changqing Li
2026-03-19  1:54 Changqing Li
2026-04-13 15:42 ` [OE-core] " Mathieu Dubois-Briand
2026-04-14  3:24   ` Changqing Li

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