public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Changqing Li <changqing.li@windriver.com>
To: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES
Date: Tue, 14 Apr 2026 11:24:10 +0800	[thread overview]
Message-ID: <979ca88a-b3d6-491e-8344-044e9bf5e8d7@windriver.com> (raw)
In-Reply-To: <DHS549F2NTOV.O6QC1JNNT3O2@bootlin.com>

[-- 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 --]

  reply	other threads:[~2026-04-14  3:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-19  1:54 [PATCH] gstreamer1.0-plugins-bad: add ANY_OF_DISTRO_FEATURES Changqing Li
2026-04-13 15:42 ` [OE-core] " Mathieu Dubois-Briand
2026-04-14  3:24   ` Changqing Li [this message]
2026-04-14  3:28   ` [PATCH V2] " Changqing Li
2026-04-14 10:27     ` [OE-core] " Peter Kjellerstedt
2026-04-15  2:13       ` Changqing Li
     [not found] <189E1B5A8FDE995D.2964266@lists.openembedded.org>
2026-04-13  3:17 ` [OE-core] [PATCH] " Changqing Li
2026-04-13  6:05   ` Mathieu Dubois-Briand
2026-04-13  6:55     ` Changqing Li

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=979ca88a-b3d6-491e-8344-044e9bf5e8d7@windriver.com \
    --to=changqing.li@windriver.com \
    --cc=mathieu.dubois-briand@bootlin.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox