diff mbox series

[meta-arago,scarthgap] packagegroup-arago-tisdk-graphics: add mesa-demos and sdl2

Message ID 20250602220116.2091347-1-rs@ti.com
State Under Review
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,scarthgap] packagegroup-arago-tisdk-graphics: add mesa-demos and sdl2 | expand

Commit Message

Randolph Sapp June 2, 2025, 10:01 p.m. UTC
From: Randolph Sapp <rs@ti.com>

Add the new mesa-demos package to this group so that users and scripts
can use eglinfo to query information about supported extensions and
configs for multiple platforms. It also adds vkgears and es2gears to
test Vulkan and GLES 2 respectively.

Add libsdl2 as it's a useful, minimal graphics toolkit that allows users
to build GLES and Vulkan applications with less boilerplate. We have a
few internal test applications that currently use it.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 .../packagegroups/packagegroup-arago-tisdk-graphics.bb          | 2 ++
 1 file changed, 2 insertions(+)

Comments

Andrew Davis June 3, 2025, 8 p.m. UTC | #1
On 6/2/25 5:01 PM, Randolph Sapp via lists.yoctoproject.org wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> Add the new mesa-demos package to this group so that users and scripts
> can use eglinfo to query information about supported extensions and
> configs for multiple platforms. It also adds vkgears and es2gears to
> test Vulkan and GLES 2 respectively.
> 
> Add libsdl2 as it's a useful, minimal graphics toolkit that allows users
> to build GLES and Vulkan applications with less boilerplate. We have a
> few internal test applications that currently use it.

Any reason these applications could not be added here? libsdl2 like other
pure libraries should only be added as dependencies of the applications
that use them.

Andrew

> 
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
>   .../packagegroups/packagegroup-arago-tisdk-graphics.bb          | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
> index fc1a5be3..3728ae52 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
> @@ -9,6 +9,8 @@ inherit packagegroup
>   GFX_WAYLAND = "\
>       weston-init-arago \
>       weston-examples \
> +    mesa-demos \
> +    libsdl2 \
>   "
>   
>   RDEPENDS:${PN} = "\
Randolph Sapp June 4, 2025, 6:05 p.m. UTC | #2
On Tue Jun 3, 2025 at 3:00 PM CDT, Andrew Davis wrote:
> On 6/2/25 5:01 PM, Randolph Sapp via lists.yoctoproject.org wrote:
>> From: Randolph Sapp <rs@ti.com>
>> 
>> Add the new mesa-demos package to this group so that users and scripts
>> can use eglinfo to query information about supported extensions and
>> configs for multiple platforms. It also adds vkgears and es2gears to
>> test Vulkan and GLES 2 respectively.
>> 
>> Add libsdl2 as it's a useful, minimal graphics toolkit that allows users
>> to build GLES and Vulkan applications with less boilerplate. We have a
>> few internal test applications that currently use it.
>
> Any reason these applications could not be added here? libsdl2 like other
> pure libraries should only be added as dependencies of the applications
> that use them.
>
> Andrew

While I normally agree with this sentiment, there are currently private test
applications that depend on SDL2. I can tack this directly into the test groups
if that's preferable. There was just some talk about introducing it as customers
who don't want the overhead of Qt may find it preferable to raw EGL+GLES.

- Randolph

>> 
>> Signed-off-by: Randolph Sapp <rs@ti.com>
>> ---
>>   .../packagegroups/packagegroup-arago-tisdk-graphics.bb          | 2 ++
>>   1 file changed, 2 insertions(+)
>> 
>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>> index fc1a5be3..3728ae52 100644
>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>> @@ -9,6 +9,8 @@ inherit packagegroup
>>   GFX_WAYLAND = "\
>>       weston-init-arago \
>>       weston-examples \
>> +    mesa-demos \
>> +    libsdl2 \
>>   "
>>   
>>   RDEPENDS:${PN} = "\
Ryan Eatmon June 4, 2025, 7:09 p.m. UTC | #3
On 6/4/2025 1:05 PM, Randolph Sapp wrote:
> On Tue Jun 3, 2025 at 3:00 PM CDT, Andrew Davis wrote:
>> On 6/2/25 5:01 PM, Randolph Sapp via lists.yoctoproject.org wrote:
>>> From: Randolph Sapp <rs@ti.com>
>>>
>>> Add the new mesa-demos package to this group so that users and scripts
>>> can use eglinfo to query information about supported extensions and
>>> configs for multiple platforms. It also adds vkgears and es2gears to
>>> test Vulkan and GLES 2 respectively.
>>>
>>> Add libsdl2 as it's a useful, minimal graphics toolkit that allows users
>>> to build GLES and Vulkan applications with less boilerplate. We have a
>>> few internal test applications that currently use it.
>>
>> Any reason these applications could not be added here? libsdl2 like other
>> pure libraries should only be added as dependencies of the applications
>> that use them.
>>
>> Andrew
> 
> While I normally agree with this sentiment, there are currently private test
> applications that depend on SDL2. I can tack this directly into the test groups
> if that's preferable. There was just some talk about introducing it as customers
> who don't want the overhead of Qt may find it preferable to raw EGL+GLES.

So... there is a non-public program that we download during testing that 
requires the library to be on the system in order to run?  And this 
program cannot be made available...

In general I agree with Andrew on this.  We should not be randomly 
including libraries in the rootfs that do not have programs that require 
them.

It might better to create a new packagegroup for running these kinds of 
tests and then the test can pull that packagegroup 
(packagegroup-arago-tisdk-graphics-testlibs?) as part of the test.  That 
way we keep the base rootfs cleaner for everyone else *not* running this 
private hidden mystery program.  We can build that packagegroup as part 
of ti-world so that is available in the package feed.


> - Randolph
> 
>>>
>>> Signed-off-by: Randolph Sapp <rs@ti.com>
>>> ---
>>>    .../packagegroups/packagegroup-arago-tisdk-graphics.bb          | 2 ++
>>>    1 file changed, 2 insertions(+)
>>>
>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>>> index fc1a5be3..3728ae52 100644
>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
>>> @@ -9,6 +9,8 @@ inherit packagegroup
>>>    GFX_WAYLAND = "\
>>>        weston-init-arago \
>>>        weston-examples \
>>> +    mesa-demos \
>>> +    libsdl2 \
>>>    "
>>>    
>>>    RDEPENDS:${PN} = "\
>
diff mbox series

Patch

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
index fc1a5be3..3728ae52 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-graphics.bb
@@ -9,6 +9,8 @@  inherit packagegroup
 GFX_WAYLAND = "\
     weston-init-arago \
     weston-examples \
+    mesa-demos \
+    libsdl2 \
 "
 
 RDEPENDS:${PN} = "\