minidlna: fix build with FFmpeg 7 and later - #30563
Open
marcinmajsc wants to merge 1 commit into
Open
marcinmajsc wants to merge 1 commit into
marcinmajsc wants to merge 1 commit into
Conversation
marcinmajsc
marked this pull request as draft
September 19, 2026 23:02
marcinmajsc
force-pushed
the
minidlna-fix
branch
from
September 19, 2026 23:10
ce91a28 to
93bce50
Compare
marcinmajsc
force-pushed
the
minidlna-fix
branch
from
September 19, 2026 23:41
93bce50 to
6c109fe
Compare
marcinmajsc
marked this pull request as ready for review
September 19, 2026 23:55
openwrt-ai
reviewed
Sep 20, 2026
| @@ -0,0 +1,22 @@ | |||
| From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 | |||
Collaborator
There was a problem hiding this comment.
The all-zero SHA plus the placeholder 12:00:00 date fabricate a git header, so this reads as a cherry-pick of commit 0000000 when nothing was cherry-picked. The comparable ffmpeg-compat patch in this feed, multimedia/gst1-libav/patches/001-gst-libav-Fix-build-failure-with-FFmpeg-9.0.patch,`` carries the real upstream SHA and a link to the upstream MR. Is there an existing ReadyMedia commit for this that should be backported instead — and if not, can the fake From/`Date` lines be dropped in favour of an upstream reference or `Upstream-Status`?
Generated by Claude Code
FFmpeg 7 removed AVCodecParameters.channels, causing MiniDLNA 1.3.3 to fail to compile when reading audio channel counts. Add an OpenWrt package patch to use ch_layout.nb_channels with libavcodec 61 and later. Retain the existing channels field access for older versions. Signed-off-by: Marcin Maj <marcinmajsc@gmail.com>
BKPepe
force-pushed
the
minidlna-fix
branch
from
September 21, 2026 06:04
6c109fe to
ef031e2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
FFmpeg 7 removed AVCodecParameters.channels, causing MiniDLNA 1.3.3 to fail to compile when reading audio channel counts.
Add an OpenWrt package patch to use ch_layout.nb_channels with libavcodec 61 and later. Retain the existing channels field access for older versions.
📦 Package Details
Maintainer: Alexandru Ardelean ardeleanalex@gmail.com
Description:
MiniDLNA (aka ReadyDLNA) is server software with the aim of being fully compliant with DLNA/UPnP-AV clients.
🧪 Run Testing Details
✅ Formalities
If your PR contains a patch:
git am(e.g., subject line, commit description, etc.)
We must try to upstream patches to reduce maintenance burden.