Re: [Qemu-devel] [PATCH] configure: fix "--target-list=<target>, <target>, ..." option

[Peter Crosthwaite]

On Wed, Sep 12, 2012 at 5:02 AM, Eduardo Habkost <address@hidden> 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
>   $
>

Broke for me too. This patch fixes it.

> This patch restores that ability.
>
> Signed-off-by: Eduardo Habkost <address@hidden>

Tested-by: Peter Crosthwaite <address@hidden>

> Cc: Daniel P. Berrange <address@hidden>
> Cc: Anthony Liguori <address@hidden>
> ---
>  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
>
>  # see if system emulation was really requested
> --
> 1.7.11.4
>
>