Skip to content

desktops: install libv4l-0 before rockchip-multimedia packages#887

Open
igorpecovnik wants to merge 1 commit intomainfrom
desktops-rockchip-multimedia-libv4l-0
Open

desktops: install libv4l-0 before rockchip-multimedia packages#887
igorpecovnik wants to merge 1 commit intomainfrom
desktops-rockchip-multimedia-libv4l-0

Conversation

@igorpecovnik
Copy link
Copy Markdown
Member

Summary

  • rockchip-multimedia-config's postinst expects libv4l-0 to already be present — otherwise the configure step can fail on a fresh minimal image where the V4L2 userspace lib hasn't been pulled in yet
  • Adds a separate pkg_install libv4l-0 between pkg_update and the rockchip-multimedia-* batch, inside the existing noble/rk3588/vendor gate in _module_desktops_rockchip_multimedia
  • Mirrors the ordering from armbian/build's extensions/mesa-vpu.sh (which does the same apt-get install libv4l-0 call before the multimedia batch)

Follow-up to #885.

Test plan

  • module_desktops install de=<any> tier=mid mode=build on an rk3588 / vendor / noble build reports libv4l-0 in ACTUALLY_INSTALLED before the rockchip-multimedia packages
  • Same install on a non-rk3588 / non-noble target stays a no-op (entire block is gated)
  • module_desktops remove de=<name> purges libv4l-0 via the manifest

rockchip-multimedia-config's postinst expects the V4L2 userspace library to
already be present. Mirror the ordering from armbian/build's
extensions/mesa-vpu.sh by doing a separate pkg_install of libv4l-0 between
pkg_update and the rockchip-multimedia-* batch, inside the existing
noble/rk3588/vendor gate in _module_desktops_rockchip_multimedia.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

Warning

Rate limit exceeded

@igorpecovnik has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 21 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 21 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 305a5faa-5c1a-4f1b-9adc-62fdc9b0e841

📥 Commits

Reviewing files that changed from the base of the PR and between 1329d6b and 30b3d9e.

📒 Files selected for processing (1)
  • tools/modules/desktops/module_desktops.sh
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch desktops-rockchip-multimedia-libv4l-0

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size/small PR with less then 50 lines 05 Milestone: Second quarter release labels Apr 21, 2026
igorpecovnik added a commit to armbian/build that referenced this pull request Apr 21, 2026
All remaining responsibilities of post_install_kernel_debs__3d moved to
armbian-configng's module_desktops, invoked via `module_desktops install
mode=build` during rootfs cache assembly:

  - Mesa runtime + Vulkan drivers + glmark2/mesa-utils/vulkan-tools:
    configng common.yaml mid tier (armbian/configng 3d8fff72).
  - panthor-gpu DT overlay: _module_desktops_rockchip_multimedia
    stage 2 (armbian/configng 41310eff).
  - amazingfated ppa:liujianfeng1994/rockchip-multimedia + pin at
    priority 1001 + rockchip-multimedia-config, libv4l-rkmpp,
    gstreamer1.0-rockchip, libwidevinecdm0 install on noble/rk3588/
    vendor: _module_desktops_rockchip_multimedia stage 1
    (armbian/configng 1329d6b4, follow-up in armbian/configng#887
    for the libv4l-0 pre-install ordering).

Caveat: configng sets up the PPA + pin and installs the multimedia
packages, but does not run a post-install `apt-get dist-upgrade
--allow-downgrades` to actively swap pre-installed chromium from
apt.armbian.com to the PPA's hardware-accelerated build. That swap
now happens on the user's first `apt upgrade` after boot (priority
1001 permits the downgrade-across-origins); shipped images carry
the apt.armbian.com chromium. If we want image-time swap back, a
follow-up in configng can add the dist-upgrade inside the same
noble/rk3588/vendor gate. The KDE-neon `apt-mark hold base-files`
and arm-noble/oracular `apt-mark hold libdav1d7` bracket calls are
dropped along with the dist-upgrade they were guarding.

No enable_extension or config references to mesa-vpu remain in the
build tree; no other callers to remove.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

05 Milestone: Second quarter release size/small PR with less then 50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant