| Message ID | 20250320182041.2349-7-reatmon@ti.com |
|---|---|
| State | Accepted |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | Wrap various packages in a compliant manner | expand |
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
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 = " \ "
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(-)