diff mbox series

[meta-ti,master/wrynose,v2] libsdl2: Build tests with dynamically linked library support

Message ID 20260701182948.1604031-1-a-christidis@ti.com
State Under Review
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,master/wrynose,v2] libsdl2: Build tests with dynamically linked library support | expand

Commit Message

Antonios Christidis July 1, 2026, 6:29 p.m. UTC
From: Antonios Christidis <a-christidis@ti.com>

Carry upstream-submitted patch that adds SDL_TESTS_SHARED_LIB option,
enabling test executables to be built with dynamic linking against
libsdl2.

Signed-off-by: Antonios Christidis <a-christidis@ti.com>
---
v2: 
- Remove unnecessary RDEPENDS for tests package, bitbake dependency
  resolution automatically handles this.
 .../recipes-graphics/libsdl2/libsdl2-ti.inc   |  8 +++
 ...TS_SHARED_LIB-option-for-dynamically.patch | 56 +++++++++++++++++++
 2 files changed, 64 insertions(+)
 create mode 100644 meta-ti-test/recipes-graphics/libsdl2/libsdl2/0001-test-New-SDL_TESTS_SHARED_LIB-option-for-dynamically.patch

Comments

PRC Automation July 1, 2026, 6:35 p.m. UTC | #1
meta-ti / na / 20260701182948.1604031-1-a-christidis

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [meta-ti][master/wrynose][PATCH v2] libsdl2: Build tests with dynamically linked library support
- Submitter: From: <a-christidis@ti.com>
From: Antonios Christidis <a-christidis@ti.com>
+From: Antonios Christidis <a-christidis@ti.com>
- Date: Date: Wed, 1 Jul 2026 13:29:48 -0500
+Date: Fri, 26 Jun 2026 13:28:24 -0500
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: e29cc0575fed64eb2e4f3f83a95bb2edd8506005

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: master-wip
- Commit Author: Antonios Christidis <a-christidis@ti.com>
- Commit Subject: libsdl2: Set runtime dependency for SDL2 tests
- Commit SHA: dc30b5847fcb72eb7d6e7d4461bd5b3a492fb66b

Patches
----------------------------------------
All patches applied

wrynose
=====================
Summary:
- Patch Series: [meta-ti][master/wrynose][PATCH v2] libsdl2: Build tests with dynamically linked library support
- Submitter: From: <a-christidis@ti.com>
From: Antonios Christidis <a-christidis@ti.com>
+From: Antonios Christidis <a-christidis@ti.com>
- Date: Date: Wed, 1 Jul 2026 13:29:48 -0500
+Date: Fri, 26 Jun 2026 13:28:24 -0500
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: e29cc0575fed64eb2e4f3f83a95bb2edd8506005

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: wrynose-wip
- Commit Author: Antonios Christidis <a-christidis@ti.com>
- Commit Subject: libsdl2: Set runtime dependency for SDL2 tests
- Commit SHA: 77289b29557f8cd8d0e85eb2ef78ef333d45cf71

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: FAIL
=========================================================
master
=====================
FAIL
    WARN: .bbappend files might need a guard to make them check-layer compliant. (GUARD-1)
        meta-ti-bsp/recipes-kernel/linux-firmware/linux-firmware_%.bbappend
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887

wrynose
=====================
FAIL
    WARN: .bbappend files might need a guard to make them check-layer compliant. (GUARD-1)
        meta-ti-bsp/recipes-kernel/linux-firmware/linux-firmware_%.bbappend
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
diff mbox series

Patch

diff --git a/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc b/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
index 33ff0632..3aa566c6 100644
--- a/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
+++ b/meta-ti-test/recipes-graphics/libsdl2/libsdl2-ti.inc
@@ -1,5 +1,13 @@ 
+FILESEXTRAPATHS:prepend := "${THISDIR}/libsdl2:"
+
+SRC_URI:append = " \
+    file://0001-test-New-SDL_TESTS_SHARED_LIB-option-for-dynamically.patch \
+"
+
 EXTRA_OECMAKE += "-DSDL_TESTS=ON \
                   -DSDL_INSTALL_TESTS=ON \
+                  -DSDL_TESTS_SHARED_LIB=ON \
                  "
+
 PACKAGE_BEFORE_PN = "${PN}-tests"
 FILES:${PN}-tests += "${libexecdir} ${datadir}/installed-tests/SDL2"
diff --git a/meta-ti-test/recipes-graphics/libsdl2/libsdl2/0001-test-New-SDL_TESTS_SHARED_LIB-option-for-dynamically.patch b/meta-ti-test/recipes-graphics/libsdl2/libsdl2/0001-test-New-SDL_TESTS_SHARED_LIB-option-for-dynamically.patch
new file mode 100644
index 00000000..234092a2
--- /dev/null
+++ b/meta-ti-test/recipes-graphics/libsdl2/libsdl2/0001-test-New-SDL_TESTS_SHARED_LIB-option-for-dynamically.patch
@@ -0,0 +1,56 @@ 
+From 96512412805f34ac6328290221c7ad83cde342b3 Mon Sep 17 00:00:00 2001
+From: Antonios Christidis <a-christidis@ti.com>
+Date: Fri, 26 Jun 2026 13:28:24 -0500
+Subject: [PATCH] test: New SDL_TESTS_SHARED_LIB option for dynamically linked
+ tests
+
+Introduce SDL_TESTS_SHARED_LIB to allow tests that expect dynamic linking
+to be built.
+
+Upstream-Status: Submitted [https://github.com/libsdl-org/SDL/pull/15907]
+
+Signed-off-by: Antonios Christidis <a-christidis@ti.com>
+---
+ test/CMakeLists.txt | 19 ++++++++++++++++---
+ 1 file changed, 16 insertions(+), 3 deletions(-)
+
+diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
+index 288645d73..8e5bd68d3 100644
+--- a/test/CMakeLists.txt
++++ b/test/CMakeLists.txt
+@@ -87,8 +87,17 @@ macro(add_sdl_test_executable TARGET)
+     endif()
+ endmacro()
+
+-if(NOT TARGET SDL2::SDL2-static)
+-    find_package(SDL2 2.0.23 REQUIRED COMPONENTS SDL2-static SDL2test)
++option(SDL_TESTS_SHARED_LIB "Link test executables against the shared SDL2 library" OFF)
++
++if(SDL_TESTS_SHARED_LIB)
++    set(SDL_SHARED ON)
++    if(NOT TARGET SDL2::SDL2)
++        find_package(SDL2 ${SDL_VERSION} REQUIRED COMPONENTS SDL2 SDL2test)
++    endif()
++else()
++    if(NOT TARGET SDL2::SDL2-static)
++        find_package(SDL2 ${SDL_VERSION} REQUIRED COMPONENTS SDL2-static SDL2test)
++    endif()
+ endif()
+
+ enable_testing()
+@@ -247,7 +256,11 @@ elseif(PS2)
+ elseif(IOS OR TVOS)
+     sdltest_link_librararies(SDL2::SDL2main SDL2::SDL2test SDL2::SDL2-static)
+ else()
+-    sdltest_link_librararies(SDL2::SDL2test SDL2::SDL2-static)
++    if(SDL_TESTS_SHARED_LIB)
++        sdltest_link_librararies(SDL2::SDL2test SDL2::SDL2)
++    else()
++        sdltest_link_librararies(SDL2::SDL2test SDL2::SDL2-static)
++    endif()
+ endif()
+
+ if(WINDOWS)
+--
+2.34.1
+