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

Please incorporate --with-fdk-aac into ffmpeg #36938

Closed
montagne27 opened this issue Feb 12, 2019 · 10 comments
Closed

Please incorporate --with-fdk-aac into ffmpeg #36938

montagne27 opened this issue Feb 12, 2019 · 10 comments
Labels
outdated PR was locked due to age

Comments

@montagne27
Copy link

Hello,

Your questionnable move to remove all options from Homebrew has left me without access to the "libfdk_aac" encoder for ffmpeg, which is the highest-quality AAC encoder, as ffmpeg's own documentation says here:

Fraunhofer FDK AAC (libfdk_aac)
This is currently the highest-quality AAC encoder available with ffmpeg.

and here:

For AAC-LC: libfdk_aac > Native FFmpeg AAC encoder (aac).
For HE-AAC you have one choice: libfdk_aac.

Furthermore, if I look at the Homebrew analytics for the ffmpeg formula, "--with-fdk-aac" was easily one of the most commonly used options. Please incorporate "--with-fdk-aac" into ffmpeg.

@SMillerDev
Copy link
Member

Please make a pull request as we don't accept issues requesting features.

@montagne27
Copy link
Author

montagne27 commented Feb 12, 2019

Unfortunately, solutions like "making a pull request" are beyond my expertise. I don't even know what they mean. If I am not a developer, how do I propose the "--with-fdk-aac" option as default?

@montagne27
Copy link
Author

Browsing the current pull requests, I found #36327, in which @MikeMcQuaid says:

We cannot legally distribute bottles with FDK AAC so we will not do so.

I guess this means that "--with-fdk-aac" will never be default, whatever arguments I make or other users make, such as @zmwangx in the same pull request:

Please add back fdk-aac. AAC is one of the (two) most widely audio codecs and fdk-aac was the de facto standard of AAC encoding with FFmpeg for years

@retokromer
Copy link
Contributor

retokromer commented Feb 12, 2019

@montagne27 There is a version of the old formula which allows you to pass options.

@montagne27
Copy link
Author

@retokromer Brilliant! Thank you very much, it worked flowlessly. Now, I just have to hope that "ffmpeg-with-options" will remain compatible with future versions of Homebrew.

@MikeMcQuaid
Copy link
Member

I guess this means that "--with-fdk-aac" will never be default, whatever arguments I make or other users make

Yes. Please do this research before opening issues in future and you'll avoid asking the Homebrew project to break the law to meet your needs, thanks.

@montagne27
Copy link
Author

montagne27 commented Feb 12, 2019

@MikeMcQuaid I did do research about existing "Issues" with the ffmpeg formula, prior to opening my own one. Only later did I find out the discussion was in "Pull requests" instead of in "Issues". Please keep in mind that I am a visitor and that it's normal if I am not familiar with where everything is. Just like you're unfamiliar the first time you're visiting a hospital or a public library. Also, I am not asking you to break the law by providing me the compiled binary: I am asking for the option to compile this binary from source. After all, isn't that Homebrew's purpose? Anyway, opening this "Issue" allowed me to find out about "ffmpeg-with-options", so I guess it wasn't meaningless for me. Who knows? Some other visitor might also find it meaningful.

@MikeMcQuaid
Copy link
Member

After all, isn't that Homebrew's purpose?

No, it's not a from-source package manager.

Some other visitor might also find it meaningful.

Yes, I hope so too.

Thanks!

@sidneys
Copy link
Contributor

sidneys commented Feb 17, 2019

After all, isn't that Homebrew's purpose?

No, it's not a from-source package manager.

Some other visitor might also find it meaningful.

Yes, I hope so too.

Thanks!

Dear @MikeMcQuaid,

with regards to your last comment: Is the removal of Homebrews' --HEAD installation feature currently planned / on the roadmap?

Cheers,
S

@SMillerDev
Copy link
Member

@sidneys that option serves a very different purpose from all options that have now been removed so afaik it isn't on the roadmap to be removed.

@lock lock bot added the outdated PR was locked due to age label Mar 19, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Mar 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

No branches or pull requests

5 participants