[FFmpeg-devel] Configure Script change request to support libcdio on macOS

[Michael Niedermayer]

On Mon, Sep 18, 2017 at 11:42:11AM +0900, 桃源老師 wrote:
> Hello, 
> 
> > 2017/09/17 午後11:58、桃源老師 <xanadu6291 at dream.jp>のメール:
> > 
> > I have tried to use libcdio function with ffmpeg on macOS. Then I get an information from libcdio team.  That is ffmpeg configure script should read libcdio's pkgconfig file, libcdio.pc.
> 
> I attach my request as patch file.  Please review it.
> 

>  configure |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 01105359616694b3c3c9a8f2e60d1f875ddef3bf  ffmpeg_configure.patch
> --- a/configure	2017-09-18 10:24:52.000000000 +0900
> +++ b/configure	2017-09-18 11:31:58.000000000 +0900
> @@ -6151,7 +6151,7 @@
>  
>  if enabled libcdio; then
>      check_lib libcdio "cdio/cdda.h cdio/paranoia.h" cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio ||
> -    check_lib libcdio "cdio/paranoia/cdda.h cdio/paranoia/paranoia.h" cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio ||
> +    require_pkg_config libcdio "cdio/paranoia/cdda.h cdio/paranoia/paranoia.h" cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio && check_lib libcdio "cdio/paranoia/cdda.h cdio/paranoia/paranoia.h" cdio_cddap_open -lcdio_paranoia -lcdio_cdda -lcdio ||
>      die "ERROR: No usable libcdio/cdparanoia found"
>  fi
>  

This breaks FFmpeg on ubuntu with

ii  libcdio-cdda-dev                                                        0.83-4.1ubuntu1                          amd64                                    library to read and control digital audio CDs (development files)
ii  libcdio-cdda0                                                           0.81-4build1                             amd64                                    library to read and control digital audio CDs
ii  libcdio-cdda1                                                           0.83-4.1ubuntu1                          amd64                                    library to read and control digital audio CDs
ii  libcdio-dev                                                             0.83-4.1ubuntu1                          amd64                                    library to read and control CD-ROM (development files)
ii  libcdio-paranoia-dev                                                    0.83-4.1ubuntu1                          amd64                                    library to read digital audio CDs with error correction (development files)
ii  libcdio-paranoia0                                                       0.81-4build1                             amd64                                    library to read digital audio CDs with error correction
ii  libcdio-paranoia1                                                       0.83-4.1ubuntu1                          amd64                                    library to read digital audio CDs with error correction
ii  libcdio-utils                                                           0.83-4.1ubuntu1                          amd64                                    sample applications based on the CDIO libraries
ii  libcdio10                                                               0.81-4build1                             amd64                                    library to read and control CD-ROM
ii  libcdio13                                                               0.83-4.1ubuntu1                          amd64                                    library to read and control CD-ROM

./configure --enable-libcdio --enable-gpl
ERROR: No usable libcdio/cdparanoia found

If you think configure made a mistake, make sure you are using the latest
version from Git.  If the latest version fails, report the problem to the
ffmpeg-user at ffmpeg.org mailing list or IRC #ffmpeg on irc.freenode.net.
Include the log file "ffbuild/config.log" produced by configure as this will help
solve the problem.


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170919/244eab5f/attachment.sig>