[FFmpeg-devel] [RFC]Use pkg-config as an alternative for x264 detection

Clément Bœsch u at pkh.me
Wed Jun 11 09:21:16 CEST 2014


On Wed, Jun 11, 2014 at 09:18:54AM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Would attached (not well tested) make the pkg-config junkies happier?
> 
> Please understand that switching to pkg-config as a requirement 
> will produce a shitload of user reports that I would really 
> like to avoid.
> 
> Please comment, Carl Eugen

> diff --git a/configure b/configure
> index fa66c4f..4dea7fb 100755
> --- a/configure
> +++ b/configure
> @@ -4718,7 +4718,8 @@ enabled libvpx            && {
>      enabled libvpx_vp9_encoder && { check_lib2 "vpx/vpx_encoder.h vpx/vp8cx.h" "vpx_codec_vp9_cx VP9E_SET_SVC" -lvpx || disable libvpx_vp9_encoder; } }
>  enabled libwavpack        && require libwavpack wavpack/wavpack.h WavpackOpenFileOutput  -lwavpack
>  enabled libwebp           && require_pkg_config libwebp webp/encode.h WebPGetEncoderVersion
> -enabled libx264           && require libx264 x264.h x264_encoder_encode -lx264 &&
> +enabled libx264           && { require libx264 x264.h x264_encoder_encode -lx264 ||
> +                               require_pkg_config x264 "stdint.h x264.h" x264_encoder_encode; } &&
>                               { check_cpp_condition x264.h "X264_BUILD >= 118" ||
>                                 die "ERROR: libx264 must be installed and version must be >= 0.118."; }
>  enabled libx265           && require_pkg_config x265 x265.h x265_encoder_encode &&

I did test my version, which also includes the proper version check. I'll
apply mine if you don't mind.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140611/8e024266/attachment.asc>


More information about the ffmpeg-devel mailing list