[FFmpeg-devel] [PATCH] When checking for compiler flags, check if warnings are reported.
Måns Rullgård
mans
Thu Oct 2 14:15:32 CEST 2008
Diego 'Flameeyes' Petten? wrote:
> M?ns Rullg?rd <mans at mansr.com> writes:
>
>> Diego 'Flameeyes' Petten? wrote:
>>> check_cmd(){
>>> log "$@"
>>> - "$@" >> $logfile 2>&1
>>> + rm -f $TMPRES
>>> + ( "$@" 2>&1 && touch $TMPRES; ) | tee $TMPLOG >> $logfile
>>> + test -f $TMPRES
>>
>> I don't like the look of this. I'm sure there's a better way, but I
>> have to think about it.
>
> Yes I sincerely don't find it the nicest approach, but again I have no
> better way to handle this just yet; if you can find one I'd be very
> pleased ;)
>
>> Just set LC_ALL=C at the start of configure instead. I've been meaning
>> to do that anyway.
>
> Probably a very good idea.
Feel free to send a patch.
>>
>>> check_cflags(){
>>> log check_cflags "$@"
>>> - check_cc "$@" <<EOF && add_cflags "$@"
>>> + check_cc "$@" <<EOF || return
>>> int x;
>>> EOF
>>> +
>>> + grep -qi "warning" $TMPLOG && return
>>> + grep -qi "illegal option" $TMPLOG && return
>>> + grep -qi "unrecognized" $TMPLOG && return
>>> +
>>> + add_cflags "$@"
>>> }
>>
>> This has two problems. Firstly, the return value from the function is
>> wrong.
>
> Good point, will fix it.
>
>> Secondly, warnings are expected, or at least accepted, in a few
>> of the tests (I don't recall which ones), so this is overly strict.
>
> Hmm, on cflags tests you mean? I do expect warnings to be excepted and
> accepted for most check_cc checks, but for flags checking?
>
> I made sure the warnings are checked for _only_ during flags checking
> because I did expect warnings in other case, but I wouldn't expect extra
> warnings while checking for flags support, sincerely, can you give me
> any example of a warning accepted during flags checking?
You're probably right, there should be no warnings in pure flag checks.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list