From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] tools/configure: Check if pixman is present on the system Date: Wed, 14 Jan 2015 10:39:45 +0000 Message-ID: <1421231985.19103.200.camel@citrix.com> References: <1421172890-29763-1-git-send-email-julien.grall@linaro.org> <54B6168602000078000C581A@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <54B6168602000078000C581A@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: ian.jackson@eu.citrix.com, julien.grall@linaro.org, wei.liu2@citrix.com, stefano.stabellini@eu.citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, 2015-01-14 at 07:11 +0000, Jan Beulich wrote: > >>> Julien Grall 01/13/15 7:17 PM >>> > >QEMU upstream requires the use of pixman. When pixman is not present the > >system, the configure of QEMU will fail with: > > > >ERROR: pixman not present. Your options: > >(1) Preferred: Install the pixman devel package (any recent > >distro should have packages as Xorg needs pixman too). > >(2) Fetch the pixman submodule, using: > >git submodule update --init pixman > > > >I think we can use by default the version on the system. So check it > >a Xen configuration time to avoid a build issue later. > > Since qemu building isn't a required part of building the tools, is a check in > the tools configure process really an appropriate thing to do? It's quite annoying as a user to have configure appear to succeed and then the build to fail due to a missing dependency (whether in a 3rd party component or not). We could make this check conditional on --with-qemu-system not being given, but given the library is readily available I don't think its the end of the world to have a potentially unnecessary check. A far more heavy weight (but probably better) solution would be to arrange for the clone (and necessary downloads) + submodule configure to happen at configure rather than build time. That's a much bigger job though and I wouldn't expect anyone to take that on unless they are super duper keen. Ian.