[meta-arago][master][PATCH 0/8] Wrap various packages in a compliant manner

[meta-arago][master][PATCH 0/8] Wrap various packages in a compliant manner

[Ryan Eatmon]

We are moving to make the meta-arago layers Yocto Project Compatible.

https://www.yoctoproject.org/development/yocto-project-compatible-layers/

We have been cleaning up the layer according to the yocto-check-layer
tool.  This series addresses any packages that were defined by wrapping
the bbappend behind a guard (see other series), or that a commercial
license is defined for a recipe and LICENSE_FLAGS_ACCEPTED is required
in the distro to be able to use them.

In order for a packagegroup to use these recipes, we can only include
them if we are building arago and applying the conf/distro/arago.conf
settings.

Ryan Eatmon (8):
  packagegroup-arago-base: Wrap various packages in a compliant manner
  packagegroup-arago-bootstrap: Wrap various packages in a compliant
    manner
  packagegroup-arago-tisdk-addons: Wrap various packages in a compliant
    manner
  nativesdk-packagegroup-arago-tisdk-host: Wrap various packages in a
    compliant manner
  ti-test: Wrap various packages in a compliant manner
  packagegroup-arago-gst-sdk-target: Wrap various packages in a
    compliant manner
  packagegroup-arago-gst: Wrap various packages in a compliant manner
  packagegroup-arago-tisdk-multimedia: Wrap various packages in a
    compliant manner

 .../recipes-core/packagegroups/packagegroup-arago-base.bb  | 5 ++++-
 .../packagegroups/packagegroup-arago-bootstrap.bb          | 5 ++++-
 .../packagegroups/packagegroup-arago-gst-sdk-target.bb     | 3 ++-
 .../recipes-core/packagegroups/packagegroup-arago-gst.bb   | 3 ++-
 .../packagegroups/packagegroup-arago-tisdk-addons.bb       | 7 +++++--
 .../packagegroups/packagegroup-arago-tisdk-multimedia.bb   | 4 ++--
 .../nativesdk-packagegroup-arago-tisdk-host.bb             | 5 ++++-
 meta-arago-test/recipes-core/packagegroups/ti-test.bb      | 2 +-
 8 files changed, 24 insertions(+), 10 deletions(-)

-- 
2.17.1

[meta-arago][master][PATCH 1/8] packagegroup-arago-base: Wrap various packages in a compliant manner

[Ryan Eatmon]

Previous clean ups wrapped some recipe .bbappend files in a compliant
manner and effectively hid the changes behind having "arago" in the
DISTRO_OVERRIDE.  This same kind wrapping is needed when including the
package in the package group.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../recipes-core/packagegroups/packagegroup-arago-base.bb    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base.bb
index 121fa432..fc2eafec 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base.bb
@@ -36,7 +36,6 @@ ARAGO_EXTRA = "\
     kms++-python \
     can-utils \
     docker \
-    dbus-broker \
     expat \
     glib-2.0 \
     libxml2 \
@@ -52,6 +51,10 @@ ARAGO_EXTRA = "\
     lldpd \
 "
 
+ARAGO_EXTRA:append:arago = "\
+    dbus-broker \
+"
+
 OPTEE_PKGS = " \
     optee-os \
     optee-client \
-- 
2.17.1

[meta-arago][master][PATCH 2/8] packagegroup-arago-bootstrap: Wrap various packages in a compliant manner

[Ryan Eatmon]

Previous clean ups wrapped some recipe .bbappend files in a compliant
manner and effectively hid the changes behind having "ti-soc" in the
MACHINE_OVERRIDES.  This same kind wrapping is needed when including the
package in the package group.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../packagegroups/packagegroup-arago-bootstrap.bb            | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
index 50f759be..3c05895a 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
@@ -19,7 +19,6 @@ UTILS = " \
 	 libdrm-tests \
 	 rt-tests \
 	 iozone3 \
-	 mtd-utils-ubifs-tests \
 	 net-tools \
 	 ethtool  \
 	 pciutils \
@@ -30,6 +29,10 @@ UTILS = " \
 	 devmem2 \
 	 phytool \
 "
+	 
+UTILS:append:ti-soc = " \
+    mtd-utils-ubifs-tests \
+"
 
 UTILS:append:ti33x = " \
     omapconf \
-- 
2.17.1

[meta-arago][master][PATCH 3/8] packagegroup-arago-tisdk-addons: Wrap various packages in a compliant manner

[Ryan Eatmon]

Previous clean ups wrapped some recipe .bbappend files in a compliant
manner and effectively hid the changes behind having "arago" in the
DISTRO_OVERRIDE.  This same kind wrapping is needed when including the
package in the package group.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../packagegroups/packagegroup-arago-tisdk-addons.bb       | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

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 00d68e78..0d39907a 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
@@ -15,12 +15,15 @@ UTILS = " \
     strongswan \
     kexec \
     kdump \
-    open62541-examples \
-    open62541-tests \
     sudo \
     watchdog \
 "
 
+UTILS:append:arago = " \
+    open62541-examples \
+    open62541-tests \
+"
+
 UTILS_UBOOT_FW = "libubootenv-bin"
 
 UTILS_DSP = " \
-- 
2.17.1

