[virglrenderer-devel] [PATCH 8/8] build: enable some more warnings by default.

[Emil Velikov]

On 21 May 2018 at 11:04, Tomeu Vizoso <tomeu.vizoso at collabora.com> wrote:
> On 05/21/2018 05:07 AM, Dave Airlie wrote:
>>
>> From: Dave Airlie <airlied at redhat.com>
>>
>> ---
>>   build-aux/ax_check_compile_flag.m4 | 74
>> ++++++++++++++++++++++++++++++++++++++
>>   configure.ac                       |  5 +++
>>   2 files changed, 79 insertions(+)
>>   create mode 100644 build-aux/ax_check_compile_flag.m4
>>
>> diff --git a/build-aux/ax_check_compile_flag.m4
>> b/build-aux/ax_check_compile_flag.m4
>> new file mode 100644
>> index 0000000..51df0c0
>> --- /dev/null
>> +++ b/build-aux/ax_check_compile_flag.m4
>> @@ -0,0 +1,74 @@
>> +#
>> ===========================================================================
>> +#
>> http://www.gnu.org/software/autoconf-archive/ax_check_compile_flag.html
>> +#
>> ===========================================================================
>> +#
>> +# SYNOPSIS
>> +#
>> +#   AX_CHECK_COMPILE_FLAG(FLAG, [ACTION-SUCCESS], [ACTION-FAILURE],
>> [EXTRA-FLAGS], [INPUT])
>> +#
>> +# DESCRIPTION
>> +#
>> +#   Check whether the given FLAG works with the current language's
>> compiler
>> +#   or gives an error.  (Warnings, however, are ignored)
>> +#
>> +#   ACTION-SUCCESS/ACTION-FAILURE are shell commands to execute on
>> +#   success/failure.
>> +#
>> +#   If EXTRA-FLAGS is defined, it is added to the current language's
>> default
>> +#   flags (e.g. CFLAGS) when the check is done.  The check is thus made
>> with
>> +#   the flags: "CFLAGS EXTRA-FLAGS FLAG".  This can for example be used
>> to
>> +#   force the compiler to issue an error when a bad flag is given.
>> +#
>> +#   INPUT gives an alternative input source to AC_COMPILE_IFELSE.
>> +#
>> +#   NOTE: Implementation based on AX_CFLAGS_GCC_OPTION. Please keep this
>> +#   macro in sync with AX_CHECK_{PREPROC,LINK}_FLAG.
>> +#
>> +# LICENSE
>> +#
>> +#   Copyright (c) 2008 Guido U. Draheim <guidod at gmx.de>
>> +#   Copyright (c) 2011 Maarten Bosmans <mkbosmans at gmail.com>
>> +#
>> +#   This program is free software: you can redistribute it and/or modify
>> it
>> +#   under the terms of the GNU General Public License as published by the
>> +#   Free Software Foundation, either version 3 of the License, or (at
>> your
>> +#   option) any later version.
>> +#
>> +#   This program is distributed in the hope that it will be useful, but
>> +#   WITHOUT ANY WARRANTY; without even the implied warranty of
>> +#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> General
>> +#   Public License for more details.
>> +#
>> +#   You should have received a copy of the GNU General Public License
>> along
>> +#   with this program. If not, see <http://www.gnu.org/licenses/>.
>> +#
>> +#   As a special exception, the respective Autoconf Macro's copyright
>> owner
>> +#   gives unlimited permission to copy, distribute and modify the
>> configure
>> +#   scripts that are the output of Autoconf when processing the Macro.
>> You
>> +#   need not follow the terms of the GNU General Public License when
>> using
>> +#   or distributing such scripts, even though portions of the text of the
>> +#   Macro appear in them. The GNU General Public License (GPL) does
>> govern
>> +#   all other use of the material that constitutes the Autoconf Macro.
>> +#
>> +#   This special exception to the GPL applies to versions of the Autoconf
>> +#   Macro released by the Autoconf Archive. When you make and distribute
>> a
>> +#   modified version of the Autoconf Macro, you may extend this special
>> +#   exception to the GPL to apply to your modified version as well.
>> +
>> +#serial 3
>
>
> Any reason to not copy in the latest version (5)?
>
The following patch says it perfectly.
https://lists.freedesktop.org/archives/mesa-dev/2017-August/168262.html

-Emil