| Message ID | 20260911182432.526086-1-t-lannynd@ti.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-arago,master/wrynose] meta-arago-disto: chromium: restrict force-h264-youtube extension to supported boards | expand |
meta-arago / na / 20260911182432.526086-1-t-lannynd
PRC Results: FAIL
=========================================================
check-yocto-patches: FAIL
=========================================================
Patches
----------------------------------------
FAIL - [meta-arago][master/wrynose][PATCH] meta-arago-disto: chromium: restrict force-h264-youtube extension to supported boards
ERROR: Commit message too long (> 80 chars): meta-arago-disto: chromium: restrict force-h264-youtube extension to supported boards. (COMMIT-MESSAGE-1)
patch:84
Subject: [meta-arago][master/wrynose][PATCH] meta-arago-disto: chromium: restrict force-h264-youtube extension to supported boards
For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
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" + +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}"
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> --- .../chromium/chromium-ozone-wayland-arago.inc | 29 ++++++++++++++----- 1 file changed, 22 insertions(+), 7 deletions(-)