From mboxrd@z Thu Jan  1 00:00:00 1970
From: Carlo Marcelo Arenas Belon <carenas@sajinet.com.pe>
Subject: Re: [ANNOUNCE] kvm-71 release
Date: Wed, 16 Jul 2008 01:16:20 -0500
Message-ID: <20080716061620.GA17992@tapir>
References: <487B6A15.10407@qumranet.com> <20080715213556.6ced5503@holly>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Avi Kivity <avi@qumranet.com>,
	"kvm-devel@lists.sourceforge.net" <kvm@vger.kernel.org>
To: Jindrich Makovicka <makovick@gmail.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from tapir.sajinet.com.pe ([66.139.79.212]:42318 "EHLO
	tapir.sajinet.com.pe" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1756085AbYGPFus (ORCPT <rfc822;kvm@vger.kernel.org>);
	Wed, 16 Jul 2008 01:50:48 -0400
Content-Disposition: inline
In-Reply-To: <20080715213556.6ced5503@holly>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Tue, Jul 15, 2008 at 09:35:56PM +0200, Jindrich Makovicka wrote:
> On Mon, 14 Jul 2008 18:00:37 +0300
> Avi Kivity <avi@qumranet.com> wrote:
> 
> > Again, this took longer than I'd like, but the new release is finally 
> > out. It's composed mainly of incremental fixes, but important ones.
> 
> [...]
> 
> I needed to trivially modify configure to be able to build ALSA audio
> 
> (via ./configure --audio-drv-list=alsa)

this will disable the default driver of "oss" and only enable alsa, where
most likely what you want is :

./configure --audio-drv-list="alsa oss"

> Fix attached.

the fix proposed wouldn't allow for a list of drivers passed and is
incorrectly assigning opt to itself and should do instead :

  opt="$optorig"

sadly, the only solution I could come with, violates the current layering
by handling the two list options in kvm instead, at least as a passthrough
that would preserve the parameter count and formatting.

Carlo