mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] host-qemu: use internel pixman instead of systemwide
@ 2016-08-21 18:49 Robert Schwebel
  2016-08-21 18:52 ` Robert Schwebel
  0 siblings, 1 reply; 3+ messages in thread
From: Robert Schwebel @ 2016-08-21 18:49 UTC (permalink / raw)
  To: ptxdist; +Cc: Robert Schwebel

Currently, host-qemu tries to use a systemwide installed pixman version.
QEMU's handwritten configure script determines the include and library
paths with pkg-config, but as our host-pkg-config doesn't find pixman-1,
the compilation breaks down with:

In file included from [...]/qemu-2.6.0/include/ui/console.h:4:0, from vl.c:75:
[...]/qemu-2.6.0/include/ui/qemu-pixman.h:14:20: fatal error: pixman.h: No such file or directory
compilation terminated.
[...]/qemu-2.6.0/rules.mak:57: recipe for target 'vl.o' failed
make[1]: *** [vl.o] Error 1

Switch to using the pixman variant shipped with qemu.

Signed-off-by: Robert Schwebel <r.schwebel@pengutronix.de>
---
 rules/host-qemu.make | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rules/host-qemu.make b/rules/host-qemu.make
index eae6a52..ab47c1a 100644
--- a/rules/host-qemu.make
+++ b/rules/host-qemu.make
@@ -121,7 +121,7 @@ HOST_QEMU_CONF_OPT	:= \
 	--disable-jemalloc \
 	--disable-tools \
 	\
-	--with-system-pixman
+	--without-system-pixman
 
 QEMU_CROSS_DL = $(shell ptxd_cross_cc_v |sed -n -e 's/.* -dynamic-linker \([^ ]*\).*/\1/p')
 QEMU_CROSS_LD_LIBRARY_PATH := $(PTXDIST_SYSROOT_TOOLCHAIN)/lib:$(SYSROOT)/$(CROSS_LIB_DIR):$(SYSROOT)/usr/$(CROSS_LIB_DIR)
-- 
2.8.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [ptxdist] [PATCH] host-qemu: use internel pixman instead of systemwide
  2016-08-21 18:49 [ptxdist] [PATCH] host-qemu: use internel pixman instead of systemwide Robert Schwebel
@ 2016-08-21 18:52 ` Robert Schwebel
  2016-08-22  8:13   ` Robert Schwebel
  0 siblings, 1 reply; 3+ messages in thread
From: Robert Schwebel @ 2016-08-21 18:52 UTC (permalink / raw)
  To: ptxdist

Hi,

On Sun, Aug 21, 2016 at 08:49:39PM +0200, Robert Schwebel wrote:
> Currently, host-qemu tries to use a systemwide installed pixman version.
> QEMU's handwritten configure script determines the include and library
> paths with pkg-config, but as our host-pkg-config doesn't find pixman-1,
> the compilation breaks down with:
> 
> In file included from [...]/qemu-2.6.0/include/ui/console.h:4:0, from vl.c:75:
> [...]/qemu-2.6.0/include/ui/qemu-pixman.h:14:20: fatal error: pixman.h: No such file or directory
> compilation terminated.
> [...]/qemu-2.6.0/rules.mak:57: recipe for target 'vl.o' failed
> make[1]: *** [vl.o] Error 1
> 
> Switch to using the pixman variant shipped with qemu.

I'm not entirely sure if the reasoning is right, but at least qemu
doesn't find pixman.h, even if it is installed. So any feedback about
this is welcome.

The OSELAS.BSP-Pengutronix-AllYes test BSP doesn't have host-qemu
enabled, so I assume this is the reason our sisiphus autobuilder doesn't
find it.

rsc
-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [ptxdist] [PATCH] host-qemu: use internel pixman instead of systemwide
  2016-08-21 18:52 ` Robert Schwebel
@ 2016-08-22  8:13   ` Robert Schwebel
  0 siblings, 0 replies; 3+ messages in thread
From: Robert Schwebel @ 2016-08-22  8:13 UTC (permalink / raw)
  To: ptxdist

On Sun, Aug 21, 2016 at 08:52:27PM +0200, Robert Schwebel wrote:
> On Sun, Aug 21, 2016 at 08:49:39PM +0200, Robert Schwebel wrote:
> > Currently, host-qemu tries to use a systemwide installed pixman version.
> > QEMU's handwritten configure script determines the include and library
> > paths with pkg-config, but as our host-pkg-config doesn't find pixman-1,
> > the compilation breaks down with:
> > 
> > In file included from [...]/qemu-2.6.0/include/ui/console.h:4:0, from vl.c:75:
> > [...]/qemu-2.6.0/include/ui/qemu-pixman.h:14:20: fatal error: pixman.h: No such file or directory
> > compilation terminated.
> > [...]/qemu-2.6.0/rules.mak:57: recipe for target 'vl.o' failed
> > make[1]: *** [vl.o] Error 1
> > 
> > Switch to using the pixman variant shipped with qemu.
> 
> I'm not entirely sure if the reasoning is right, but at least qemu
> doesn't find pixman.h, even if it is installed. So any feedback about
> this is welcome.
> 
> The OSELAS.BSP-Pengutronix-AllYes test BSP doesn't have host-qemu
> enabled, so I assume this is the reason our sisiphus autobuilder doesn't
> find it.

Discussed with mol and found the reson: the new host-qemu.in has a
select to host-pixman. If we use the upstream .in file as well, the
issue is fixed.

Please drop this patch.

rsc
-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-08-22  8:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-21 18:49 [ptxdist] [PATCH] host-qemu: use internel pixman instead of systemwide Robert Schwebel
2016-08-21 18:52 ` Robert Schwebel
2016-08-22  8:13   ` Robert Schwebel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox