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 59 libavfomat support (with photos) #2388

Closed
CanadianJeff opened this issue Jan 6, 2022 · 7 comments
Closed

FFmpeg 59 libavfomat support (with photos) #2388

CanadianJeff opened this issue Jan 6, 2022 · 7 comments

Comments

@CanadianJeff
Copy link

CanadianJeff commented Jan 6, 2022

https://github.com/audacity/audacity/blob/release-3.1.3/libraries/lib-ffmpeg-support/impl/ffmpeg-4.2.4-single-header.h#L4161
version 59 support

would suggest that libavformat 59 "should just work"

however trying to import it into audacity yields only 58.77.100 works

import in audacity

downloaded 59 from ffmpegs autobuild github........

58 works
https://github.com/BtbN/FFmpeg-Builds/releases/tag/autobuild-2021-03-31-12-39
59 does not
https://github.com/BtbN/FFmpeg-Builds/releases/tag/autobuild-2021-04-30-12-35

@SteveDaulton
Copy link
Member

would suggest that libavformat 59 "should just work"

Why should it work?
It says < 59 (less than).
Also, the commit message

@CanadianJeff
Copy link
Author

CanadianJeff commented Jan 7, 2022

so why hard code the requirement of under 59?????? something seems really fishy here........

based on the date of that autobuild being 2021-04-30 for ffmpeg 59 and audacity 3.1.3 release date being "after" 2021-04-30 something doesn't add up

that commit was Sept 2021 several months after the release of ffmpeg 59

@crsib
Copy link
Contributor

crsib commented Jan 7, 2022

The latest released version is 4.4, which bundles avformat 58.76.100. avformat 59 is not yet released and is a subject of an ABI break. The support for avformat 59 will be added after the official FFmpeg release

@crsib crsib closed this as completed Jan 7, 2022
@CanadianJeff
Copy link
Author

CanadianJeff commented Jan 7, 2022

that just sounds like a major cover up

why prevent unreleased versions and why close the issue?

@LWinterberg
Copy link
Member

As per https://wiki.audacityteam.org/wiki/Release_Notes_3.1

FFMPEG now supports avformat 55, 57 and 58.

Also, note that https://github.com/audacity/audacity/tree/release-3.1.3/libraries/lib-ffmpeg-support/impl has 3 different files, one per supported avformat version. So I don't really get why you call this a "cover up".

why prevent unreleased versions

Because avformat 59 is not yet released and is a subject of an ABI break.

why close the issue?

Because avformat 59 is not released yet and as long as it's not, it won't be added. Once it is released, we plan to add support for it to the next version we release.

@CanadianJeff
Copy link
Author

CanadianJeff commented Jan 7, 2022

again I dont know much about ABI but my take is how compat ffmpeg is with audacity and how compat audacity is with ffmpeg...........

I still think giving the user a choice to load in an unsupported "External Lib" should be allowed that way people can find out whats supported and whats not.....

maybe prompt the user to "proceed at his/her own risk" if they attempt to use an unsupported ffmpeg version

@BtbN
Copy link

BtbN commented Jan 7, 2022

The 59 bump ABI instability period is long over.
5.0 has been branched and is in its final polishing before being tagged as first 5.0 release. No further ABI breaks will occur there anymore (or in master for a long while).

Generally, FFmpeg does not break ABI outside of a short ABI instability period after every major bump in master.

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

No branches or pull requests

5 participants