diff mbox series

[meta-arago,master/wrynose,v7,1/2] chromium: auto-load arago extensions at runtime

Message ID 20260930160338.951001-1-t-lannynd@ti.com
State New
Headers show
Series [meta-arago,master/wrynose,v7,1/2] chromium: auto-load arago extensions at runtime | expand

Commit Message

Thorsten Lannynd Sept. 30, 2026, 4:03 p.m. UTC 
Scan $HERE/extensions/arago/ at runtime and pass any installed
subdirectories to Chromium via --load-extension.

Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
---
v7:
- Split generic chromium extension loader and h264 youtube extension into
  two seperate patches

v6:
- Shorten commit message title

v5:
- Use generic approach to loading extensions by creating arago subdirectory
  and loading all extensions that live in the subdirectory

v4:
- Add branch and layer to subject line

v3:
- Separate the extension into its own package and install on boards with
  Wave5 firmware
- Enable extension if extension directory exists

v2:
- Shorten commit message title

 .../chromium/chromium-ozone-wayland-arago.inc | 14 +-------
 ...-Auto-load-arago-extensions-at-runti.patch | 34 +++++++++++++++++++
 2 files changed, 35 insertions(+), 13 deletions(-)
 create mode 100644 meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch

Comments

Ryan Eatmon Sept. 30, 2026, 4:06 p.m. UTC | #1 
On 9/30/2026 11:03 AM, Thorsten Lannynd wrote:
> Scan $HERE/extensions/arago/ at runtime and pass any installed
> subdirectories to Chromium via --load-extension.
> 
> Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
> ---
> v7:
> - Split generic chromium extension loader and h264 youtube extension into
>    two seperate patches
> 
> v6:
> - Shorten commit message title
> 
> v5:
> - Use generic approach to loading extensions by creating arago subdirectory
>    and loading all extensions that live in the subdirectory
> 
> v4:
> - Add branch and layer to subject line
> 
> v3:
> - Separate the extension into its own package and install on boards with
>    Wave5 firmware
> - Enable extension if extension directory exists
> 
> v2:
> - Shorten commit message title
> 
>   .../chromium/chromium-ozone-wayland-arago.inc | 14 +-------
>   ...-Auto-load-arago-extensions-at-runti.patch | 34 +++++++++++++++++++
>   2 files changed, 35 insertions(+), 13 deletions(-)
>   create mode 100644 meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch
> 
> 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..40b74c18 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,7 +1,5 @@
>   PR:append = ".arago0"
>   
> -EXTENSION_DIR = "/usr/lib/chromium/extensions/force-h264-youtube"
> -
>   FILESEXTRAPATHS:prepend := "${THISDIR}/chromium-ozone-wayland:"
>   
>   PACKAGECONFIG:append = " proprietary-codecs"
> @@ -11,15 +9,5 @@ SRC_URI:append = " \
>                     file://0002-chromium-gpu-sandbox-Allow-GPU-sandbox-access-to-V4L.patch \
>                     file://0003-chromium-gpu-v4l2-Fix-OUTPUT-queue-streaming-in-V4L2.patch \
>                     file://0004-media-gpu-v4l2-Avoid-placing-incomplete-H264-access-.patch \
> -                  file://force-h264-youtube/force_h264.js \
> -                  file://force-h264-youtube/manifest.json \

I think these changes should be in the second patch.

> +                  file://0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch \
>                     "
> -
> -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}/
> -}
> -
> -FILES:${PN}:append = " ${EXTENSION_DIR}"

I think these changes should be in the second patch.

