diff mbox

[3/3] build: check if libm is needed in configure

Message ID 1329886533-4435-3-git-send-email-roger.pau@entel.upc.edu
State New
Headers show

Commit Message

Roger Pau Monne Feb. 22, 2012, 4:55 a.m. UTC
Remove the hardcoded use of libm and instead rely on configure to
check for it. It is needed at least for qemu-ga and qemu-system.

Signed-off-by: Roger Pau Monne <roger.pau@entel.upc.edu>
---
 Makefile.target |    4 ----
 configure       |   14 ++++++++++++++
 2 files changed, 14 insertions(+), 4 deletions(-)

Comments

Blue Swirl March 17, 2012, 1:04 p.m. UTC | #1
On Wed, Feb 22, 2012 at 04:55, Roger Pau Monne <roger.pau@entel.upc.edu> wrote:
> Remove the hardcoded use of libm and instead rely on configure to
> check for it. It is needed at least for qemu-ga and qemu-system.

This would break linux-user build, linker can't find 'sin' etc.

>
> Signed-off-by: Roger Pau Monne <roger.pau@entel.upc.edu>
> ---
>  Makefile.target |    4 ----
>  configure       |   14 ++++++++++++++
>  2 files changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/Makefile.target b/Makefile.target
> index 68a5641..c230aff 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -42,10 +42,6 @@ PROGS+=$(QEMU_PROGW)
>  endif
>  STPFILES=
>
> -ifndef CONFIG_HAIKU
> -LIBS+=-lm
> -endif
> -
>  config-target.h: config-target.h-timestamp
>  config-target.h-timestamp: config-target.mak
>
> diff --git a/configure b/configure
> index 790d495..b0cb175 100755
> --- a/configure
> +++ b/configure
> @@ -2524,6 +2524,20 @@ elif compile_prog "" "-lrt" ; then
>   libs_qga="-lrt $libs_qga"
>  fi
>
> +##########################################
> +# Do we need libm
> +cat > $TMPC <<EOF
> +#include <math.h>
> +int main(void) { double a, b; return modf(a, &b);}
> +EOF
> +
> +if compile_prog "" "" ; then
> +  :
> +elif compile_prog "" "-lm" ; then
> +  LIBS="-lm $LIBS"
> +  libs_qga="-lm $libs_qga"
> +fi
> +
>  if test "$darwin" != "yes" -a "$mingw32" != "yes" -a "$solaris" != yes -a \
>         "$aix" != "yes" -a "$haiku" != "yes" ; then
>     libs_softmmu="-lutil $libs_softmmu"
> --
> 1.7.9
>
>
diff mbox

Patch

diff --git a/Makefile.target b/Makefile.target
index 68a5641..c230aff 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -42,10 +42,6 @@  PROGS+=$(QEMU_PROGW)
 endif
 STPFILES=
 
-ifndef CONFIG_HAIKU
-LIBS+=-lm
-endif
-
 config-target.h: config-target.h-timestamp
 config-target.h-timestamp: config-target.mak
 
diff --git a/configure b/configure
index 790d495..b0cb175 100755
--- a/configure
+++ b/configure
@@ -2524,6 +2524,20 @@  elif compile_prog "" "-lrt" ; then
   libs_qga="-lrt $libs_qga"
 fi
 
+##########################################
+# Do we need libm
+cat > $TMPC <<EOF
+#include <math.h>
+int main(void) { double a, b; return modf(a, &b);}
+EOF
+
+if compile_prog "" "" ; then
+  :
+elif compile_prog "" "-lm" ; then
+  LIBS="-lm $LIBS"
+  libs_qga="-lm $libs_qga"
+fi
+
 if test "$darwin" != "yes" -a "$mingw32" != "yes" -a "$solaris" != yes -a \
         "$aix" != "yes" -a "$haiku" != "yes" ; then
     libs_softmmu="-lutil $libs_softmmu"