Re: [meta-arago][master/wrynose][PATCH v2] chromium: restrict force-h264-youtube extension to supported boards

[Ryan Eatmon <reatmon@ti.com>]

On 9/11/2026 1:39 PM, Thorsten Lannynd wrote:
> Split force-h264-youtube into its own package, enabled only on boards that support
> H264 YouTube hardware accelerated playback.
> 
> Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
> ---
> 
> v2:
> - Shorten commit message title
> 
>   .../chromium/chromium-ozone-wayland-arago.inc | 29 ++++++++++++++-----
>   1 file changed, 22 insertions(+), 7 deletions(-)
> 
> diff --git a/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland-arago.inc b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland-arago.inc
> index d8eb0204..108f77c7 100644
> --- a/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland-arago.inc
> +++ b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland-arago.inc
> @@ -1,6 +1,6 @@
>   PR:append = ".arago0"
>   
> -EXTENSION_DIR = "/usr/lib/chromium/extensions/force-h264-youtube"
> +EXTENSION_DIR = "${libdir}/chromium/extensions/force-h264-youtube"
>   
>   FILESEXTRAPATHS:prepend := "${THISDIR}/chromium-ozone-wayland:"
>   
> @@ -15,11 +15,26 @@ SRC_URI:append = " \
>                     file://force-h264-youtube/manifest.json \
>                     "
>   
> +PACKAGES:append = " ${PN}-force-h264-youtube"
> +FILES:${PN}-force-h264-youtube = "${EXTENSION_DIR}"
> +RDEPENDS:${PN}-force-h264-youtube = "${PN}"
> +
> +CHROMIUM_H264_EXT ?= ""
> +CHROMIUM_H264_EXT:am62pxx-evm = "1"
> +CHROMIUM_H264_EXT:j721s2-evm  = "1"
> +CHROMIUM_H264_EXT:j722s-evm   = "1"
> +CHROMIUM_H264_EXT:j742s2-evm  = "1"
> +CHROMIUM_H264_EXT:j784s4-evm  = "1"

Would it be better to create a MACHINE_FEATURES for h264 and then have 
this just key off that.  Then you could send a second patch to 
meta-ti-bsp that adds that feature?

Denys, thoughts?


> +RRECOMMENDS:${PN}:append = " ${@bb.utils.contains('CHROMIUM_H264_EXT', '1', '${PN}-force-h264-youtube', '', d)}"
> +
>   do_install:append() {
> -    install -d ${D}${EXTENSION_DIR}
> -    install -m 0644 ${WORKDIR}/sources/force-h264-youtube/force_h264.js \
> -                    ${WORKDIR}/sources/force-h264-youtube/manifest.json \
> -                    ${D}${EXTENSION_DIR}/
> +    if [ -n "${CHROMIUM_H264_EXT}" ]; then
> +        install -d ${D}${EXTENSION_DIR}
> +        install -m 0644 ${WORKDIR}/sources/force-h264-youtube/force_h264.js \
> +                        ${WORKDIR}/sources/force-h264-youtube/manifest.json \
> +                        ${D}${EXTENSION_DIR}/
> +        sed -i "s|CHROME_EXTRA_ARGS=\"\(.*\)\"|CHROME_EXTRA_ARGS=\"\1 --load-extension=${EXTENSION_DIR}\"|" \
> +            ${D}${libdir}/chromium/chromium-wrapper
> +    fi
>   }
> -
> -FILES:${PN}:append = " ${EXTENSION_DIR}"

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS