From 693fa27fedfd0aa1abad754a8ee27e85487b8fd8 Mon Sep 17 00:00:00 2001
From: Jeff Mitchell <jeff@tomahawk-player.org>
Date: Tue, 7 May 2013 12:38:41 -0400
Subject: [PATCH] Make finding boost conditional upon examples being built;
 remove building when possible by only using header versions of files

---
 CMakeLists.txt             | 282 +++++++++++++++++++------------------
 cmake/CMakeHelpers.cmake   |  43 +-----
 websocketpp/CMakeLists.txt |  23 ---
 3 files changed, 145 insertions(+), 203 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index ee2e24e033..29e64297bb 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -21,89 +21,6 @@ set (CMAKE_CONFIGURATION_TYPES "Release;RelWithDebInfo;Debug" CACHE STRING "Conf
 set (CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} ${CMAKE_CURRENT_SOURCE_DIR}/cmake)
 include (CMakeHelpers)
 
-
-############  Build customization
-
-# Override from command line "CMake -D<OPTION>=TRUE/FALSE/0/1"
-option (ENABLE_CPP11 "Build websocketpp with CPP11 features enabled." TRUE)
-option (BUILD_EXAMPLES "Build websocketpp examples." TRUE)
-option (BUILD_TESTS "Build websocketpp tests." FALSE)
-
-
-############ Compiler specific setup
-
-set (WEBSOCKETPP_PLATFORM_LIBS "")
-set (WEBSOCKETPP_PLATFORM_TSL_LIBS "")
-set (WEBSOCKETPP_BOOST_LIBS "")
-
-# VC9 and C++11 reasoning
-if (ENABLE_CPP11 AND MSVC AND MSVC90)
-    message("* Detected Visual Studio 9 2008, disabling C++11 support.")
-    set (ENABLE_CPP11 FALSE)
-endif ()
-
-# Detect clang. Not officially reported by cmake.
-execute_process(COMMAND "${CMAKE_CXX_COMPILER}" "-v" ERROR_VARIABLE CXX_VER_STDERR)
-if ("${CXX_VER_STDERR}" MATCHES ".*clang.*")
-    set (CMAKE_COMPILER_IS_CLANGXX 1)
-endif ()
-
-# C++11 defines
-if (ENABLE_CPP11)
-    add_definitions (-D_WEBSOCKETPP_CPP11_STL_)
-endif ()
-
-# Visual studio
-if (MSVC)
-    set (WEBSOCKETPP_BOOST_LIBS system thread regex)
-    set (CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /GL /Gy /GF /Ox /Ob2 /Ot /Oi /MP /arch:SSE2 /fp:fast")
-    set (CMAKE_SHARED_LINKER_FLAGS_RELEASE "${CMAKE_SHARED_LINKER_FLAGS_RELEASE} /LTCG /INCREMENTAL:NO /OPT:REF /OPT:ICF")
-    add_definitions (/W3 /wd4996 /wd4995 /wd4355)
-    add_definitions (-DUNICODE -D_UNICODE)
-    add_definitions (-D_CRT_SECURE_NO_WARNINGS -D_SCL_SECURE_NO_WARNINGS)
-    add_definitions (-DNOMINMAX)
-endif ()
-
-# g++
-if (CMAKE_COMPILER_IS_GNUCXX)
-    set (WEBSOCKETPP_PLATFORM_LIBS pthread rt)
-    set (WEBSOCKETPP_PLATFORM_TSL_LIBS ssl crypto)
-    set (WEBSOCKETPP_BOOST_LIBS system thread regex)
-    set (CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS} "-std=c++0x")
-    if (NOT APPLE)
-        add_definitions (-DNDEBUG -Wall -Wcast-align) # todo: should we use CMAKE_C_FLAGS for these?
-    endif ()
-
-    # Try to detect version. Note: Not tested!
-    execute_process (COMMAND ${CMAKE_CXX_COMPILER} "-dumpversion" OUTPUT_VARIABLE GCC_VERSION)
-    if ("${GCC_VERSION}" STRGREATER "4.4.0")
-        message("* C++11 support partially enabled due to GCC version ${GCC_VERSION}")
-        set (WEBSOCKETPP_BOOST_LIBS system thread regex)
-        add_definitions (-D_WEBSOCKETPP_NO_CPP11_REGEX_)
-    endif ()
-endif ()
-
-# clang
-if (CMAKE_COMPILER_IS_CLANGXX)
-    if (NOT APPLE)
-        set (WEBSOCKETPP_PLATFORM_LIBS pthread rt)
-    else()
-        set (WEBSOCKETPP_PLATFORM_LIBS pthread)
-    endif()
-    set (WEBSOCKETPP_PLATFORM_TSL_LIBS ssl crypto)
-    set (WEBSOCKETPP_BOOST_LIBS system thread regex)
-    set (CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS} "-std=c++0x -stdlib=libc++") # todo: is libc++ really needed here?
-    if (NOT APPLE)
-        add_definitions (-DNDEBUG -Wall -Wno-padded) # todo: should we use CMAKE_C_FLAGS for these?
-    endif ()
-endif ()
-
-# OSX, can override above.
-if (APPLE)
-    add_definitions (-DNDEBUG -Wall)
-endif ()
-
-
 ############ Paths
 
 set (WEBSOCKETPP_ROOT ${CMAKE_CURRENT_SOURCE_DIR})
