Improve finding gperftools and disable profiling by default (even with gperftools...

diff --git a/CMakeLists.txt b/CMakeLists.txt
index c99783b7555404f26dbe54ea0d4f690e65b0b26b..c472aff6f15dc79ef2cb54fd049d1b8f5a9c62d0 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -105,7 +105,7 @@ option(ENABLE_JSBSIM     "Set to ON to build FlightGear with JSBSim FDM (default
 option(EVENT_INPUT       "Set to ON to build FlightGear with event-based Input support" ${EVENT_INPUT_DEFAULT})
 option(ENABLE_LIBSVN     "Set to ON to build FlightGear/terrasync with libsvnclient support (default)" ON)
 option(ENABLE_RTI        "Set to ON to build FlightGear with RTI support" OFF)
-option(ENABLE_PROFILE    "Set to ON to build FlightGear with gperftools profiling support" ON)
+option(ENABLE_PROFILE    "Set to ON to build FlightGear with gperftools profiling support" OFF)
 option(JPEG_FACTORY      "Set to ON to build FlightGear with JPEG-factory support" OFF)
 option(SYSTEM_SQLITE     "Set to ON to build FlightGear with the system's SQLite3 library" OFF)
 
@@ -225,10 +225,9 @@ check_include_file(sys/time.h HAVE_SYS_TIME_H)
 check_include_file(windows.h  HAVE_WINDOWS_H)
 
 if(ENABLE_PROFILE)
-    find_package(GooglePerfTools)
-    if(GOOGLE_PERFTOOLS_FOUND)
-        set(FG_HAVE_GPERFTOOLS 1)
-    endif()
+    find_package(GooglePerfTools REQUIRED)
+    set(FG_HAVE_GPERFTOOLS 1)
+    message(STATUS "Built-in profiler using gperftools available")
 endif()
 
 if(ENABLE_RTI)

diff --git a/CMakeModules/FindGooglePerfTools.cmake b/CMakeModules/FindGooglePerfTools.cmake
index 9dab9ff69cf081efe2a96c296642150ccc533cf5..c787e87e16a287030a79d9e0d04135336b861d58 100644 (file)
--- a/CMakeModules/FindGooglePerfTools.cmake
+++ b/CMakeModules/FindGooglePerfTools.cmake
@@ -1,66 +1,48 @@
-# -*- cmake -*-
-
-# - Find Google perftools
-# Find the Google perftools includes and libraries
-# This module defines
-#  GOOGLE_PERFTOOLS_INCLUDE_DIR, where to find heap-profiler.h, etc.
-#  GOOGLE_PERFTOOLS_FOUND, If false, do not try to use Google perftools.
-# also defined for general use are
-#  TCMALLOC_LIBRARIES, where to find the tcmalloc library.
-#  STACKTRACE_LIBRARIES, where to find the stacktrace library.
-#  PROFILER_LIBRARIES, where to find the profiler library.
-
-FIND_PATH(GOOGLE_PERFTOOLS_INCLUDE_DIR google/heap-profiler.h
-/usr/local/include
-/usr/include
+# - Try to find GooglePerfTools headers and libraries
+#
+# Usage of this module as follows:
+#
+#   find_package(GooglePerfTools)
+#
+# Variables used by this module, they can change the default behaviour and need
+# to be set before calling find_package:
+#
+#  GooglePerfTools_ROOT_DIR  Set this variable to the root installation of
+#                            GooglePerfTools if the module has problems finding
+#                            the proper installation path.
+#
+# Variables defined by this module:
+#
+#  GooglePerfTools_FOUND              System has GooglePerfTools libs/headers
+#  GooglePerfTools_LIBRARIES          The GooglePerfTools libraries
+#  GooglePerfTools_INCLUDE_DIR        The location of GooglePerfTools headers
+
+find_path(GooglePerfTools_ROOT_DIR
+  NAMES include/google/profiler.h
 )
 
-SET(TCMALLOC_NAMES ${TCMALLOC_NAMES} tcmalloc_and_profiler)
-FIND_LIBRARY(TCMALLOC_LIBRARY
-  NAMES ${TCMALLOC_NAMES}
-  PATHS /usr/lib /usr/local/lib
+find_path(GooglePerfTools_INCLUDE_DIR
+  NAMES google/profiler.h
+  HINTS ${GooglePerfTools_ROOT_DIR}
 )
 
-IF (TCMALLOC_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-    SET(TCMALLOC_LIBRARIES ${TCMALLOC_LIBRARY})
-    SET(GOOGLE_PERFTOOLS_FOUND "YES")
-ELSE (TCMALLOC_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-  SET(GOOGLE_PERFTOOLS_FOUND "NO")
-ENDIF (TCMALLOC_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-
-SET(STACKTRACE_NAMES ${STACKTRACE_NAMES} stacktrace)
-FIND_LIBRARY(STACKTRACE_LIBRARY
-  NAMES ${STACKTRACE_LIBRARY}
-  PATHS /usr/lib /usr/local/lib
+find_library(GooglePerfTools_PROFILER_LIBRARY
+  NAMES profiler
+  HINTS ${GooglePerfTools_ROOT_DIR}
 )
 
-IF (STACKTRACE_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-    SET(STACKTRACE_LIBRARIES ${STACKTRACE_LIBRARY})
-ENDIF (STACKTRACE_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
+set(GooglePerfTools_LIBRARIES ${GooglePerfTools_PROFILER_LIBRARY})
 
-SET(PROFILER_NAMES ${PROFILER_NAMES} profiler)
-FIND_LIBRARY(PROFILER_LIBRARY
-  NAMES ${PROFILER_LIBRARY}
-  PATHS /usr/lib /usr/local/lib
+include(FindPackageHandleStandardArgs)
+find_package_handle_standard_args(GooglePerfTools
+  "Try setting GooglePerfTools_ROOT_DIR to root of your gperftools installation"
+  GooglePerfTools_LIBRARIES
+  GooglePerfTools_INCLUDE_DIR
 )
 
-IF (PROFILER_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-    SET(PROFILER_LIBRARIES ${PROFILER_LIBRARY})
-ENDIF (PROFILER_LIBRARY AND GOOGLE_PERFTOOLS_INCLUDE_DIR)
-
-IF (GOOGLE_PERFTOOLS_FOUND)
-   IF (NOT GOOGLE_PERFTOOLS_FIND_QUIETLY)
-      MESSAGE(STATUS "Found Google perftools: ${GOOGLE_PERFTOOLS_LIBRARIES}")
-   ENDIF (NOT GOOGLE_PERFTOOLS_FIND_QUIETLY)
-ELSE (GOOGLE_PERFTOOLS_FOUND)
-   IF (GOOGLE_PERFTOOLS_FIND_REQUIRED)
-      MESSAGE(FATAL_ERROR "Could not find Google perftools library")
-   ENDIF (GOOGLE_PERFTOOLS_FIND_REQUIRED)
-ENDIF (GOOGLE_PERFTOOLS_FOUND)
-
-MARK_AS_ADVANCED(
-  TCMALLOC_LIBRARY
-  STACKTRACE_LIBRARY
-  PROFILER_LIBRARY
-  GOOGLE_PERFTOOLS_INCLUDE_DIR
+mark_as_advanced(
+  GooglePerfTools_ROOT_DIR
+  GooglePerfTools_LIBRARIES
+  GooglePerfTools_PROFILER_LIBRARY
+  GooglePerfTools_INCLUDE_DIR
 )
\ No newline at end of file

diff --git a/src/Main/CMakeLists.txt b/src/Main/CMakeLists.txt
index b6e00a208909688c9b833f079b6724f62835a7c7..e69a6d39b1d19af519b301c0afe6f8c228516dcd 100644 (file)
--- a/src/Main/CMakeLists.txt
+++ b/src/Main/CMakeLists.txt
@@ -91,7 +91,8 @@ if(ENABLE_JSBSIM)
 endif()
 
 if(FG_HAVE_GPERFTOOLS)
-    target_link_libraries(fgfs profiler tcmalloc)
+    include_directories(${GooglePerfTools_INCLUDE_DIR})
+    target_link_libraries(fgfs ${GooglePerfTools_LIBRARIES})
 endif()
 
 target_link_libraries(fgfs