[FFmpeg-devel] [PATCH] configure: Update libmysofa check with a new symbol.

[Andrey Semashev]

On 2019-08-28 18:41, James Almer wrote:
> On 8/28/2019 12:24 PM, Andrey Semashev wrote:
>> The current code in libavfilter/af_sofalizer.c requires
>> mysofa_neighborhood_init_withstepdefine function, which only appeared
>> in libmysofa 0.7. Add this function to configure script to bail out
>> early if a too old libmysofa is found in the system instead of failing
>> at compile time.
>> ---
>>   configure | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 51dc77c780..51fb8a7dd0 100755
>> --- a/configure
>> +++ b/configure
>> @@ -6237,8 +6237,8 @@ enabled libmfx            && { check_pkg_config libmfx libmfx "mfx/mfxvideo.h" M
>>                                  { require libmfx "mfx/mfxvideo.h" MFXInit "-llibmfx $advapi32_extralibs" && warn "using libmfx without pkg-config"; } }
>>   enabled libmodplug        && require_pkg_config libmodplug libmodplug libmodplug/modplug.h ModPlug_Load
>>   enabled libmp3lame        && require "libmp3lame >= 3.98.3" lame/lame.h lame_set_VBR_quality -lmp3lame $libm_extralibs
>> -enabled libmysofa         && { check_pkg_config libmysofa libmysofa mysofa.h mysofa_load ||
>> -                               require libmysofa mysofa.h mysofa_load -lmysofa $zlib_extralibs; }
>> +enabled libmysofa         && { check_pkg_config libmysofa libmysofa mysofa.h "mysofa_load mysofa_neighborhood_init_withstepdefine" ||
>> +                               require libmysofa mysofa.h "mysofa_load mysofa_neighborhood_init_withstepdefine" -lmysofa $zlib_extralibs; }
> 
> You could instead make the pkg-config check for "libmysofa >= 0.7",
> assuming the project generated .pc file reports correct versions.

I don't know exactly how the library version will be represented in the 
.pc file (e.g. it may include a revision or some such). There is no .pc 
file for libmysofa in Ubuntu, which is where I'm working. Someone who 
can test the .pc file can add the version check later.

>>   enabled libnpp            && { check_lib libnpp npp.h nppGetLibVersion -lnppig -lnppicc -lnppc -lnppidei ||
>>                                  check_lib libnpp npp.h nppGetLibVersion -lnppi -lnppc -lnppidei ||
>>                                  die "ERROR: libnpp not found"; }
>>
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>