@@ -117,74 +34,159 @@ set (WEBSOCKETPP_PREFIX ${WEBSOCKETPP_ROOT}/build)
 # - Windows: Build the INSTALL project in your solution file.
 # - Linux/OSX: make install.
 if (MSVC)
-	set (CMAKE_INSTALL_PREFIX ${WEBSOCKETPP_PREFIX})
+    set (CMAKE_INSTALL_PREFIX ${WEBSOCKETPP_PREFIX})
+else ()
+    set (CMAKE_INSTALL_PREFIX "${CMAKE_INSTALL_PREFIX}/websocketpp")
 endif ()
 
-if (BUILD_EXAMPLES)
-  list (APPEND WEBSOCKETPP_BOOST_LIBS random)
-endif()
+############  Build customization
 
-############ Dependencies
+# Override from command line "CMake -D<OPTION>=TRUE/FALSE/0/1"
+option (ENABLE_CPP11 "Build websocketpp with CPP11 features enabled." TRUE)
+option (BUILD_EXAMPLES "Build websocketpp examples." TRUE)
+option (BUILD_TESTS "Build websocketpp tests." FALSE)
 
-# Set BOOST_ROOT env variable or pass with cmake -DBOOST_ROOT=path.
-# BOOST_ROOT can also be defined by a previous run from cmake cache.
-if (NOT "$ENV{BOOST_ROOT_CPP11}" STREQUAL "")
-    # Scons documentation for BOOST_ROOT_CPP11:
-    # "look for optional second boostroot compiled with clang's libc++ STL library
-    # this prevents warnings/errors when linking code built with two different
-    # incompatible STL libraries."
-    file (TO_CMAKE_PATH "$ENV{BOOST_ROOT_CPP11}" BOOST_ROOT)
-    set (BOOST_ROOT ${BOOST_ROOT} CACHE PATH "BOOST_ROOT dependency path" FORCE)
-endif ()
-if ("${BOOST_ROOT}" STREQUAL "")
-    file (TO_CMAKE_PATH "$ENV{BOOST_ROOT}" BOOST_ROOT)
-    # Cache BOOST_ROOT for runs that do not define $ENV{BOOST_ROOT}.
-    set (BOOST_ROOT ${BOOST_ROOT} CACHE PATH "BOOST_ROOT dependency path" FORCE)
-endif ()
+if (BUILD_TESTS OR BUILD_EXAMPLES)
 
-message ("* Configuring Boost")
-message (STATUS "-- Using BOOST_ROOT")
-message (STATUS "       " ${BOOST_ROOT})
+    ############ Compiler specific setup
 
