diff mbox series

[meta-arago,master] meta-arago: break dependency on meta-arago-demos

Message ID 20250116192950.3391293-2-rs@ti.com
State New
Headers show
Series [meta-arago,master] meta-arago: break dependency on meta-arago-demos | expand

Commit Message

Randolph Sapp Jan. 16, 2025, 7:29 p.m. UTC
From: Randolph Sapp <rs@ti.com>

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.

Signed-off-by: Randolph Sapp <rs@ti.com>
---

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

Comments

Andrew Davis Jan. 17, 2025, 12:32 a.m. UTC | #1
On 1/16/25 1:29 PM, Randolph Sapp via lists.yoctoproject.org wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> 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.
> 
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
> 
> Trying to do this gracefully. Break the dependency first, then update the
> oe-layersetup configs, then drop the layer entirely.
> 

Acked-by: Andrew Davis <afd@ti.com>

>   .../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} \
>   "
Chirag Shilwant Jan. 19, 2025, 8:51 a.m. UTC | #2
On 17/01/25 00:59, rs@ti.com wrote:
> From: Randolph Sapp <rs@ti.com>
>
> 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.
>
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---

Acked-by: Chirag Shilwant <c-shilwant@ti.com>

>
> 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} \
>   "
Denys Dmytriyenko Jan. 20, 2025, 9:13 p.m. UTC | #3
On Thu, Jan 16, 2025 at 01:29:51PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> 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.


> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
> 
> 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
Ryan Eatmon Jan. 21, 2025, 3:16 p.m. UTC | #4
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 <rs@ti.com>
>>
>> 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.


> 
>> Signed-off-by: Randolph Sapp <rs@ti.com>
>> ---
>>
>> 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
Andrew Davis Jan. 21, 2025, 6:54 p.m. UTC | #5
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 <rs@ti.com>
>>>
>>> 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.

Andrew

> 
>>
>>> Signed-off-by: Randolph Sapp <rs@ti.com>
>>> ---
>>>
>>> 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
>
Denys Dmytriyenko Jan. 21, 2025, 7:17 p.m. UTC | #6
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 <rs@ti.com>
> >>>
> >>>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?


> >>>Signed-off-by: Randolph Sapp <rs@ti.com>
> >>>---
> >>>
> >>>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
Andrew Davis Jan. 21, 2025, 7:29 p.m. UTC | #7
On 1/21/25 1:17 PM, 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 <rs@ti.com>
>>>>>
>>>>> 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

Yes, and I thought that was already the plan after this patch [0]

Andrew

[0] https://lists.yoctoproject.org/g/meta-arago/message/15695

> does it only apply to a few select that are still in meta-arago-distro?
> 
> 
>>>>> Signed-off-by: Randolph Sapp <rs@ti.com>
>>>>> ---
>>>>>
>>>>> 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
Randolph Sapp Jan. 21, 2025, 8:33 p.m. UTC | #8
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 <rs@ti.com>
> > >>>
> > >>>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 <rs@ti.com>
> > >>>---
> > >>>
> > >>>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
diff mbox series

Patch

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} \
 "