Homebrew / homebrew-core

🍻 Default formulae for the missing package manager for macOS (or Linux)
https://brew.sh
BSD 2-Clause "Simplified" License
13.7k stars 12.4k forks source link

flacenc not included with gst-plugins-good #13015

Closed theosib closed 7 years ago

theosib commented 7 years ago

Please always follow these steps:

According to "http://brewformulas.org/GstPluginsGood", flac should be included with this package, but it seems to be only partially included. I don't know how the problem could be due to anything other than this package.

I ran those. There isn't any malfunction with brew. It's that a plugin

[EDIT] Here you go: https://gist.github.com/83d969fdb730aa881153769d21bb9f8b

I don't think these are relevant, and the output I get says nothing about this package.

To help us debug your issue please explain:

In general and on other systems, the gst-plugins-good package includes full FLAC support. So when I install this package, gstreamer features like "flacenc" should work.

For instance, after I have installed gstreamer base and streamer plugin packages on any Linux system, this command works as intended:

gst-launch-1.0 -q autoaudiosrc ! flacenc ! filesink location=test2.flac

But with gst-plugins-good from homebrew, this gives me an error: WARNING: erroneous pipeline: no element "flacenc"

Note that if I substitute "wavenc", it works fine. So this package is just missing a key component.

theosib commented 7 years ago

So, if I understand this correctly, /usr/local/lib/gstreamer-1.0 should contain libgstflac.so or similar, but it doesn't.

theosib commented 7 years ago

I built from source and added the log to the description. As you can see, it says this in the log:

configure: checking feature: FLAC lossless audio configure: for plug-ins: flac checking for FLAC... no configure: No package 'flac' found configure: *** These plugins will not be built: flac

There are two problems with this: (1) flac is supposed to be an automatic dep of this, and (2) I manually installed flac using brew and reinstalled gst-plugins-good, and it still didn't detect flac.

$ brew install flac Warning: flac-1.3.2 already installed

theosib commented 7 years ago

Ok, I finally figured it out. For some bizarre reason, unlike every other gstreamer installation I've ever encountered, flac is made optional with brew, requiring the use of the --with-flac option. The problem is that the documentation on this module fails to mention this key piece of information.

Where do I file bugs on documentation?

ibrokemypie commented 7 years ago

well, to start you would just do brew info gst-plugins-good which lists all of the options... however i think the real issue is that flac should be enabled by default, much more likely that users want flac, so make it recommended rather than optional.

dunn commented 7 years ago

The problem is that the documentation on this module fails to mention this key piece of information.

Which documentation do you mean?

We'd happily review a pull request making that option on by default; take a look at https://github.com/Homebrew/brew/blob/master/docs/How-To-Open-a-Homebrew-Pull-Request.md if you're interested and let us know if you have questions!

stale[bot] commented 7 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

ilovezfs commented 7 years ago

@theosib I'm glad you found your way to the --with-flac option. Thanks for taking the time to report this.