-if (MSVC)
+    set (WEBSOCKETPP_PLATFORM_LIBS "")
+    set (WEBSOCKETPP_PLATFORM_TSL_LIBS "")
+    set (WEBSOCKETPP_BOOST_LIBS "")
+
+    # VC9 and C++11 reasoning
+    if (ENABLE_CPP11 AND MSVC AND MSVC90)
+        message("* Detected Visual Studio 9 2008, disabling C++11 support.")
+        set (ENABLE_CPP11 FALSE)
+    endif ()
+
+    # Detect clang. Not officially reported by cmake.
+    execute_process(COMMAND "${CMAKE_CXX_COMPILER}" "-v" ERROR_VARIABLE CXX_VER_STDERR)
+    if ("${CXX_VER_STDERR}" MATCHES ".*clang.*")
+        set (CMAKE_COMPILER_IS_CLANGXX 1)
+    endif ()
+
+    # C++11 defines
+    if (ENABLE_CPP11)
+        add_definitions (-D_WEBSOCKETPP_CPP11_STL_)
+    endif ()
+
+    # Visual studio
+    if (MSVC)
+        set (WEBSOCKETPP_BOOST_LIBS system thread regex)
+        set (CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /GL /Gy /GF /Ox /Ob2 /Ot /Oi /MP /arch:SSE2 /fp:fast")
+        set (CMAKE_SHARED_LINKER_FLAGS_RELEASE "${CMAKE_SHARED_LINKER_FLAGS_RELEASE} /LTCG /INCREMENTAL:NO /OPT:REF /OPT:ICF")
+        add_definitions (/W3 /wd4996 /wd4995 /wd4355)
+        add_definitions (-DUNICODE -D_UNICODE)
+        add_definitions (-D_CRT_SECURE_NO_WARNINGS -D_SCL_SECURE_NO_WARNINGS)
+        add_definitions (-DNOMINMAX)
+    endif ()
+
+    # g++
+    if (CMAKE_COMPILER_IS_GNUCXX)
+        set (WEBSOCKETPP_PLATFORM_LIBS pthread rt)
+        set (WEBSOCKETPP_PLATFORM_TSL_LIBS ssl crypto)
+        set (WEBSOCKETPP_BOOST_LIBS system thread regex)
+        set (CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS} "-std=c++0x")
+        if (NOT APPLE)
+            add_definitions (-DNDEBUG -Wall -Wcast-align) # todo: should we use CMAKE_C_FLAGS for these?
+        endif ()
+
+        # Try to detect version. Note: Not tested!
+        execute_process (COMMAND ${CMAKE_CXX_COMPILER} "-dumpversion" OUTPUT_VARIABLE GCC_VERSION)
+        if ("${GCC_VERSION}" STRGREATER "4.4.0")
+            message("* C++11 support partially enabled due to GCC version ${GCC_VERSION}")
+            set (WEBSOCKETPP_BOOST_LIBS system thread regex)
+            add_definitions (-D_WEBSOCKETPP_NO_CPP11_REGEX_)
+        endif ()
+    endif ()
+
+    # clang
+    if (CMAKE_COMPILER_IS_CLANGXX)
+        if (NOT APPLE)
+            set (WEBSOCKETPP_PLATFORM_LIBS pthread rt)
+        else()
+            set (WEBSOCKETPP_PLATFORM_LIBS pthread)
+        endif()
+        set (WEBSOCKETPP_PLATFORM_TSL_LIBS ssl crypto)
+        set (WEBSOCKETPP_BOOST_LIBS system thread regex)
+        set (CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS} "-std=c++0x -stdlib=libc++") # todo: is libc++ really needed here?
+        if (NOT APPLE)
+            add_definitions (-DNDEBUG -Wall -Wno-padded) # todo: should we use CMAKE_C_FLAGS for these?
+        endif ()
+    endif ()
+
+    # OSX, can override above.
+    if (APPLE)
+        add_definitions (-DNDEBUG -Wall)
+    endif ()
+
+    if (BUILD_EXAMPLES)
+        list (APPEND WEBSOCKETPP_BOOST_LIBS random)
+    endif()
+
+    ############ Dependencies
+
+    # Set BOOST_ROOT env variable or pass with cmake -DBOOST_ROOT=path.
+    # BOOST_ROOT can also be defined by a previous run from cmake cache.
+    if (NOT "$ENV{BOOST_ROOT_CPP11}" STREQUAL "")
+        # Scons documentation for BOOST_ROOT_CPP11:
+        # "look for optional second boostroot compiled with clang's libc++ STL library
+        # this prevents warnings/errors when linking code built with two different
+        # incompatible STL libraries."
+        file (TO_CMAKE_PATH "$ENV{BOOST_ROOT_CPP11}" BOOST_ROOT)
+        set (BOOST_ROOT ${BOOST_ROOT} CACHE PATH "BOOST_ROOT dependency path" FORCE)
+    endif ()
+    if ("${BOOST_ROOT}" STREQUAL "")
+        file (TO_CMAKE_PATH "$ENV{BOOST_ROOT}" BOOST_ROOT)
+        # Cache BOOST_ROOT for runs that do not define $ENV{BOOST_ROOT}.
+        set (BOOST_ROOT ${BOOST_ROOT} CACHE PATH "BOOST_ROOT dependency path" FORCE)
+    endif ()
+
+    message ("* Configuring Boost")
+    message (STATUS "-- Using BOOST_ROOT")
+    message (STATUS "       " ${BOOST_ROOT})
+
+    if (MSVC)
+        set (Boost_USE_MULTITHREADED TRUE)
+        set (Boost_USE_STATIC_LIBS TRUE)
+    else ()
+        set (Boost_USE_MULTITHREADED FALSE)
+        set (Boost_USE_STATIC_LIBS FALSE)
+    endif ()
+
+    set (Boost_FIND_REQUIRED TRUE)
+    set (Boost_FIND_QUIETLY TRUE)
+    set (Boost_DEBUG FALSE)
     set (Boost_USE_MULTITHREADED TRUE)
