On 10/2/2017 6:48 AM, Diego Biurrun wrote:
> On Fri, Sep 29, 2017 at 10:56:09PM -0300, James Almer wrote:
>> --- a/configure
>> +++ b/configure
>> @@ -4873,10 +4873,7 @@ fi
>>  
>>  # SDL is "special" and adds some CFLAGS that should not pollute anything 
>> else.
>>  if enabled avplay; then
>> -    CFLAGS_SAVE=$CFLAGS
>> -    check_pkg_config sdl "sdl >= 1.2.1 sdl < 1.3.0" SDL_events.h 
>> SDL_PollEvent &&
>> -    sdl_cflags=$pkg_cflags
>> -    CFLAGS=$CFLAGS_SAVE
>> +    test_pkg_config sdl "sdl >= 1.2.1 sdl < 1.3.0" SDL_events.h 
>> SDL_PollEvent
>>  fi
> 
> Uh, no. You're adding the CFLAGS globally. That's the very thing this,
> admittedly, hack tries to prevent.

No. I'm using test_pkg_config(), added in the last patch, which will set
sdl_cflags but not affect global CFLAGS.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to