| Message ID | 20260225190339.3295692-1-b-brnich@ti.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-arago,master] ti-test: Add Required Multimedia Packages to ti-test | expand |
meta-arago / na / 20260225190339.3295692-1-b-brnich PRC Results: FAIL ========================================================= check-yocto-patches: PASS ========================================================= Patches ---------------------------------------- All patches passed ========================================================= apply-yocto-patch: PASS ========================================================= master ===================== Summary: - Patch Series: [meta-arago][master][PATCH] ti-test: Add Required Multimedia Packages to ti-test - Submitter: From: Brandon Brnich <b-brnich@ti.com> - Date: Date: Wed, 25 Feb 2026 13:03:39 -0600 - Num Patches: 1 - Mailing List (public inbox) Commit SHA: 4fe33ef16607e8b6e93423d85c1b95b250df14e4 Applied to: - Repository: lcpd-prc-meta-arago - Base Branch: master-wip - Commit Author: Ryan Eatmon <reatmon@ti.com> - Commit Subject: arago.conf: Remove duplicate include for yocto-space-optimize.inc - Commit SHA: a69841493f652f427ee322e394fb3a040b873193 Patches ---------------------------------------- All patches applied ========================================================= check-yocto-repo: PASS ========================================================= master ===================== PASS ========================================================= yocto-check-layers: FAIL ========================================================= master - FAIL ===================== ERROR: Nothing RPROVIDES 'ffmpeg' (but <build-dir>/sources/meta-arago/meta-arago-test/recipes-core/packagegroups/ti-test.bb RDEPENDS on or otherwise requires it) ERROR: Required build target 'meta-world-pkgdata' has no buildable providers.
On Wed, Feb 25, 2026 at 01:03:39PM -0600, Brandon Brnich via lists.yoctoproject.org wrote: > FFMPEG and MPV are required during automated testing. Explicitly add > these to ti-test. > > Signed-off-by: Brandon Brnich <b-brnich@ti.com> > --- > meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- > 1 file changed, 3 insertions(+), 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 f2ae1aef..c694545d 100644 > --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > @@ -16,6 +16,7 @@ TI_TEST_BASE = "\ > dosfstools \ > ethtool \ > evtest \ > + ffmpeg \ > fio \ > git \ > gst-devtools \ > @@ -36,6 +37,7 @@ TI_TEST_BASE = "\ > lsof \ > media-ctl \ > memtester \ > + mpv \ > mstpd \ > mtd-utils \ > mtd-utils-ubifs \ > @@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ > " > > # Disable due to breakage > -# viddec-test-app > +# viddec-test-app Was there a meaningful change here? > TI_TEST_TI_TOOLS:append:j721e = " \ > pru-icss \ > ufs-utils \ > -- > 2.43.0
On 2/25/26 13:57, Denys Dmytriyenko wrote: > On Wed, Feb 25, 2026 at 01:03:39PM -0600, Brandon Brnich via lists.yoctoproject.org wrote: >> FFMPEG and MPV are required during automated testing. Explicitly add >> these to ti-test. >> >> Signed-off-by: Brandon Brnich <b-brnich@ti.com> >> --- >> meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- >> 1 file changed, 3 insertions(+), 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 f2ae1aef..c694545d 100644 >> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> @@ -16,6 +16,7 @@ TI_TEST_BASE = "\ >> dosfstools \ >> ethtool \ >> evtest \ >> + ffmpeg \ >> fio \ >> git \ >> gst-devtools \ >> @@ -36,6 +37,7 @@ TI_TEST_BASE = "\ >> lsof \ >> media-ctl \ >> memtester \ >> + mpv \ >> mstpd \ >> mtd-utils \ >> mtd-utils-ubifs \ >> @@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ >> " >> >> # Disable due to breakage >> -# viddec-test-app >> +# viddec-test-app > > Was there a meaningful change here? not really sure why it added the lines above because i didn't touch those files on my end.. > > >> TI_TEST_TI_TOOLS:append:j721e = " \ >> pru-icss \ >> ufs-utils \ >> -- >> 2.43.0
On 2/25/2026 1:03 PM, Brandon Brnich wrote: > FFMPEG and MPV are required during automated testing. Explicitly add > these to ti-test. > > Signed-off-by: Brandon Brnich <b-brnich@ti.com> > --- > meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- > 1 file changed, 3 insertions(+), 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 f2ae1aef..c694545d 100644 > --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > @@ -16,6 +16,7 @@ TI_TEST_BASE = "\ > dosfstools \ > ethtool \ > evtest \ > + ffmpeg \ This might be a problem... there are licensing issues related to ffmpeg that we address in meta-arago-distro, but moving this to meta-arago-test (a layer that *should* be able to be stand alone) is going to cause issues. Denys, Any ideas on how to address this? > fio \ > git \ > gst-devtools \ > @@ -36,6 +37,7 @@ TI_TEST_BASE = "\ > lsof \ > media-ctl \ > memtester \ > + mpv \ > mstpd \ > mtd-utils \ > mtd-utils-ubifs \ > @@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ > " > > # Disable due to breakage > -# viddec-test-app > +# viddec-test-app > TI_TEST_TI_TOOLS:append:j721e = " \ > pru-icss \ > ufs-utils \
On 2/25/26 1:58 PM, Brandon Brnich via lists.yoctoproject.org wrote: > > > On 2/25/26 13:57, Denys Dmytriyenko wrote: >> On Wed, Feb 25, 2026 at 01:03:39PM -0600, Brandon Brnich via lists.yoctoproject.org wrote: >>> FFMPEG and MPV are required during automated testing. Explicitly add >>> these to ti-test. >>> >>> Signed-off-by: Brandon Brnich <b-brnich@ti.com> >>> --- >>> meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- >>> 1 file changed, 3 insertions(+), 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 f2ae1aef..c694545d 100644 >>> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb >>> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >>> @@ -16,6 +16,7 @@ TI_TEST_BASE = "\ >>> dosfstools \ >>> ethtool \ >>> evtest \ >>> + ffmpeg \ >>> fio \ >>> git \ >>> gst-devtools \ >>> @@ -36,6 +37,7 @@ TI_TEST_BASE = "\ >>> lsof \ >>> media-ctl \ >>> memtester \ >>> + mpv \ >>> mstpd \ >>> mtd-utils \ >>> mtd-utils-ubifs \ >>> @@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ >>> " >>> # Disable due to breakage >>> -# viddec-test-app >>> +# viddec-test-app >> >> Was there a meaningful change here? > > not really sure why it added the lines above because i didn't touch those files on my end.. > Only change looks to be removing the trailing space, your editor must have (correctly) removed it while saving the file. Andrew >> >> >>> TI_TEST_TI_TOOLS:append:j721e = " \ >>> pru-icss \ >>> ufs-utils \ >>> -- >>> 2.43.0 > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#17148): https://lists.yoctoproject.org/g/meta-arago/message/17148 > Mute This Topic: https://lists.yoctoproject.org/mt/117999946/3619733 > Group Owner: meta-arago+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [afd@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On Wed, Feb 25, 2026 at 02:04:30PM -0600, Ryan Eatmon via lists.yoctoproject.org wrote: > > > On 2/25/2026 1:03 PM, Brandon Brnich wrote: > >FFMPEG and MPV are required during automated testing. Explicitly add > >these to ti-test. > > > >Signed-off-by: Brandon Brnich <b-brnich@ti.com> > >--- > > meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- > > 1 file changed, 3 insertions(+), 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 f2ae1aef..c694545d 100644 > >--- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > >+++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > >@@ -16,6 +16,7 @@ TI_TEST_BASE = "\ > > dosfstools \ > > ethtool \ > > evtest \ > >+ ffmpeg \ > > This might be a problem... there are licensing issues related to > ffmpeg that we address in meta-arago-distro, but moving this to > meta-arago-test (a layer that *should* be able to be stand alone) is > going to cause issues. > > Denys, > > Any ideas on how to address this? So, we already explicitly include ffmpeg and mpv in the multimedia packagegroup in meta-arago-distro. Since ffmpeg and libav may include some codecs with questionable patent statuses, they are marked as "commercial" and need to be allowed explicitly, which we do in arago.conf with the corresponding LICENSE_FLAGS_ACCEPTED. That simply allows building those packages, but whether the corresponding codecs are enabled or not is determined by their PACKAGECONFIGs. https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/arago.conf#n149 https://docs.yoctoproject.org/dev-manual/licenses.html#enabling-commercially-licensed-recipes And I'd think for meta-arago-test layer it would be even easier, as it is not expected to be used for production. And we can set LICENSE_FLAGS_ACCEPTED in its layer.conf to make it work with other Distros besides Arago. > > fio \ > > git \ > > gst-devtools \ > >@@ -36,6 +37,7 @@ TI_TEST_BASE = "\ > > lsof \ > > media-ctl \ > > memtester \ > >+ mpv \ > > mstpd \ > > mtd-utils \ > > mtd-utils-ubifs \ > >@@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ > > " > > # Disable due to breakage > >-# viddec-test-app > >+# viddec-test-app > > TI_TEST_TI_TOOLS:append:j721e = " \ > > pru-icss \ > > ufs-utils \
diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb index f2ae1aef..c694545d 100644 --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb @@ -16,6 +16,7 @@ TI_TEST_BASE = "\ dosfstools \ ethtool \ evtest \ + ffmpeg \ fio \ git \ gst-devtools \ @@ -36,6 +37,7 @@ TI_TEST_BASE = "\ lsof \ media-ctl \ memtester \ + mpv \ mstpd \ mtd-utils \ mtd-utils-ubifs \ @@ -145,7 +147,7 @@ TI_TEST_TI_TOOLS:append:am65xx = " \ " # Disable due to breakage -# viddec-test-app +# viddec-test-app TI_TEST_TI_TOOLS:append:j721e = " \ pru-icss \ ufs-utils \
FFMPEG and MPV are required during automated testing. Explicitly add these to ti-test. Signed-off-by: Brandon Brnich <b-brnich@ti.com> --- meta-arago-test/recipes-core/packagegroups/ti-test.bb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)