-    set (Boost_USE_STATIC_LIBS TRUE)
-else ()
-    set (Boost_USE_MULTITHREADED FALSE)
-    set (Boost_USE_STATIC_LIBS FALSE)
-endif ()
+    set (Boost_ADDITIONAL_VERSIONS "1.39.0" "1.40.0" "1.41.0" "1.42.0" "1.43.0" "1.44.0" "1.46.1") # todo: someone who knows better spesify these!
 
-set (Boost_FIND_REQUIRED TRUE)
-set (Boost_FIND_QUIETLY TRUE)
-set (Boost_DEBUG FALSE)
-set (Boost_USE_MULTITHREADED TRUE)
-set (Boost_ADDITIONAL_VERSIONS "1.39.0" "1.40.0" "1.41.0" "1.42.0" "1.43.0" "1.44.0" "1.46.1") # todo: someone who knows better spesify these!
+    find_package (Boost 1.39.0 COMPONENTS "${WEBSOCKETPP_BOOST_LIBS}")
 
-find_package (Boost 1.39.0 COMPONENTS "${WEBSOCKETPP_BOOST_LIBS}")
+    if (Boost_FOUND)
+        # Boost is a project wide global dependency.
+        include_directories (${Boost_INCLUDE_DIRS})
+        link_directories (${Boost_LIBRARY_DIRS})
 
-if (Boost_FOUND)
-    # Boost is a project wide global dependency.
-    include_directories (${Boost_INCLUDE_DIRS})
-    link_directories (${Boost_LIBRARY_DIRS})
-
-    # Pretty print status
-    message (STATUS "-- Include Directories")
-    foreach (include_dir ${Boost_INCLUDE_DIRS})
-        message (STATUS "       " ${include_dir})
-    endforeach ()
-    message (STATUS "-- Library Directories")
-    foreach (library_dir ${Boost_LIBRARY_DIRS})
-        message (STATUS "       " ${library_dir})
-    endforeach ()
-    message (STATUS "-- Libraries")
-    foreach (boost_lib ${Boost_LIBRARIES})
-        message (STATUS "       " ${boost_lib})
-    endforeach ()
-    message ("")
-else ()
-    message (FATAL_ERROR "Failed to find required dependency: boost")
-endif ()
+        # Pretty print status
+        message (STATUS "-- Include Directories")
+        foreach (include_dir ${Boost_INCLUDE_DIRS})
+            message (STATUS "       " ${include_dir})
+        endforeach ()
+        message (STATUS "-- Library Directories")
+        foreach (library_dir ${Boost_LIBRARY_DIRS})
+            message (STATUS "       " ${library_dir})
+        endforeach ()
+        message (STATUS "-- Libraries")
+        foreach (boost_lib ${Boost_LIBRARIES})
+            message (STATUS "       " ${boost_lib})
+        endforeach ()
+        message ("")
+    else ()
+        message (FATAL_ERROR "Failed to find required dependency: boost")
+    endif ()
 
+endif()
 
 ############ Add projects
 
diff --git a/cmake/CMakeHelpers.cmake b/cmake/CMakeHelpers.cmake
index 42f8edd471..0623237158 100644
--- a/cmake/CMakeHelpers.cmake
+++ b/cmake/CMakeHelpers.cmake
@@ -45,37 +45,12 @@ endmacro()
 macro (init_target NAME)
     set (TARGET_NAME ${NAME})
     message ("** " ${TARGET_NAME})
-    
+
     # Include our own module path. This makes #include "x.h" 
     # work in project subfolders to include the main directory headers.
     include_directories (${CMAKE_CURRENT_SOURCE_DIR})
 endmacro ()
 
