Re: [PATCH 1/4] ppc: Get rid of CPU detection in configure.ac

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

----- Original Message -----
> If we add support for big endian CPUs, this check becomes obsolete, we
> should fix bugs as they are reported on the untested archs.
> ---
>  configure.ac | 18 ------------------
>  1 file changed, 18 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 4e57ef3..69d4991 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -60,24 +60,6 @@ AC_SUBST(SPICE_LT_VERSION)
>  m4_ifndef([AS_VAR_APPEND],
>            AC_DEFUN([AS_VAR_APPEND], $1=$$1$2))
>  
> -# Check for the CPU we are using
> -#
> -AC_MSG_CHECKING(for x86, x86-64 or armv6+ platform)
> -case $host_cpu in
> -  i386|i486|i586|i686|i786|k6|k7|armv6|armv6l|armv6hl|armv7|armv7l|armv7hl)
> -	variant=32
> -        ;;
> -  x86_64)
> -	variant=64
> -	;;
> -  *)
> -	AC_MSG_RESULT(no)
> -	echo Only x86 and x86-64 are supported
> -	exit 1

Perhaps it should be a warning instead of an error? To show that nobody tried it? 

> -esac
> -AC_MSG_RESULT($variant bit)
> -AM_CONDITIONAL([X86_64], [test "$variant" = 64])

This conditional is never used?
> -
>  dnl
>  =========================================================================
>  dnl Check optional features
>  AC_ARG_ENABLE(opengl,
> --
> 2.1.0
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]