* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-16 19:29 [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos rs
@ 2025-01-17 0:32 ` Andrew Davis
2025-01-19 8:51 ` Chirag Shilwant
2025-01-20 21:13 ` Denys Dmytriyenko
2 siblings, 0 replies; 12+ messages in thread
From: Andrew Davis @ 2025-01-17 0:32 UTC (permalink / raw)
To: rs, reatmon, denis, c-shilwant; +Cc: meta-arago
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} \
> "
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-16 19:29 [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos rs
2025-01-17 0:32 ` Andrew Davis
@ 2025-01-19 8:51 ` Chirag Shilwant
2025-01-20 21:13 ` Denys Dmytriyenko
2 siblings, 0 replies; 12+ messages in thread
From: Chirag Shilwant @ 2025-01-19 8:51 UTC (permalink / raw)
To: meta-arago; +Cc: denis, reatmon, rs
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} \
> "
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-16 19:29 [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos rs
2025-01-17 0:32 ` Andrew Davis
2025-01-19 8:51 ` Chirag Shilwant
@ 2025-01-20 21:13 ` Denys Dmytriyenko
2025-01-21 15:16 ` Ryan Eatmon
2 siblings, 1 reply; 12+ messages in thread
From: Denys Dmytriyenko @ 2025-01-20 21:13 UTC (permalink / raw)
To: rs; +Cc: reatmon, c-shilwant, meta-arago
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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-20 21:13 ` Denys Dmytriyenko
@ 2025-01-21 15:16 ` Ryan Eatmon
2025-01-21 18:54 ` Andrew Davis
0 siblings, 1 reply; 12+ messages in thread
From: Ryan Eatmon @ 2025-01-21 15:16 UTC (permalink / raw)
To: Denys Dmytriyenko, rs; +Cc: c-shilwant, meta-arago
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
--
Ryan Eatmon reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-21 15:16 ` Ryan Eatmon
@ 2025-01-21 18:54 ` Andrew Davis
2025-01-21 19:17 ` Denys Dmytriyenko
0 siblings, 1 reply; 12+ messages in thread
From: Andrew Davis @ 2025-01-21 18:54 UTC (permalink / raw)
To: reatmon, Denys Dmytriyenko, rs; +Cc: c-shilwant, meta-arago
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
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-21 18:54 ` Andrew Davis
@ 2025-01-21 19:17 ` Denys Dmytriyenko
2025-01-21 19:29 ` Andrew Davis
2025-01-21 20:33 ` Randolph Sapp
0 siblings, 2 replies; 12+ messages in thread
From: Denys Dmytriyenko @ 2025-01-21 19:17 UTC (permalink / raw)
To: afd; +Cc: reatmon, rs, c-shilwant, meta-arago
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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-21 19:17 ` Denys Dmytriyenko
@ 2025-01-21 19:29 ` Andrew Davis
2025-01-21 20:33 ` Randolph Sapp
1 sibling, 0 replies; 12+ messages in thread
From: Andrew Davis @ 2025-01-21 19:29 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: reatmon, rs, c-shilwant, meta-arago
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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-21 19:17 ` Denys Dmytriyenko
2025-01-21 19:29 ` Andrew Davis
@ 2025-01-21 20:33 ` Randolph Sapp
2025-01-22 22:51 ` Denys Dmytriyenko
1 sibling, 1 reply; 12+ messages in thread
From: Randolph Sapp @ 2025-01-21 20:33 UTC (permalink / raw)
To: Denys Dmytriyenko, afd; +Cc: reatmon, c-shilwant, meta-arago
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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-21 20:33 ` Randolph Sapp
@ 2025-01-22 22:51 ` Denys Dmytriyenko
2025-01-22 23:12 ` Randolph Sapp
0 siblings, 1 reply; 12+ messages in thread
From: Denys Dmytriyenko @ 2025-01-22 22:51 UTC (permalink / raw)
To: Randolph Sapp; +Cc: afd, reatmon, c-shilwant, meta-arago
On Tue, Jan 21, 2025 at 02:33:12PM -0600, Randolph Sapp wrote:
> 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.
I would like to request better commit messages for both of these patches -
this one and the one Andrew linked for the config change.
I do appreciate the difficulty with cross-dependency of thses changes - first
you update meta-arago-distro to not depend on meta-arago-demos, then you need
to remove meta-arago-demos references from the layer setup configs, and only
then you can finally remove meta-arago-demos layer itself, if that's the plan.
But the first 2 patches do not paint a bigger picture, specifically not
explaining the actual reason behind these very invasive changes.
I do know about the Qt6 plans and you briefly mention it here later in the
discussion. But for anyone outside the internal discussions it is not clear
what is going on and why. Moreover, is the plan to have any Qt6 demos and
re-use the meta-arago-demos layer? Or is the plan to not have any demos and
relegate that to downstream SDK layers, if nedded?
Thanks.
> > > >>>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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-22 22:51 ` Denys Dmytriyenko
@ 2025-01-22 23:12 ` Randolph Sapp
2025-01-23 21:11 ` Denys Dmytriyenko
0 siblings, 1 reply; 12+ messages in thread
From: Randolph Sapp @ 2025-01-22 23:12 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: afd, reatmon, c-shilwant, meta-arago
On Wed Jan 22, 2025 at 4:51 PM CST, Denys Dmytriyenko wrote:
> On Tue, Jan 21, 2025 at 02:33:12PM -0600, Randolph Sapp wrote:
> > 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.
>
> I would like to request better commit messages for both of these patches -
> this one and the one Andrew linked for the config change.
Noted.
> I do appreciate the difficulty with cross-dependency of thses changes - first
> you update meta-arago-distro to not depend on meta-arago-demos, then you need
> to remove meta-arago-demos references from the layer setup configs, and only
> then you can finally remove meta-arago-demos layer itself, if that's the plan.
Funny thing about that, meta-arago-demos was created without properly creating a
dependency in the meta-arago-distro to track it. The rest of this message is
outlined in the tear-off note, almost word for word.
> But the first 2 patches do not paint a bigger picture, specifically not
> explaining the actual reason behind these very invasive changes.
This was discussed in the Webex Yocto channel and 2 other email threads. I could
have sent out a message on the mailing list but it would have arrived 5 minutes
before this patch because I was already testing things back in December.
> I do know about the Qt6 plans and you briefly mention it here later in the
> discussion. But for anyone outside the internal discussions it is not clear
> what is going on and why. Moreover, is the plan to have any Qt6 demos and
> re-use the meta-arago-demos layer? Or is the plan to not have any demos and
> relegate that to downstream SDK layers, if nedded?
>
> Thanks.
The plan is to not have any demos that are not strictly used for testing in this
repo. Downstream SDK layers will be reworking their existing demos for Qt6,
picking these up if they are applicable (but nobody seemed to care about these
demos in particular).
The Qt6 stuff is somewhat new. First week of December was when a decision was
made and an email was sent out shortly after announcing that. I seem to have
incorrectly assumed you guys were in the loop, my bad.
> > > > >>>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
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [meta-arago][master][PATCH] meta-arago: break dependency on meta-arago-demos
2025-01-22 23:12 ` Randolph Sapp
@ 2025-01-23 21:11 ` Denys Dmytriyenko
0 siblings, 0 replies; 12+ messages in thread
From: Denys Dmytriyenko @ 2025-01-23 21:11 UTC (permalink / raw)
To: Randolph Sapp; +Cc: afd, reatmon, c-shilwant, meta-arago
On Wed, Jan 22, 2025 at 05:12:23PM -0600, Randolph Sapp wrote:
> On Wed Jan 22, 2025 at 4:51 PM CST, Denys Dmytriyenko wrote:
> > On Tue, Jan 21, 2025 at 02:33:12PM -0600, Randolph Sapp wrote:
> > > 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.
> >
> > I would like to request better commit messages for both of these patches -
> > this one and the one Andrew linked for the config change.
>
> Noted.
>
> > I do appreciate the difficulty with cross-dependency of thses changes - first
> > you update meta-arago-distro to not depend on meta-arago-demos, then you need
> > to remove meta-arago-demos references from the layer setup configs, and only
> > then you can finally remove meta-arago-demos layer itself, if that's the plan.
>
> Funny thing about that, meta-arago-demos was created without properly creating a
> dependency in the meta-arago-distro to track it.
Ah, yeah, that was my oversight - I take full responsibility fot that. At the
time we were cleaning meta-arago-extras and all the demos were extracted from
that layer, but I totally missed updating meta-arago-distro:
https://git.yoctoproject.org/meta-arago/commit/?id=ff0d17778adbccda2d97b206976b45fe199c970e
> > But the first 2 patches do not paint a bigger picture, specifically not
> > explaining the actual reason behind these very invasive changes.
>
> This was discussed in the Webex Yocto channel and 2 other email threads. I could
> have sent out a message on the mailing list but it would have arrived 5 minutes
> before this patch because I was already testing things back in December.
>
> > I do know about the Qt6 plans and you briefly mention it here later in the
> > discussion. But for anyone outside the internal discussions it is not clear
> > what is going on and why. Moreover, is the plan to have any Qt6 demos and
> > re-use the meta-arago-demos layer? Or is the plan to not have any demos and
> > relegate that to downstream SDK layers, if nedded?
> >
> > Thanks.
>
> The plan is to not have any demos that are not strictly used for testing in this
> repo. Downstream SDK layers will be reworking their existing demos for Qt6,
> picking these up if they are applicable (but nobody seemed to care about these
> demos in particular).
>
> The Qt6 stuff is somewhat new. First week of December was when a decision was
> made and an email was sent out shortly after announcing that. I seem to have
> incorrectly assumed you guys were in the loop, my bad.
I believe there was a slight misunderstanding here. I do have some internal
insight, but what I was looking for and asking for here, is to have a commit
message that would clearly explain for any outsider what are the high level
plans for these changes and why they are being made. A concise version of the
above 2 paragraphs w/o referring to any internal communications... :)
Both of the patches mostly say "breaking the dependency", which is a bit odd
reason for completely removing the entire layer and all the demos.
--
Denys
^ permalink raw reply [flat|nested] 12+ messages in thread