On Tue Jan 21, 2025 at 1:17 PM CST, Denys Dmytriyenko wrote:
> On Tue, Jan 21, 2025 at 12:54:39PM -0600, Andrew Davis via 
> lists.yoctoproject.org wrote:
> > On 1/21/25 9:16 AM, Ryan Eatmon via lists.yoctoproject.org wrote:
> > >
> > >
> > >On 1/20/2025 3:13 PM, Denys Dmytriyenko wrote:
> > >>On Thu, Jan 16, 2025 at 01:29:51PM -0600, Randolph Sapp via 
> > >>lists.yoctoproject.org wrote:
> > >>>From: Randolph Sapp <[email protected]>
> > >>>
> > >>>Remove any explicit dependencies on meta-arago-demos. These were the
> > >>>last of the "demos" in this layer and they were only being built for
> > >>>legacy devices. Demos should primarily be in layers above this unless
> > >>>they are used for functionality testing.
> > >>
> > >>Why not move those pieces to meta-arago-demos then?
> > >>
> > >>E.g. move the packagegroups there and add bbappends for the default image 
> > >>and
> > >>world recipes, etc.
> > >
> > >Basically make meta-arago-demos an island unto itself.  If you include it, 
> > >then you get it, if you don't then no harm.  Make sense.
> > >
> > 
> > Will we test with and without this layer? If not then we will end up 
> > breaking
> > things and missing dependencies between layers.
> > 
> > Seems like a lot of maintenence for a layer with just a couple demos. Demos
> > that should probably all go up into focus SDK layers anyway.
>
> Should meta-arago-demos be deprecated and removed in its entirety then? Or 
> does it only apply to a few select that are still in meta-arago-distro?

Yeah, current plan is to drop this layer entirely, just wanted to do it in a
sane order. I put a note of that in the tear-off section. Still waiting on these
patches to be merged before continuing though. Still got a few more patches for
reworking Qt dependencies and enabling Qt6.

