[libvirt] [PATCH] build: ask for -std=gnu99 explicitly

Fabiano Fidêncio fidencio at redhat.com
Mon Oct 7 14:23:06 UTC 2019


On Mon, Oct 7, 2019 at 4:16 PM Daniel P. Berrangé <berrange at redhat.com> wrote:
>
> We previously got -std=gnu99 secretly enabled as a side-effect
> of requesting the 'stdarg' gnulib module. We rely on some
> extensions from c99/gnu99 and while RHEL-7 supports this, it
> still defaults to gnu89.  RHEL-7 also supports some newer
> standards but declares them experimental/incomplete, so sticking
> with gnu99 is best bet for now & matches historical usage.
>
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
>  m4/virt-compile-warnings.m4 | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4
> index f9460e82ba..502f7384ff 100644
> --- a/m4/virt-compile-warnings.m4
> +++ b/m4/virt-compile-warnings.m4
> @@ -189,6 +189,10 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[
>        wantwarn="$wantwarn -Werror"
>      fi
>
> +    # Request the gnu99 standard which is the best choice with
> +    # gcc 4.8.0
> +    wantwarn="-std=gnu99"
> +
>      # Check for $CC support of each warning
>      for w in $wantwarn; do
>        gl_WARN_ADD([$w])
> --
> 2.21.0
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

Reviewed-by: Fabiano Fidêncio <fidencio at redhat.com>




More information about the libvir-list mailing list