On 17 February 2012 08:22, Aneesh Kumar K.V
<aneesh.ku...@linux.vnet.ibm.com> wrote:
> On Thu, 9 Feb 2012 20:31:17 -0600, Meador Inge <mead...@codesourcery.com> 
> wrote:
>> --- a/configure
>> +++ b/configure
>> @@ -121,6 +121,7 @@ docs=""
>>  fdt=""
>>  nptl=""
>>  sdl=""
>> +virtfs=""
>
> do we want to say
>
> virtfs="yes"

No. The standard pattern for configure feature options should be:
 * default is to probe and use if present, not use if not present
 * --enable-foo should probe and fail configure if not present
 * --disable-foo should not probe and not use

See the discussion on this point on v1 of this patch:
 http://lists.gnu.org/archive/html/qemu-devel/2012-02/msg01211.html

> That would make it explicit like vnc below.

Some of our legacy options don't follow the standard pattern, alas.

-- PMM

Reply via email to