> -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > Derek Buitenhuis > Sent: Friday, May 20, 2022 6:55 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2] avcodec/libx264: allow to > disable definition of X264_API_IMPORTS macro > > On 5/20/2022 5:51 PM, Martin Storsjö wrote: > > Maybe just drop support for older versions when on Windows? That > should > > cover those cases (even if ffmpeg is built with msvc but x264 with > mingw, > > or vice versa) quite well, while still not bothering other platforms > at > > all. > > Yeah, that seems reasonable to me.
To me as well! This is the code in configure after (fixing and) applying Matt's patch: enabled libwebp && { enabled libwebp_encoder && require_pkg_config libwebp "libwebp >= 0.2.0" webp/encode.h WebPGetEncoderVersion enabled libwebp_anim_encoder && check_pkg_config libwebp_anim_encoder "libwebpmux >= 0.4.0" webp/mux.h WebPAnimEncoderOptionsInit; } enabled libx264 && check_pkg_config libx264 x264 "stdint.h x264.h" x264_encoder_encode && require_cpp_condition libx264 x264.h "X264_BUILD >= 158" enabled libx265 && require_pkg_config libx265 x265 x265.h x265_api_get && require_cpp_condition libx265 x265.h "X265_BUILD >= 70" How would the check for this need to look like when we want to require version 158 only in case of msvc compilation? And what should be the minimum for non-MSVC builds? Keep 118 like it was? Thanks, sw _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".