> > >>>Signed-off-by: Randolph Sapp <[email protected]>
> > >>>---
> > >>>
> > >>>Trying to do this gracefully. Break the dependency first, then update the
> > >>>oe-layersetup configs, then drop the layer entirely.
> > >>>
> > >>>  .../images/tisdk-default-image.bb             | 14 +-------
> > >>>  ...kagegroup-arago-tisdk-addons-sdk-target.bb | 17 ---------
> > >>>  .../packagegroup-arago-tisdk-addons.bb        |  2 --
> > >>>  .../packagegroup-arago-tisdk-hmi.bb           | 36 -------------------
> > >>>  .../recipes-core/packagegroups/ti-demos.bb    | 18 ----------
> > >>>  .../recipes-core/packagegroups/ti-world.bb    | 10 +-----
> > >>>  6 files changed, 2 insertions(+), 95 deletions(-)
> > >>>  delete mode 100644 
> > >>>meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
> > >>>  delete mode 100644 
> > >>>meta-arago-distro/recipes-core/packagegroups/ti-demos.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 6c46c423..10493415 100644
> > >>>--- a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
> > >>>+++ b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
> > >>>@@ -23,24 +23,13 @@ IMAGE_INSTALL += "\
> > >>>      packagegroup-arago-tisdk-multimedia \
> > >>>      packagegroup-arago-tisdk-addons \
> > >>>      packagegroup-arago-tisdk-addons-extra \
> > >>>-    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-hmi','packagegroup-arago-base-tisdk-server-extra',d)}
> > >>> \
> > >>>-    ti-demos \
> > >>>+    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl','','packagegroup-arago-base-tisdk-server-extra',d)}
> > >>> \
> > >>>      ${ARAGO_DEFAULT_IMAGE_EXTRA_INSTALL} \
> > >>>      packagegroup-arago-tisdk-sysrepo \
> > >>>  "
> > >>>  export IMAGE_BASENAME = "tisdk-default-image${ARAGO_IMAGE_SUFFIX}"
> > >>>-EXTRABROWSERS = " \
> > >>>-    qtwebbrowser-examples \
> > >>>-    qtwebengine-qmlplugins \
> > >>>-    qtwebengine-examples \
> > >>>-"
> > >>>-
> > >>>-PYTHON2APPS = " \
> > >>>-    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl',"${EXTRABROWSERS}",'',d)}
> > >>> \
> > >>>-"
> > >>>-
> > >>>  DEVTOOLS = " \
> > >>>      linux-libc-headers-dev \
> > >>>      build-essential \
> > >>>@@ -50,7 +39,6 @@ DEVTOOLS = " \
> > >>>  "
> > >>>  IMAGE_INSTALL += "\
> > >>>-    ${@bb.utils.contains("BBFILE_COLLECTIONS", "meta-python2", 
> > >>>"${PYTHON2APPS}", "", d)} \
> > >>>      ${DEVTOOLS} \
> > >>>      ${@bb.utils.contains('TUNE_FEATURES', 'armv7a', 'valgrind', '', 
> > >>>d)} \
> > >>>      docker \
> > >>>diff --git 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
> > >>> 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
> > >>>index 3e0a2720..8690a874 100644
> > >>>--- 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
> > >>>+++ 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
> > >>>@@ -31,24 +31,7 @@ EXTRA_LIBS:append:dra7xx = " libulm-dev \
> > >>>                               gdbserver-c6x-dev \
> > >>>  "
> > >>>-PDM_ANOMALY_PKG_DEV = "\
> > >>>-    ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 
> > >>>'pdm-anomaly-detection-dev', '', d)} \
> > >>>-"
> > >>>-
> > >>>-
> > >>>  EXTRA_PACKAGES = ""
> > >>>-#EXTRA_PACKAGES:append:ti33x = " \
> > >>>-#                                ${PDM_ANOMALY_PKG_DEV} \
> > >>>-#"
> > >>>-#EXTRA_PACKAGES:append:ti43x = " \
> > >>>-#                                ${PDM_ANOMALY_PKG_DEV} \
> > >>>-#"
> > >>>-#EXTRA_PACKAGES:append:omap-a15 = " \
> > >>>-#                                ${PDM_ANOMALY_PKG_DEV} \
> > >>>-#"
> > >>>-#EXTRA_PACKAGES:append:am65xx = " \
> > >>>-#                             ${PDM_ANOMALY_PKG_DEV} \
> > >>>-#"
> > >>>  IPCDEV = " \
> > >>>      ti-ipc-dev \
> > >>>diff --git 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb
> > >>> 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb
> > >>>index 6b6533f5..b0d76b28 100644
> > >>>--- 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb
> > >>>+++ 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb
> > >>>@@ -85,10 +85,8 @@ EXTRA_PACKAGES:omapl138 = " \
> > >>>      protobuf \
> > >>>  "
> > >>>-# acontis-atemsys
> > >>>  #EXTRA_PACKAGES:append:am335x-evm = " pruss-lld-apps"
> > >>>  #EXTRA_PACKAGES:append:am437x-evm = " pruss-lld-apps"
> > >>>-# acontis-atemsys
> > >>>  #EXTRA_PACKAGES:append:am57xx-evm = " pruss-lld-apps"
> > >>>  RDEPENDS:${PN} = "\
> > >>>diff --git 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
> > >>> 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
> > >>>deleted file mode 100644
> > >>>index a8261bf2..00000000
> > >>>--- 
> > >>>a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
> > >>>+++ /dev/null
> > >>>@@ -1,36 +0,0 @@
> > >>>-SUMMARY = "Task to add HMI related packages"
> > >>>-LICENSE = "MIT"
> > >>>-PR = "r0"
> > >>>-
> > >>>-PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >>>-
> > >>>-inherit packagegroup
> > >>>-
> > >>>-HMI = " \
> > >>>-"
> > >>>-
> > >>>-
> > >>>-HMI:append:ti33x = " \
> > >>>-    evse-hmi \
> > >>>-    protection-relays-hmi \
> > >>>-    mmwavegesture-hmi \
> > >>>-"
> > >>>-
> > >>>-HMI:append:ti43x = " \
> > >>>-    evse-hmi \
> > >>>-    mmwavegesture-hmi \
> > >>>-"
> > >>>-
> > >>>-HMI:append:omap-a15 = " \
> > >>>-    evse-hmi \
> > >>>-    mmwavegesture-hmi \
> > >>>-"
> > >>>-
> > >>>-HMI:append:am65xx = " \
> > >>>-    evse-hmi \
> > >>>-    mmwavegesture-hmi \
> > >>>-"
> > >>>-
> > >>>-RDEPENDS:${PN} = "\
> > >>>-    ${HMI} \
> > >>>-"
> > >>>diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
> > >>>deleted file mode 100644
> > >>>index 7bccb4de..00000000
> > >>>--- a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
> > >>>+++ /dev/null
> > >>>@@ -1,18 +0,0 @@
> > >>>-SUMMARY = "TI World packagegroup"
> > >>>-LICENSE = "MIT"
> > >>>-
> > >>>-PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >>>-
> > >>>-inherit packagegroup
> > >>>-
> > >>>-PDM_ANOMALY_PKG = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 
> > >>>'pdm-anomaly-detection', '', d)}"
> > >>>-
> > >>>-PDM_ANOMALY = ""
> > >>>-PDM_ANOMALY:ti33x = "${PDM_ANOMALY_PKG}"
> > >>>-PDM_ANOMALY:ti43x = "${PDM_ANOMALY_PKG}"
> > >>>-PDM_ANOMALY:omap-a15 = "${PDM_ANOMALY_PKG}"
> > >>>-PDM_ANOMALY:am65xx = "${PDM_ANOMALY_PKG}"
> > >>>-
> > >>>-RDEPENDS:${PN} = "\
> > >>>-    ${PDM_ANOMALY} \
> > >>>-"
> > >>>diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-world.bb 
> > >>>b/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
> > >>>index 438dd8d0..fc3eac74 100644
> > >>>--- a/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
> > >>>+++ b/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
> > >>>@@ -15,14 +15,7 @@ CHROMIUM:append:k3 = "\
> > >>>      chromium-ozone-wayland \
> > >>>  "
> > >>>-EXTRABROWSERS = " \
> > >>>-    qtwebbrowser-examples \
> > >>>-    qtwebengine-qmlplugins \
> > >>>-    qtwebengine-examples \
> > >>>-"
> > >>>-
> > >>>  PYTHON2APPS = " \
> > >>>-    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl',"${EXTRABROWSERS}",'',d)}
> > >>> \
> > >>>      
> > >>>${@bb.utils.contains("BBFILE_COLLECTIONS","browser-layer",bb.utils.contains('DISTRO_FEATURES','wayland',"${CHROMIUM}",'',d),'',d)}
> > >>> \
> > >>>  "
> > >>>@@ -45,11 +38,10 @@ RDEPENDS:${PN} = "\
> > >>>      packagegroup-arago-tisdk-multimedia \
> > >>>      packagegroup-arago-tisdk-addons \
> > >>>      packagegroup-arago-tisdk-addons-extra \
> > >>>-    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-hmi','packagegroup-arago-base-tisdk-server-extra',d)}
> > >>> \
> > >>>+    
> > >>>${@bb.utils.contains('DISTRO_FEATURES','opengl','','packagegroup-arago-base-tisdk-server-extra',d)}
> > >>> \
> > >>>      ${@bb.utils.contains("BBFILE_COLLECTIONS", "meta-python2", 
> > >>>"${PYTHON2APPS}", "", d)} \
> > >>>      ${DEVTOOLS} \
> > >>>      ${@bb.utils.contains('TUNE_FEATURES', 'armv7a', 'valgrind', '', 
> > >>>d)} \
> > >>>      packagegroup-arago-misc \
> > >>>-    ti-demos \
> > >>>      ${PREFERRED_PROVIDER_virtual/docker} \
> > >>>  "
> > >>>-- 
> > >>>2.47.1



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15732): 
https://lists.yoctoproject.org/g/meta-arago/message/15732
Mute This Topic: https://lists.yoctoproject.org/mt/110653645/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to