Re: [PATCH] qemu-kvm: fix typo in configure

[Dustin Kirkland <kirkland@canonical.com>]

On Wed, Jul 1, 2009 at 4:27 AM, Michael S. Tsirkin<mst@redhat.com> wrote:
> Kill extra \). Also escape ! for clarity and bourne shell
> compatibility.

Agreed.  Looks like this one was introduced during the last merge from
qemu.  It bit me too.

:-Dustin

> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> ---
>
> This patch fixes configure on next for me
>
>  configure |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index f76f511..b62e3d7 100755
> --- a/configure
> +++ b/configure
> @@ -2141,10 +2141,10 @@ configure_kvm() {
>  }
>
>  # Make sure the target and host cpus are compatible
> -if test ! \( "$target_cpu" = "$cpu" -o \
> +if test \! \( "$target_cpu" = "$cpu" -o \
>   \( "$target_cpu" = "ppcemb" -a "$cpu" = "ppc" \) -o \
> -  \( "$target_cpu" = "x86_64" -a "$cpu" = "i386"   \) -o \
> -  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) -o \
> +  \( "$target_cpu" = "x86_64" -a "$cpu" = "i386" \) -o \
> +  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) -o \
>   \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then
>   target_kvm="no"
>  fi