[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] tools/configure: Check if pixman is present on the system



On 14/01/15 12:31, Ian Campbell wrote:
> On Wed, 2015-01-14 at 12:28 +0000, Julien Grall wrote:
>> On 14/01/15 12:24, Ian Campbell wrote:
>>> On Wed, 2015-01-14 at 12:13 +0000, Julien Grall wrote:
>>>> Hi Jan,
>>>>
>>>> On 14/01/15 07:11, Jan Beulich wrote:
>>>>>>>> Julien Grall <julien.grall@xxxxxxxxxx> 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?
>>>>
>>>> QEMU is a required part of building the tools.
>>>
>>> Not stricty true. Pass --with-system-qemu if you wish to avoid it.
>>
>> Right. So the check for glib in the configure for ARM is not necessary
>> when --with-system-qemu is used.
>>
>> Might be interesting to avoid checking dependency that is not useful.
> 
> I didn't think it was worth it, but given the objections perhaps putting
> all the checks which are there solely for qemu-upstream under a
> conditional based on --with-system-qemu is the way to go.

On x86, glib seems to be required for seabios and ipxe.

Regards,

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.