-# Build library for static and shared libraries
-macro (build_library TARGET_NAME LIB_TYPE)
-    set (TARGET_LIB_TYPE ${LIB_TYPE})
-    message (STATUS "-- Build Type:")
-    message (STATUS "       " ${TARGET_LIB_TYPE} " library")
-
-    add_library (${TARGET_NAME} ${TARGET_LIB_TYPE} ${ARGN})
-
-    target_link_libraries (${TARGET_NAME} ${WEBSOCKETPP_PLATFORM_LIBS})
-
-    if (MSVC)
-        if (${TARGET_LIB_TYPE} STREQUAL "SHARED")
-            set_target_properties (${TARGET_NAME} PROPERTIES LINK_FLAGS_RELEASE ${CMAKE_SHARED_LINKER_FLAGS_RELEASE})
-        else ()
-            set_target_properties (${TARGET_NAME} PROPERTIES STATIC_LIBRARY_FLAGS_RELEASE "/LTCG")
-        endif ()
-    endif ()
-
-    message (STATUS "-- Build Destination:")
-    message (STATUS "       " ${WEBSOCKETPP_LIB})
-
-    set_target_properties (${TARGET_NAME} PROPERTIES ARCHIVE_OUTPUT_DIRECTORY ${WEBSOCKETPP_LIB})
-    set_target_properties (${TARGET_NAME} PROPERTIES DEBUG_POSTFIX d)
-endmacro ()
-
 # Build executable for executables
 macro (build_executable TARGET_NAME)
     set (TARGET_LIB_TYPE "EXECUTABLE")
@@ -92,24 +67,12 @@ endmacro ()
 
 # Finalize target for all types
 macro (final_target)
-    if (${TARGET_LIB_TYPE} STREQUAL "SHARED")
-        install (TARGETS ${TARGET_NAME} 
-                 LIBRARY DESTINATION "bin" # non DLL platforms shared libs are LIBRARY
-                 RUNTIME DESTINATION "bin" # DLL platforms shared libs are RUNTIME
-                 ARCHIVE DESTINATION "lib" # DLL platforms static link part of the shared lib are ARCHIVE
-                 CONFIGURATIONS ${CMAKE_CONFIGURATION_TYPES})
-    endif ()
-    if (${TARGET_LIB_TYPE} STREQUAL "STATIC")
-        install (TARGETS ${TARGET_NAME} 
-                 ARCHIVE DESTINATION "lib" 
-                 CONFIGURATIONS ${CMAKE_CONFIGURATION_TYPES})
-    endif ()
-    if (${TARGET_LIB_TYPE} STREQUAL "EXECUTABLE")
+    if ("${TARGET_LIB_TYPE}" STREQUAL "EXECUTABLE")
         install (TARGETS ${TARGET_NAME} 
                  RUNTIME DESTINATION "bin" 
                  CONFIGURATIONS ${CMAKE_CONFIGURATION_TYPES})
     endif ()
-    
+
     # install headers, directly from current source dir and look for subfolders with headers
     file (GLOB TARGET_INSTALL_HEADERS *.hpp)
     install (FILES ${TARGET_INSTALL_HEADERS} DESTINATION "include/${TARGET_NAME}")
diff --git a/websocketpp/CMakeLists.txt b/websocketpp/CMakeLists.txt
index 8c5361bc3d..bfd1d2f789 100644
--- a/websocketpp/CMakeLists.txt
+++ b/websocketpp/CMakeLists.txt
@@ -1,24 +1 @@
-
-init_target (libwebsocketpp)
-
-# Sources from root dir
-file (GLOB WSPP_SOURCE_FILES *.cpp)
-file (GLOB WSPP_HEADER_FILES *.hpp)
-
-# Sources from subdirectories.
-list_subdirectories (SUBDIRS ${CMAKE_CURRENT_SOURCE_DIR})
-foreach (SUBDIR ${SUBDIRS})
-    # Filter out certain subdirs from 3rd party code
-    if (NOT ${SUBDIR} STREQUAL "sha1" AND NOT ${SUBDIR} STREQUAL "md5") # todo: AND NOT ${SUBDIR} STREQUAL "base64"
-        add_source_folder (${SUBDIR})
-    endif ()
-endforeach ()
-
-string(TOLOWER "${WEBSOCKETPP_LIB_BUILD_TYPE}" LOWER_WEBSOCKETPP_BUILD_TYPE)
-if ("${LOWER_WEBSOCKETPP_BUILD_TYPE}" STREQUAL "shared" )
-    build_library (${TARGET_NAME} SHARED ${WSPP_SOURCE_FILES} ${WSPP_HEADER_FILES})
-else()
-    build_library (${TARGET_NAME} STATIC ${WSPP_SOURCE_FILES} ${WSPP_HEADER_FILES})
-endif()
-
 final_target ()