Re: [meta-arago][master/kirkstone][PATCH] ti-test-extras: add package group for large tests

[Ryan Eatmon via lists.yoctoproject.org]

On 6/15/2023 12:45 PM, Denys Dmytriyenko wrote:
On Thu, Jun 15, 2023 at 12:28:07PM -0500, r...@ti.com wrote:
From: Randolph Sapp <r...@ti.com>

This is required for images that expect ti-test to be installed but
actually use a ramdisk for primary storage. Piglit and the corresponding
cts tests are the real problems at the moment so we're moving those
first. Other tests may follow.

Why not make it a single packagegroup recipe producing 2 packages? Other
packagegroups already doing it - tisdk-matrix, tisdk-addons, tisdk-opencl.

In the longer term, I want to move to this.  But this is an "emergency" 
patch to try and fix a test issue for the CICD flow.  I'll migrate this 
to that new style after we get a promotion.


Signed-off-by: Randolph Sapp <r...@ti.com>
---
  .../recipes-core/images/tisdk-default-image.bb   |  1 +
  .../recipes-core/packagegroups/ti-test-extras.bb | 16 ++++++++++++++++
  .../recipes-core/packagegroups/ti-test.bb        |  3 ---
  3 files changed, 17 insertions(+), 3 deletions(-)
  create mode 100644 
meta-arago-distro/recipes-core/packagegroups/ti-test-extras.bb

diff --git a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb 
b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
index 3b9747c1..bb825e09 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
@@ -12,6 +12,7 @@ IMAGE_INSTALL += "\
      packagegroup-arago-console \
      packagegroup-arago-base-tisdk \
      ti-test \
+    ti-test-extras \
      
${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-graphics','',d)}
 \
      
${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-gtk','',d)}
 \
      
${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-qte','',d)}
 \
diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-test-extras.bb 
b/meta-arago-distro/recipes-core/packagegroups/ti-test-extras.bb
new file mode 100644
index 00000000..6d69b165
--- /dev/null
+++ b/meta-arago-distro/recipes-core/packagegroups/ti-test-extras.bb
@@ -0,0 +1,16 @@
+SUMMARY = "TI Testing packagegroup for big tests"
+LICENSE = "MIT"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+inherit packagegroup
+
+ARAGO_TEST_EXTRAS = "\
+    piglit \
+    ${@bb.utils.contains("DISTRO_FEATURES", "opengl", "opengl-es-cts", "", d)} 
\
+    ${@bb.utils.contains("DISTRO_FEATURES", "vulkan", "vulkan-cts", "", d)} \
+"
+
+RDEPENDS:${PN} = "\
+    ${ARAGO_TEST_EXTRAS} \
+"
diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-test.bb 
b/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
index 210bc5a9..3e45077b 100644
--- a/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
@@ -38,9 +38,6 @@ ARAGO_TEST = "\
      nbench-byte \
      stream \
      cryptodev-tests \
-    piglit \
-    ${@bb.utils.contains("DISTRO_FEATURES", "opengl", "opengl-es-cts", "", d)} 
\
-    ${@bb.utils.contains("DISTRO_FEATURES", "vulkan", "vulkan-cts", "", d)} \
  "
  
  ARAGO_TEST:append:armv7a = " \
--
2.41.0

--
Ryan Eatmon                reat...@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14638): 
https://lists.yoctoproject.org/g/meta-arago/message/14638
Mute This Topic: https://lists.yoctoproject.org/mt/99554005/21656
Group Owner: meta-arago+ow...@lists.yoctoproject.org
Unsubscribe: 
https://lists.yoctoproject.org/g/meta-arago/leave/10763299/21656/89520264/xyzzy 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-