Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ffmpeg: add swresample and more decoders/parsers to audio-dec variant #25657

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

willmo
Copy link

@willmo willmo commented Jan 2, 2025

Add the AAC decoder + demuxer and the AAC, AC3, FLAC, and MPEG Audio parsers, since these seem obviously important for audio decoding.

Also add the swresample library, which is quite small and seems plausibly useful for audio decoding. For example, shairport-sync requires it for AirPlay 2 support, so this allows changing shairport-sync's dependency from libffmpeg-full to libffmpeg-audio-dec, which makes for a significant reduction in image size.

I verified that shairport-sync AirPlay 2 still works correctly with its dependency changed to libffmpeg-audio-dec. (I'll submit a separate PR for that if this is accepted.)

Maintainer: @neheb @BKPepe @thess
Compile/run tested: ramips, Cudy TR1200 v1, OpenWrt 24.10-SNAPSHOT, r28242-1eff737906

CC #22993 @mikebrady @bklang

Add the AAC decoder + demuxer and the AAC, AC3, FLAC, and MPEG Audio
parsers, since these seem obviously important for audio decoding.

Also add the swresample library, which is quite small and seems
plausibly useful for audio decoding. For example, shairport-sync
requires it for AirPlay 2 support, so this allows changing
shairport-sync's dependency from libffmpeg-full to libffmpeg-audio-dec,
which makes for a significant reduction in image size.

Signed-off-by: Will Mortensen <[email protected]>
@neheb
Copy link
Contributor

neheb commented Jan 2, 2025

I think the original reason for aac’s absence is patents.

@Neustradamus
Copy link

@systemcrash: ^

@systemcrash
Copy link
Contributor

Decoders and parsers do not require patent licenses.

@mikebrady
Copy link
Contributor

Thanks for all this -- good work!

@neheb
Copy link
Contributor

neheb commented Jan 7, 2025

hrm

Package libffmpeg-audio-dec is missing dependencies for the following libraries:
libatomic.so.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants