[FFmpeg-devel] [PATCH] configure: fix clang-cl check in the MSVC section

Hendrik Leppkes h.leppkes at gmail.com
Thu Apr 19 10:59:52 EEST 2018


On Wed, Apr 18, 2018 at 4:57 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
> On Wed, Apr 18, 2018 at 4:34 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
>> Without properly grouping the checks, the second test would execute for
>> MSVC cl.exe, which results in configure getting stuck since cl.exe -? is
>> an interactive paginated help screen, waiting for input.
>> ---
>>  configure | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 535a59b737..0074f36eba 100755
>> --- a/configure
>> +++ b/configure
>> @@ -4411,7 +4411,7 @@ probe_cc(){
>>          _flags_filter=msvc_flags
>>          _ld_lib='lib%.a'
>>          _ld_path='-libpath:'
>> -    elif $_cc -nologo- 2>&1 | grep -q Microsoft || $_cc -v 2>&1 | grep -q clang && $_cc -? > /dev/null 2>&1; then
>> +    elif $_cc -nologo- 2>&1 | grep -q Microsoft || { $_cc -v 2>&1 | grep -q clang && $_cc -? > /dev/null 2>&1; } then
>
> There was some concern that some pedantic shells may require a
> semicolon after the closing brace even if bash doesn't, so I have
> added that locally.
>

Applied.

- Hendrik


More information about the ffmpeg-devel mailing list