From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:53961)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <sw@weilnetz.de>) id 1TCZps-0002oa-Uh
	for qemu-devel@nongnu.org; Fri, 14 Sep 2012 13:37:49 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <sw@weilnetz.de>) id 1TCZpm-0003ej-V4
	for qemu-devel@nongnu.org; Fri, 14 Sep 2012 13:37:48 -0400
Received: from v220110690675601.yourvserver.net ([78.47.199.172]:40380)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <sw@weilnetz.de>) id 1TCZpm-0003eX-Og
	for qemu-devel@nongnu.org; Fri, 14 Sep 2012 13:37:42 -0400
Message-ID: <50536B64.4000403@weilnetz.de>
Date: Fri, 14 Sep 2012 19:37:40 +0200
From: Stefan Weil <sw@weilnetz.de>
MIME-Version: 1.0
References: <20120911185615.GX6223@otherpad.lan.raisama.net>	<1347390153-28682-1-git-send-email-ehabkost@redhat.com>	<CABoDooN2XKUMasOqmS3OjKadi_Xp0upGm9HDJEn0uPg0cHv=2g@mail.gmail.com>
	<CAFEAcA_D=Ydd2zj1Bnd_dvj3mbaYdWqLojnzhPbpyuumfWCWTw@mail.gmail.com>
In-Reply-To: <CAFEAcA_D=Ydd2zj1Bnd_dvj3mbaYdWqLojnzhPbpyuumfWCWTw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH] configure: fix "--target-list=<target>,
 <target>, ..." option
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Laurent Desnogues <laurent.desnogues@gmail.com>, Eduardo Habkost <ehabkost@redhat.com>, Anthony Liguori <anthony@codemonkey.ws>, qemu-devel@nongnu.org

Am 14.09.2012 14:53, schrieb Peter Maydell:
> On 12 September 2012 14:20, Laurent Desnogues
> <laurent.desnogues@gmail.com>  wrote:
>> Sorry, I had missed this patch...
>>
>> On Tue, Sep 11, 2012 at 9:02 PM, Eduardo Habkost<ehabkost@redhat.com>  wrote:
>>> commit 66d5499b3754b83c09487259c08fe2ce73188a59 broke the support for
>>> comma-separated target lists on the --target-list option. e.g.:
>>>
>>>    $ ./configure --target-list=x86_64-linux-user,x86_64-softmmu
>>>    [...]
>>>    ERROR: Target 'x86_64-linux-user,x86_64-softmmu' not recognised
>>>    $
>>>
>>> This patch restores that ability.
>>>
>>> Signed-off-by: Eduardo Habkost<ehabkost@redhat.com>
>>> Cc: Daniel P. Berrange<berrange@redhat.com>
>>> Cc: Anthony Liguori<anthony@codemonkey.ws>
>>> ---
>>>   configure | 4 +++-
>>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/configure b/configure
>>> index 7656c32..9ee7038 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -1323,7 +1323,9 @@ if ! "$python" -c 'import sys; sys.exit(sys.version_info<  (2,4) or sys.version_
>>>   fi
>>>
>>>   if test "$target_list" = "DEFAULT" ; then
>>> -    target_list=`echo "$default_target_list" | sed -e 's/,/ /g'`
>>> +    target_list="$default_target_list"
>>> +else
>>> +    target_list=`echo "$target_list" | sed -e 's/,/ /g'`
>>>   fi
>>
>> This works for me too.
>>
>> But I still can't get what the original patch posted by
>> Daniel Berrange intended to do:
>>
>> $ ./configure --target-list=
>> $ make V=1
>> cat  | grep =y | sort -u>  config-all-devices.mak
>>
>> And it of course hangs there.
>
> Hmm. Perhaps we should just revert 66d5499b3 and then recommit
> a working implementation later?
>
> -- PMM


make hangs while waiting for input on stdin:
if there is no emulation target, 'cat' is called without arguments.

I have sent a patch which fixes this. It can be applied after
reverting 66d5499b3 (which is not needed to get builds
without emulation targets).

Regards
Stefan W.