[meta-arago][master][PATCH 4/8] nativesdk-packagegroup-arago-tisdk-host: Wrap various packages in a compliant manner

[Ryan Eatmon]

Previous clean ups wrapped some recipe .bbappend files in a compliant
manner and effectively hid the changes behind having "arago" in the
DISTRO_OVERRIDE.  This same kind wrapping is needed when including the
package in the package group.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../packagegroups/nativesdk-packagegroup-arago-tisdk-host.bb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta-arago-extras/recipes-core/packagegroups/nativesdk-packagegroup-arago-tisdk-host.bb b/meta-arago-extras/recipes-core/packagegroups/nativesdk-packagegroup-arago-tisdk-host.bb
index 177fa668..b20bcb7e 100644
--- a/meta-arago-extras/recipes-core/packagegroups/nativesdk-packagegroup-arago-tisdk-host.bb
+++ b/meta-arago-extras/recipes-core/packagegroups/nativesdk-packagegroup-arago-tisdk-host.bb
@@ -10,9 +10,12 @@ PACKAGEGROUP_DISABLE_COMPLEMENTARY = "1"
 EXTRA_TI_TOOLS = " \
     nativesdk-ti-cgt6x \
     nativesdk-ti-cgt-pru \
+    nativesdk-gcc-arm-baremetal \
+"
+
+UTILS:append:arago = " \
     nativesdk-open62541-examples \
     nativesdk-open62541-tests \
-    nativesdk-gcc-arm-baremetal \
 "
 
 RDEPENDS:${PN} = "\
-- 
2.17.1

[meta-arago][master][PATCH 5/8] ti-test: Wrap various packages in a compliant manner

[Ryan Eatmon]

Previous clean ups wrapped some recipe .bbappend files in a compliant
manner and effectively hid the changes behind having "ti-soc" in the
MACHINE_OVERRIDES.  This same kind wrapping is needed when including the
package in the package group.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-arago-test/recipes-core/packagegroups/ti-test.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
index a5a8838f..a73e2869 100644
--- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
+++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
@@ -58,7 +58,7 @@ TI_TEST_BASE = "\
     yavta \
 "
 
-TI_TEST_BASE:ti-soc:append = " \
+TI_TEST_BASE:append:ti-soc = " \
     mtd-utils-ubifs-tests \
 "
 
-- 
2.17.1

[meta-arago][master][PATCH 6/8] packagegroup-arago-gst-sdk-target: Wrap various packages in a compliant manner

[Ryan Eatmon]

yocto-check-layer does not load the conf/distro/arago.conf file during
its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
set according to what we are willing to accept.

If a packagegroup is trying to include a recipe that is marked as being
"commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
fails.

The solution is to wrap the inclusion of these recipes in the packagegroup
in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
"arago" build, should the package be included and this under the direction
of the LICENSE_FLAGS_ACCEPTED variable.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../packagegroups/packagegroup-arago-gst-sdk-target.bb         | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
index fd749711..ad25f4dd 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
@@ -11,9 +11,10 @@ GSTREAMER_DEPS = " \
     gstreamer1.0-plugins-base-dev \
     gstreamer1.0-plugins-good-dev \
     gstreamer1.0-plugins-bad-dev \
-    gstreamer1.0-libav-dev \
 "
 
+GSTREAMER_DEPS:arago += "gstreamer1.0-libav-dev"
+
 #    gstreamer1.0-plugins-hevc-dev 
 GSTREAMER_DEPS:append:dra7xx = " \
 "
-- 
2.17.1

Re: [meta-arago][master][PATCH 6/8] packagegroup-arago-gst-sdk-target: Wrap various packages in a compliant manner

[Denys Dmytriyenko]

On Thu, Mar 20, 2025 at 01:20:39PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> yocto-check-layer does not load the conf/distro/arago.conf file during
> its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
> set according to what we are willing to accept.
> 
> If a packagegroup is trying to include a recipe that is marked as being
> "commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
> fails.
> 
> The solution is to wrap the inclusion of these recipes in the packagegroup
> in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
> "arago" build, should the package be included and this under the direction
> of the LICENSE_FLAGS_ACCEPTED variable.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../packagegroups/packagegroup-arago-gst-sdk-target.bb         | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> index fd749711..ad25f4dd 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> @@ -11,9 +11,10 @@ GSTREAMER_DEPS = " \
>      gstreamer1.0-plugins-base-dev \
>      gstreamer1.0-plugins-good-dev \
>      gstreamer1.0-plugins-bad-dev \
> -    gstreamer1.0-libav-dev \
>  "
>  
> +GSTREAMER_DEPS:arago += "gstreamer1.0-libav-dev"

It's not recommended to mix overrides (:arago) and appending operands (+=)

GSTREAMER_DEPS:append:arago = " gstreamer1.0-libav-dev"


> +
>  #    gstreamer1.0-plugins-hevc-dev 
>  GSTREAMER_DEPS:append:dra7xx = " \
>  "
> -- 
> 2.17.1

[meta-arago][master][PATCH 7/8] packagegroup-arago-gst: Wrap various packages in a compliant manner