> diff --git a/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch
> new file mode 100644
> index 00000000..ec87e35d
> --- /dev/null
> +++ b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch
> @@ -0,0 +1,34 @@
> +From 5bcdc4db88d908abdeba11270740864a13bafad1 Mon Sep 17 00:00:00 2001
> +From: Thorsten Lannynd <t-lannynd@ti.com>
> +Date: Mon, 28 Sep 2026 13:22:56 -0500
> +Subject: [PATCH] chromium: wrapper: Auto-load arago extensions at runtime
> +
> +Upstream-Status: Inappropriate [TI-specific extension directory]
> +
> +Scan $HERE/extensions/arago/ at runtime and pass any installed
> +subdirectories through --load-extension, allowing all TI extensions to be
> +enabled by default.
> +
> +Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
> +---
> + chrome/installer/linux/common/wrapper | 6 ++++++
> + 1 file changed, 6 insertions(+)
> +
> +diff --git a/chrome/installer/linux/common/wrapper b/chrome/installer/linux/common/wrapper
> +index 3547069c2a..76588abda8 100755
> +--- a/chrome/installer/linux/common/wrapper
> ++++ b/chrome/installer/linux/common/wrapper
> +@@ -28,5 +28,11 @@ exec 2> >(exec cat >&2)
> +
> + CHROME_EXTRA_ARGS=""
> +
> ++# Auto-load arago extensions at runtime
> ++for _ext in "$HERE/extensions/arago"/*/; do
> ++    [ -d "$_ext" ] && _ext_list="${_ext_list:+$_ext_list,}${_ext%/}"
> ++done
> ++[ -n "$_ext_list" ] && CHROME_EXTRA_ARGS="$CHROME_EXTRA_ARGS --load-extension=$_ext_list"
> ++
> + # Note: exec -a below is a bashism.
> + exec -a "$0" "$HERE/@@PROGNAME@@" ${CHROME_EXTRA_ARGS} "$@"
> +--
> +2.34.1
PRC Automation Sept. 30, 2026, 4:25 p.m. UTC | #2 
meta-arago / na / 20260930160338.951001-1-t-lannynd

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
WARN - [meta-arago][master/wrynose][PATCH v7 2/2] chromium: add force-h264-youtube extension
    WARN: Commit message does not include file/recipe name: chromium: add force-h264-youtube extension. (COMMIT-MESSAGE-2)
        patch
    
    WARN: Revised patch missing a comment. (REVISION-1)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887



=========================================================
  apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [meta-arago][master/wrynose][PATCH v7 1/2] chromium: auto-load arago extensions at runtime
- Submitter: From: Thorsten Lannynd <t-lannynd@ti.com>
+From: Thorsten Lannynd <t-lannynd@ti.com>
- Date: Date: Wed, 30 Sep 2026 11:03:37 -0500
+Date: Mon, 28 Sep 2026 13:22:56 -0500
- Num Patches: 2
- Mailing List (public inbox) Commit SHA: ba6391ad6bf62f32a072f2336cd0cdcff0d49567

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: master-wip
- Commit Author: Ryan Eatmon <reatmon@ti.com>
- Commit Subject: open62541: upgrade 1.5.7 -> 1.5.8
- Commit SHA: 170fff909c58178a4c21b76fb3866b17bd9b7cc3

Patches
----------------------------------------
All patches applied

wrynose
=====================
Summary:
- Patch Series: [meta-arago][master/wrynose][PATCH v7 1/2] chromium: auto-load arago extensions at runtime
- Submitter: From: Thorsten Lannynd <t-lannynd@ti.com>
+From: Thorsten Lannynd <t-lannynd@ti.com>
- Date: Date: Wed, 30 Sep 2026 11:03:37 -0500
+Date: Mon, 28 Sep 2026 13:22:56 -0500
- Num Patches: 2
- Mailing List (public inbox) Commit SHA: ba6391ad6bf62f32a072f2336cd0cdcff0d49567

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: wrynose-wip
- Commit Author: Ryan Eatmon <reatmon@ti.com>
- Commit Subject: check_yocto_rules: Add exception for pipewire update
- Commit SHA: cc1013940d3b41154ab5e3e6e82a9b584a02820f

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
master
=====================
PASS

wrynose
=====================
PASS



=========================================================
  yocto-check-layers: FAIL
=========================================================
master - FAIL
=====================

ERROR: Multiple recipes found for mesa-demos
    oe-core/meta/recipes-graphics/mesa/mesa-demos_9.0.0.bb
    meta-arago/meta-arago-distro/recipes-graphics/mesa/mesa-demos_git.bb

wrynose - PASS
=====================
All checks passed
diff mbox series

Patch

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..40b74c18 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,7 +1,5 @@ 
 PR:append = ".arago0"
 
-EXTENSION_DIR = "/usr/lib/chromium/extensions/force-h264-youtube"
-
 FILESEXTRAPATHS:prepend := "${THISDIR}/chromium-ozone-wayland:"
 
 PACKAGECONFIG:append = " proprietary-codecs"
@@ -11,15 +9,5 @@  SRC_URI:append = " \
                   file://0002-chromium-gpu-sandbox-Allow-GPU-sandbox-access-to-V4L.patch \
                   file://0003-chromium-gpu-v4l2-Fix-OUTPUT-queue-streaming-in-V4L2.patch \
                   file://0004-media-gpu-v4l2-Avoid-placing-incomplete-H264-access-.patch \
-                  file://force-h264-youtube/force_h264.js \
-                  file://force-h264-youtube/manifest.json \
+                  file://0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch \
                   "
-
-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}/
-}
-
-FILES:${PN}:append = " ${EXTENSION_DIR}"
diff --git a/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch
new file mode 100644
index 00000000..ec87e35d
--- /dev/null
+++ b/meta-arago-distro/dynamic-layers/chromium-browser-layer/recipes-browser/chromium/chromium-ozone-wayland/0005-chromium-wrapper-Auto-load-arago-extensions-at-runti.patch
@@ -0,0 +1,34 @@ 
+From 5bcdc4db88d908abdeba11270740864a13bafad1 Mon Sep 17 00:00:00 2001
+From: Thorsten Lannynd <t-lannynd@ti.com>
+Date: Mon, 28 Sep 2026 13:22:56 -0500
+Subject: [PATCH] chromium: wrapper: Auto-load arago extensions at runtime
+
+Upstream-Status: Inappropriate [TI-specific extension directory]
+
+Scan $HERE/extensions/arago/ at runtime and pass any installed
+subdirectories through --load-extension, allowing all TI extensions to be
+enabled by default.
+
+Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
+---
+ chrome/installer/linux/common/wrapper | 6 ++++++
+ 1 file changed, 6 insertions(+)
+
+diff --git a/chrome/installer/linux/common/wrapper b/chrome/installer/linux/common/wrapper
+index 3547069c2a..76588abda8 100755
+--- a/chrome/installer/linux/common/wrapper
++++ b/chrome/installer/linux/common/wrapper
+@@ -28,5 +28,11 @@ exec 2> >(exec cat >&2)
+
+ CHROME_EXTRA_ARGS=""
+
++# Auto-load arago extensions at runtime
++for _ext in "$HERE/extensions/arago"/*/; do
++    [ -d "$_ext" ] && _ext_list="${_ext_list:+$_ext_list,}${_ext%/}"
++done
++[ -n "$_ext_list" ] && CHROME_EXTRA_ARGS="$CHROME_EXTRA_ARGS --load-extension=$_ext_list"
++
+ # Note: exec -a below is a bashism.
+ exec -a "$0" "$HERE/@@PROGNAME@@" ${CHROME_EXTRA_ARGS} "$@"
+--
+2.34.1