On Wed, Apr 18, 2018 at 4:57 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote:
> On Wed, Apr 18, 2018 at 4:34 PM, Hendrik Leppkes <h.lepp...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