[Ryan Eatmon]

yocto-check-layer does not load the conf/distro/arago.conf file during
its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
set according to what we are willing to accept.

If a packagegroup is trying to include a recipe that is marked as being
"commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
fails.

The solution is to wrap the inclusion of these recipes in the packagegroup
in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
"arago" build, should the package be included and this under the direction
of the LICENSE_FLAGS_ACCEPTED variable.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../recipes-core/packagegroups/packagegroup-arago-gst.bb       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
index 0aee4773..f5832109 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
@@ -14,9 +14,10 @@ GSTREAMER_DEPS = " \
     gstreamer1.0-plugins-base-meta \
     gstreamer1.0-plugins-good-meta \
     gstreamer1.0-plugins-bad-meta \
-    gstreamer1.0-libav \
 "
 
+GSTREAMER_DEPS:arago += "gstreamer1.0-libav"
+
 #    gstreamer1.0-plugins-hevc 
 GSTREAMER_DEPS:append:dra7xx = " \
 "
-- 
2.17.1

Re: [meta-arago][master][PATCH 7/8] packagegroup-arago-gst: Wrap various packages in a compliant manner

[Denys Dmytriyenko]

On Thu, Mar 20, 2025 at 01:20:40PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> yocto-check-layer does not load the conf/distro/arago.conf file during
> its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
> set according to what we are willing to accept.
> 
> If a packagegroup is trying to include a recipe that is marked as being
> "commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
> fails.
> 
> The solution is to wrap the inclusion of these recipes in the packagegroup
> in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
> "arago" build, should the package be included and this under the direction
> of the LICENSE_FLAGS_ACCEPTED variable.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../recipes-core/packagegroups/packagegroup-arago-gst.bb       | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> index 0aee4773..f5832109 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> @@ -14,9 +14,10 @@ GSTREAMER_DEPS = " \
>      gstreamer1.0-plugins-base-meta \
>      gstreamer1.0-plugins-good-meta \
>      gstreamer1.0-plugins-bad-meta \
> -    gstreamer1.0-libav \
>  "
>  
> +GSTREAMER_DEPS:arago += "gstreamer1.0-libav"

It's not recommended to mix overrides (:arago) and appending operands (+=)

GSTREAMER_DEPS:append:arago = " gstreamer1.0-libav"


>  #    gstreamer1.0-plugins-hevc 
>  GSTREAMER_DEPS:append:dra7xx = " \
>  "
> -- 
> 2.17.1

[meta-arago][master][PATCH 8/8] packagegroup-arago-tisdk-multimedia: Wrap various packages in a compliant manner

[Ryan Eatmon]

yocto-check-layer does not load the conf/distro/arago.conf file during
its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
set according to what we are willing to accept.

If a packagegroup is trying to include a recipe that is marked as being
"commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
fails.

The solution is to wrap the inclusion of these recipes in the packagegroup
in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
"arago" build, should the package be included and this under the direction
of the LICENSE_FLAGS_ACCEPTED variable.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../packagegroups/packagegroup-arago-tisdk-multimedia.bb      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
index 6a1b25b6..b7410e6f 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
@@ -7,12 +7,12 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
 inherit packagegroup
 
 MULTIMEDIA = " \
-    ffmpeg \
     libcamera \
     libcamera-gst \
-    mpv \
 "
 
+MULTIMEDIA:arago += "ffmpeg mpv"
+
 MULTIMEDIA:omapl138 = ""
 
 MULTIMEDIA:append:dra7xx = " \
-- 
2.17.1

Re: [meta-arago][master][PATCH 8/8] packagegroup-arago-tisdk-multimedia: Wrap various packages in a compliant manner

[Denys Dmytriyenko]

On Thu, Mar 20, 2025 at 01:20:41PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> yocto-check-layer does not load the conf/distro/arago.conf file during
> its execution.  This means that the LICENSE_FLAGS_ACCEPTED variable is not
> set according to what we are willing to accept.
> 
> If a packagegroup is trying to include a recipe that is marked as being
> "commercial" and the LICENSE_FLAGS_ACCEPTED is not set, then yocto-check-layer
> fails.
> 
> The solution is to wrap the inclusion of these recipes in the packagegroup
> in a complaint manner.  Only if the DISTRO_OVERRIDE says that this is an
> "arago" build, should the package be included and this under the direction
> of the LICENSE_FLAGS_ACCEPTED variable.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../packagegroups/packagegroup-arago-tisdk-multimedia.bb      | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
> index 6a1b25b6..b7410e6f 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
> @@ -7,12 +7,12 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
>  inherit packagegroup
>  
>  MULTIMEDIA = " \
> -    ffmpeg \
>      libcamera \
>      libcamera-gst \
> -    mpv \
>  "
>  
> +MULTIMEDIA:arago += "ffmpeg mpv"

It's not recommended to mix overrides (:arago) and appending operands (+=)

MULTIMEDIA:append:arago = " ffmpeg mpv"


>  MULTIMEDIA:omapl138 = ""
>  
>  MULTIMEDIA:append:dra7xx = " \
> -- 
> 2.17.1