Re: [meta-ti][master/wrynose][PATCH] libsdl2: Set runtime dependency for SDL2 tests

On 6/29/26 9:50 AM, Antonios Christidis wrote:
>=20
> On 6/25/26 5:38 PM, Andrew Davis wrote:
>> On 6/25/26 3:09 PM, Antonios Christidis via lists.yoctoproject.org wro=
te:
>>> From: Antonios Christidis <[email protected]>
>>>
>>> Set a runtime dependency on libsdl2 for the libsdl2-tests package.
>>>
>>
>> Any clue on why this isn't automatic? Do the application in the test
>> package not link to libsdl2?
>>
> This isn't automatic because a custom package is created using the PACK=
AGE_BEFORE_PN
> variable, so I need to explicitly set its dependencies.
>=20

Odd, wasn't aware that would mess up the automatic dependency tracking. S=
o side
question then, why use `PACKAGE_BEFORE_PN`? What is special about this re=
cipe
that the test package can't be a normally added sub-package?

> Currently, when SDL_TESTS is enabled, the test binaries have SDL2 libs =
statically
> compiled in. However, I'm sending a revision to add support for dynamic=
 linking instead.
> This is needed because customers using the sysroots to cross-compile th=
eir applications
> have the SDL2 libraries available in their build environment, but the b=
inaries can't run
> since those libraries don't exist in the target filesystem.

Sounds good. BTW, any idea why the tests statically linked with the libra=
ry before?
Even if the library includes both dynamic and static libs, I would expect=
 the default
to always be dynamic linking (saves space, upgradable, etc..)

Andrew

>=20
> |
>=20
> |
>=20
>> Andrew
>>
>>> Signed-off-by: Antonios Christidis <[email protected]>
>>> ---
>>> =C2=A0 meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc | 1 +
>>> =C2=A0 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc b/m=
eta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
>>> index 33ff0632..750a5603 100644
>>> --- a/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
>>> +++ b/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
>>> @@ -3,3 +3,4 @@ EXTRA_OECMAKE +=3D "-DSDL_TESTS=3DON \
>>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "
>>> =C2=A0 PACKAGE_BEFORE_PN =3D "${PN}-tests"
>>> =C2=A0 FILES:${PN}-tests +=3D "${libexecdir} ${datadir}/installed-tes=
ts/SDL2"
>>> +RDEPENDS:${PN}-tests +=3D "${PN}"
>>>
>>>
>>>
>>> -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
>>> Links: You receive all messages sent to this group.
>>> View/Reply Online (#20098): https://lists.yoctoproject.org/g/meta-ti/=
message/20098
>>> Mute This Topic: https://lists.yoctoproject.org/mt/119980196/3619733
>>> Group Owner: [email protected]
>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [[email protected]=
om]
>>> -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
>>>
>>