Link pthread when it is available
authorJohann <johannkoenig@google.com>
Fri, 18 Jan 2013 19:31:22 +0000 (11:31 -0800)
committerJohann <johannkoenig@google.com>
Fri, 18 Jan 2013 20:32:15 +0000 (12:32 -0800)
Linking when we don't use it but it is available is probably harmless.

Gtest requires pthreads. Don't automatically enable unit tests if we
don't have it.

Change-Id: I5e6c3b609f840c7b6dbb36fc65809f0ef84685f8

configure
libs.mk

index d551b38..89a1eb1 100755 (executable)
--- a/configure
+++ b/configure
@@ -655,7 +655,7 @@ process_toolchain() {
         enabled postproc || die "postproc_visualizer requires postproc to be enabled"
     fi
 
-    # Enable unit tests if we have a working C++ compiler
+    # Enable unit tests by default if we have a working C++ compiler.
     case "$toolchain" in
         *-vs*)
             soft_enable unit_tests
@@ -668,7 +668,7 @@ process_toolchain() {
             # x86 targets.
         ;;
         *)
-            check_cxx "$@" <<EOF && soft_enable unit_tests
+            enabled pthread_h && check_cxx "$@" <<EOF && soft_enable unit_tests
 int z;
 EOF
         ;;
diff --git a/libs.mk b/libs.mk
index 77ff2fc..a6f1392 100644 (file)
--- a/libs.mk
+++ b/libs.mk
@@ -17,7 +17,6 @@ else
   ASM:=.asm
 endif
 
-
 #
 # Calculate platform- and compiler-specific offsets for hand coded assembly
 #
@@ -327,7 +326,11 @@ vpx.pc: config.mk libs.mk
        $(qexec)echo 'Requires:' >> $@
        $(qexec)echo 'Conflicts:' >> $@
        $(qexec)echo 'Libs: -L$${libdir} -lvpx -lm' >> $@
+ifeq ($(HAVE_PTHREAD_H),yes)
        $(qexec)echo 'Libs.private: -lm -lpthread' >> $@
+else
+       $(qexec)echo 'Libs.private: -lm' >> $@
+endif
        $(qexec)echo 'Cflags: -I$${includedir}' >> $@
 INSTALL-LIBS-yes += $(LIBSUBDIR)/pkgconfig/vpx.pc
 INSTALL_MAPS += $(LIBSUBDIR)/pkgconfig/%.pc %.pc