build.yml: convert ffmpeg+mpv-debian install-deps to apt-get (closes #55) #56

Merged
marfrit merged 1 commits from fix/debian-runner-deps-2026-05-20 into main 2026-05-20 19:13:50 +00:00
Owner

Fixes the latent break flagged in #55.

PR #47 routed ffmpeg-v4l2-request-debian and mpv-fourier-debian to runs-on: debian-aarch64 (bohr) but their install-deps steps still called pacman -Syu. Currently silent-skipped by check-already-published.sh (pool versions match the staged PKGVER); next pkgver bump would surface the failure.

This patch follows PR #50's pattern (daedalus-debian + dkms-debian):

  • Replace retry pacman -Syu ... with retry apt-get install ... in both jobs
  • Translate Arch package names to Debian (base-devel→build-essential, pkgconf→pkg-config, libdrm→libdrm-dev, x264→libx264-dev, etc.)
  • For mpv: drop the configure [marfrit] repo + pre-install ffmpeg-v4l2-request-fourier step. Under apt, stock libavcodec-dev/libavformat-dev/libavutil-dev provide trixie-ABI headers matching what mpv-fourier's binary will see at runtime; the daemon dlopen's the fourier libs if installed but doesn't link against them at build time.

Diff stats: 38 insertions, 30 deletions, ~30 line patch as estimated.

Validated upstream: equivalent debian build-deps installed cleanly in PRs #44 (libva) and #50 (daedalus).

Fixes the latent break flagged in #55. PR #47 routed `ffmpeg-v4l2-request-debian` and `mpv-fourier-debian` to `runs-on: debian-aarch64` (bohr) but their install-deps steps still called `pacman -Syu`. Currently silent-skipped by `check-already-published.sh` (pool versions match the staged PKGVER); next pkgver bump would surface the failure. This patch follows PR #50's pattern (daedalus-debian + dkms-debian): - Replace `retry pacman -Syu ...` with `retry apt-get install ...` in both jobs - Translate Arch package names to Debian (base-devel→build-essential, pkgconf→pkg-config, libdrm→libdrm-dev, x264→libx264-dev, etc.) - For mpv: drop the `configure [marfrit] repo + pre-install ffmpeg-v4l2-request-fourier` step. Under apt, stock `libavcodec-dev`/`libavformat-dev`/`libavutil-dev` provide trixie-ABI headers matching what mpv-fourier's binary will see at runtime; the daemon dlopen's the fourier libs if installed but doesn't link against them at build time. Diff stats: 38 insertions, 30 deletions, ~30 line patch as estimated. Validated upstream: equivalent debian build-deps installed cleanly in PRs #44 (libva) and #50 (daedalus).
marfrit added 1 commit 2026-05-20 19:10:10 +00:00
Closes #55.

PR #47 routed ffmpeg-v4l2-request-debian and mpv-fourier-debian to
runs-on: debian-aarch64 (bohr), but their install-deps steps still
called pacman -Syu. That is a latent break that would surface on the
next pkgver bump (currently silent-skipped by check-already-published.sh
since pool versions match the staged PKGVER).

This patch follows PR #50's pattern (daedalus-v4l2{,-dkms}-debian):

- Replace retry pacman -Syu ... with retry apt-get install ...
- Translate Arch package names to Debian (base-devel -> build-essential,
  pkgconf -> pkg-config, libdrm -> libdrm-dev, x264 -> libx264-dev, etc.).
- For mpv: drop the "configure [marfrit] repo + pre-install
  ffmpeg-v4l2-request-fourier" step entirely. Under apt, stock
  libavcodec-dev / libavformat-dev / libavutil-dev provide trixie-ABI
  headers matching what mpv-fourier's binary will see at runtime; the
  daemon dlopens the fourier libs if installed but doesn't link against
  them at build time.

Validated upstream: equivalent debian build-deps installed cleanly in
PRs #44 (libva) and #50 (daedalus).
marfrit merged commit eab66cfab8 into main 2026-05-20 19:13:50 +00:00
marfrit deleted branch fix/debian-runner-deps-2026-05-20 2026-05-20 19:13:50 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: marfrit/marfrit-packages#56