qemu-devel
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH COLO-Frame v10 01/38] configure: Add parameter f

From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH COLO-Frame v10 01/38] configure: Add parameter for configure to enable/disable COLO support
Date: Thu, 5 Nov 2015 07:52:59 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 
On 11/03/2015 04:56 AM, zhanghailiang wrote:
> configure --enable-colo/--disable-colo to switch COLO
> support on/off.
> COLO support is off by default.

Off by default risks bit-rot for people not building it; it's generally
best to default to off only if the feature requires dragging in extra
libraries or similar build items not likely to be present in all
development environments (and even then, auto-probing for required
prerequisites is nicer than hard-coding off). Would on-by-default be a
saner choice, or would that drag in too many extra dependencies for
building?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to
[Prev in Thread] Current Thread [Next in Thread]