Re: [Qemu-devel] [PATCH 2/2] pixman: fix warning

[Gerd Hoffmann <kraxel@redhat.com>]

  Hi,

> Unfortunately this did not help on SLES11 SP2. I don't spot any typo in
> your patches, it looked neat; CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE does
> not get defined in config-host.h it seems.

Hmm, anything in config.log?  Maybe the sles11 gcc doesn't support the
-Wunused-but-set-variable used in the configure test?

> Using the pragmas unconditionally results in change from:
> 
>   CC    qemu-timer.o
> cc1: warnings being treated as errors
> In file included from
> /home/andreas/QEMU/qemu-s390/include/ui/qemu-pixman.h:13,
>                  from /home/andreas/QEMU/qemu-s390/include/ui/console.h:4,
>                  from /home/andreas/QEMU/qemu-s390/qemu-timer.c:27:
> /usr/include/pixman-1/pixman.h:225: error: redundant redeclaration of
> ‘pixman_transform_from_pixman_f_transform’
> /usr/include/pixman-1/pixman.h:221: error: previous declaration of
> ‘pixman_transform_from_pixman_f_transform’ was here
> make: *** [qemu-timer.o] Fehler 1
> 
> to:
> 
>   CC    qemu-timer.o
> cc1: warnings being treated as errors
> In file included from
> /home/andreas/QEMU/qemu-s390/include/ui/qemu-pixman.h:13,
>                  from /home/andreas/QEMU/qemu-s390/include/ui/console.h:4,
>                  from /home/andreas/QEMU/qemu-s390/qemu-timer.c:27:
> /usr/include/pixman-1/pixman.h:221: error: previous declaration of
> ‘pixman_transform_from_pixman_f_transform’ was here
> make: *** [qemu-timer.o] Fehler 1

That looks like a gcc bug to me.

cheers,
  Gerd