libcdio is missing headers for "libcdio-paranoia"

[slhck]

I'm not an expert when it comes to compilation and linking, but here's the problem:

• I want to install ffmpeg with libcdio support
• I install libcdio via Homebrew
• I configure ffmpeg with --enable-libcdio
• It errors out with ERROR: libcdio-paranoia not found

Here's some configuration log:

check_lib2 cdio/paranoia/cdda.h cdio/paranoia/paranoia.h cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio
check_func_headers cdio/paranoia/cdda.h cdio/paranoia/paranoia.h cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio
check_ld cc -lcdio_paranoia -lcdio_cdda -lcdio
check_cc
BEGIN /var/folders/lr/6q31nm4x0zz9znwpx8mkq9kr0000gn/T//ffconf.NbtoXn1g.c
    1 #include <cdio/paranoia/cdda.h>
    2 #include <cdio/paranoia/paranoia.h>
    3 long check_cdio_cddap_open(void) { return (long) cdio_cddap_open; }
    4 int main(void) { return 0; }
END /var/folders/lr/6q31nm4x0zz9znwpx8mkq9kr0000gn/T//ffconf.NbtoXn1g.c
gcc -D_ISOC99_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -DPIC -std=c99 -mdynamic-no-pic -fomit-frame-pointer -fPIC -pthread -D_GNU_SOURCE=1 -D_THREAD_SAFE -I/usr/local/include/SDL -c -o /var/folders/lr/6q31nm4x0zz9znwpx8mkq9kr0000gn/T//ffconf.Cg1rgGDZ.o /var/folders/lr/6q31nm4x0zz9znwpx8mkq9kr0000gn/T//ffconf.NbtoXn1g.c
/var/folders/lr/6q31nm4x0zz9znwpx8mkq9kr0000gn/T//ffconf.NbtoXn1g.c:1:10: fatal error: 'cdio/paranoia/cdda.h' file not found
#include <cdio/paranoia/cdda.h>
         ^
1 error generated.
ERROR: libcdio-paranoia not found

Similar issues with other projects tell me that:

Now the problem is solved by compiling cdio from source, brew install libcdio doesn't do the work.

Apparently, under Ubuntu, installing libcdio-paranoia-dev is enough to fix the issue.

What needs to be changed for Homebrew to support this? Can this be added easily?

[adamv]

What do your changes to the ffmpeg formula look like so far?

[slhck]

@adamv I haven't changed the ffmpeg formula now. I've tried to compile it manually first, with the simplest configure command possible (./configure --enable-libcdio --enable-gpl).

If I were to change the ffmpeg formula I'd just add something like this:

option "with-cdio", "Enable the GNU Compact Disc Input and Control Library"
...
depends_on 'libcdio' => :optional
...
args << "--enable-libcdio" if build.with? 'cdio'

[jacknagel]

It seems that the cdio-paranoia bits were removed from the main libcdio tree, so presumably we'd need a new formula.

[MikeMcQuaid]

@slhck Could you try and submit a pull request? Thanks!

[slhck]

@MikeMcQuaid I would already have if there was an easy solution for the ffmpeg part, but as @jacknagel said, the parts that are needed require a new formula altogether, and I haven't had time (nor am I an expert with cdio) to create a new formula.

[MikeMcQuaid]

Ok. I think we're going to remove this functionality from the formula then. Sorry.

[slhck]

There's no functionality you have to remove — I just was manually changing the ffmpeg formula to see if I can get support for libcdio.

[MikeMcQuaid]

Ah ok, thanks.