* [Qemu-devel] ui causes latest master build failure on rhel6
@ 2014-09-11 14:59 Michael S. Tsirkin
2014-09-11 15:21 ` Gerd Hoffmann
2014-09-11 15:23 ` Gerd Hoffmann
0 siblings, 2 replies; 7+ messages in thread
From: Michael S. Tsirkin @ 2014-09-11 14:59 UTC (permalink / raw)
To: kraxel, qemu-devel
I observe two errors with latest master:
ui/qemu-pixman.c: In function ‘qemu_pixelformat_from_pixman’:
ui/qemu-pixman.c:42: error: ‘PIXMAN_TYPE_RGBA’ undeclared (first use in this function)
ui/qemu-pixman.c:42: error: (Each undeclared identifier is reported only once
ui/qemu-pixman.c:42: error: for each function it appears in.)
make: *** [ui/qemu-pixman.o] Error 1
Also:
In file included from /usr/include/gtk-2.0/gtk/gtk.h:228,
from ui/gtk.c:48:
/usr/include/gtk-2.0/gtk/gtkitemfactory.h:47: warning: function
declaration isn’t a prototype
Note: 2.1.1 builds fine.
--
MST
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-11 14:59 [Qemu-devel] ui causes latest master build failure on rhel6 Michael S. Tsirkin
@ 2014-09-11 15:21 ` Gerd Hoffmann
2014-09-11 18:17 ` Michael S. Tsirkin
2014-09-12 14:58 ` Peter Maydell
2014-09-11 15:23 ` Gerd Hoffmann
1 sibling, 2 replies; 7+ messages in thread
From: Gerd Hoffmann @ 2014-09-11 15:21 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: qemu-devel
On Do, 2014-09-11 at 17:59 +0300, Michael S. Tsirkin wrote:
> I observe two errors with latest master:
>
> ui/qemu-pixman.c: In function ‘qemu_pixelformat_from_pixman’:
> ui/qemu-pixman.c:42: error: ‘PIXMAN_TYPE_RGBA’ undeclared (first use in this function)
> ui/qemu-pixman.c:42: error: (Each undeclared identifier is reported only once
> ui/qemu-pixman.c:42: error: for each function it appears in.)
> make: *** [ui/qemu-pixman.o] Error 1
Guess we need to raise the minimum pixman version in configure ...
pixman-0.21.8 is the oldest release which has PIXMAN_TYPE_RGBA.
We don't check for a minimum version.
RHEL-6 is probably older.
Try 'configure --without-system-pixman' as workaround (after checking
out the pixman submodule).
I'll cook up a patch,
Gerd
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-11 14:59 [Qemu-devel] ui causes latest master build failure on rhel6 Michael S. Tsirkin
2014-09-11 15:21 ` Gerd Hoffmann
@ 2014-09-11 15:23 ` Gerd Hoffmann
1 sibling, 0 replies; 7+ messages in thread
From: Gerd Hoffmann @ 2014-09-11 15:23 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: qemu-devel
Hi,
> In file included from /usr/include/gtk-2.0/gtk/gtk.h:228,
> from ui/gtk.c:48:
> /usr/include/gtk-2.0/gtk/gtkitemfactory.h:47: warning: function
> declaration isn’t a prototype
That isn't new. Bug in the gtk2 header files. gtk3 has it fixed, but
gtk2.x seems to stay bug compatible forever ...
Maybe we should throw in a #pragma to turn off that warning in gtk.c ...
cheers,
Gerd
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-11 15:21 ` Gerd Hoffmann
@ 2014-09-11 18:17 ` Michael S. Tsirkin
2014-09-12 6:32 ` Gerd Hoffmann
2014-09-12 14:58 ` Peter Maydell
1 sibling, 1 reply; 7+ messages in thread
From: Michael S. Tsirkin @ 2014-09-11 18:17 UTC (permalink / raw)
To: Gerd Hoffmann; +Cc: qemu-devel
On Thu, Sep 11, 2014 at 05:21:13PM +0200, Gerd Hoffmann wrote:
> On Do, 2014-09-11 at 17:59 +0300, Michael S. Tsirkin wrote:
> > I observe two errors with latest master:
> >
> > ui/qemu-pixman.c: In function ‘qemu_pixelformat_from_pixman’:
> > ui/qemu-pixman.c:42: error: ‘PIXMAN_TYPE_RGBA’ undeclared (first use in this function)
> > ui/qemu-pixman.c:42: error: (Each undeclared identifier is reported only once
> > ui/qemu-pixman.c:42: error: for each function it appears in.)
> > make: *** [ui/qemu-pixman.o] Error 1
>
> Guess we need to raise the minimum pixman version in configure ...
>
> pixman-0.21.8 is the oldest release which has PIXMAN_TYPE_RGBA.
> We don't check for a minimum version.
> RHEL-6 is probably older.
>
> Try 'configure --without-system-pixman' as workaround (after checking
> out the pixman submodule).
I guess but is it a must?
We used to have this within an ifdef:
+#if PIXMAN_VERSION >= PIXMAN_VERSION_ENCODE(0, 21, 8)
+ type = PIXMAN_TYPE_RGBA;
+#endif
+ }
can't we keep system pixman working a bit longer?
> I'll cook up a patch,
> Gerd
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-11 18:17 ` Michael S. Tsirkin
@ 2014-09-12 6:32 ` Gerd Hoffmann
0 siblings, 0 replies; 7+ messages in thread
From: Gerd Hoffmann @ 2014-09-12 6:32 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: qemu-devel
> > pixman-0.21.8 is the oldest release which has PIXMAN_TYPE_RGBA.
> > We don't check for a minimum version.
> > RHEL-6 is probably older.
Uhm, well, it isn't (any more):
[root@rhel6 ~]# rpm -q pixman
pixman-0.26.2-5.1.el6_5.x86_64
Seems to be rebased somewhen (used to be 0.16 for RHEL-6.0 IIRC).
Looks like you should simply update your RHEL-6 machine.
> > Try 'configure --without-system-pixman' as workaround (after checking
> > out the pixman submodule).
>
> I guess but is it a must?
> We used to have this within an ifdef:
> +#if PIXMAN_VERSION >= PIXMAN_VERSION_ENCODE(0, 21, 8)
> + type = PIXMAN_TYPE_RGBA;
> +#endif
> + }
That ifdef is still there, we got new PIXMAN_TYPE_RGBA references
though.
> can't we keep system pixman working a bit longer?
I'd much prefer to raise the minimum pixman version and ditch the
#ifdefs. The common kvm case will work fine without RGBA, you may run
into trouble though when emulating bigendian guests on a little endian
host (or visa versa), because pixman will have to deal with uncommon
pixel formats then.
cheers,
Gerd
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-11 15:21 ` Gerd Hoffmann
2014-09-11 18:17 ` Michael S. Tsirkin
@ 2014-09-12 14:58 ` Peter Maydell
2014-09-15 6:06 ` Gerd Hoffmann
1 sibling, 1 reply; 7+ messages in thread
From: Peter Maydell @ 2014-09-12 14:58 UTC (permalink / raw)
To: Gerd Hoffmann; +Cc: QEMU Developers, Michael S. Tsirkin
On 11 September 2014 16:21, Gerd Hoffmann <kraxel@redhat.com> wrote:
> pixman-0.21.8 is the oldest release which has PIXMAN_TYPE_RGBA.
> We don't check for a minimum version.
> RHEL-6 is probably older.
Ubuntu LTS hits this too.
> Try 'configure --without-system-pixman' as workaround (after checking
> out the pixman submodule).
The version of pixman we have in our submodule doesn't
have TYPE_RGBA either as far as I can tell...
> I'll cook up a patch,
That would be nice :-)
thanks
-- PMM
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] ui causes latest master build failure on rhel6
2014-09-12 14:58 ` Peter Maydell
@ 2014-09-15 6:06 ` Gerd Hoffmann
0 siblings, 0 replies; 7+ messages in thread
From: Gerd Hoffmann @ 2014-09-15 6:06 UTC (permalink / raw)
To: Peter Maydell; +Cc: QEMU Developers, Michael S. Tsirkin
On Fr, 2014-09-12 at 15:58 +0100, Peter Maydell wrote:
> On 11 September 2014 16:21, Gerd Hoffmann <kraxel@redhat.com> wrote:
> > pixman-0.21.8 is the oldest release which has PIXMAN_TYPE_RGBA.
> > We don't check for a minimum version.
> > RHEL-6 is probably older.
>
> Ubuntu LTS hits this too.
>
> > Try 'configure --without-system-pixman' as workaround (after checking
> > out the pixman submodule).
>
> The version of pixman we have in our submodule doesn't
> have TYPE_RGBA either as far as I can tell...
Oops, indeed. There was a windows build fix in pixman somewhere around
0.26, I thougth we've upgraded to that version, but apparently not, in
fact we are still at 0.18.x ...
cheers,
Gerd
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-09-15 6:07 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-11 14:59 [Qemu-devel] ui causes latest master build failure on rhel6 Michael S. Tsirkin
2014-09-11 15:21 ` Gerd Hoffmann
2014-09-11 18:17 ` Michael S. Tsirkin
2014-09-12 6:32 ` Gerd Hoffmann
2014-09-12 14:58 ` Peter Maydell
2014-09-15 6:06 ` Gerd Hoffmann
2014-09-11 15:23 ` Gerd Hoffmann
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.