From 8ca2ee4dfa46348f333afbbd092b6455082073bb Mon Sep 17 00:00:00 2001
From: Viktor Gal <viktor.gal@maeth.com>
Date: Fri, 13 Apr 2018 23:34:18 +0200
Subject: [PATCH] ship third_party libs in shogun/third_party

---
 CTestConfig.cmake           |  6 ------
 cmake/Findrxcpp.cmake       |  8 +-------
 cmake/external/Eigen3.cmake |  7 ++++---
 cmake/external/rxcpp.cmake  |  2 +-
 src/shogun/CMakeLists.txt   | 10 +++++-----
 5 files changed, 11 insertions(+), 22 deletions(-)
 delete mode 100644 CTestConfig.cmake

diff --git a/CTestConfig.cmake b/CTestConfig.cmake
deleted file mode 100644
index 93411561fea..00000000000
--- a/CTestConfig.cmake
+++ /dev/null
@@ -1,6 +0,0 @@
-SET (CTEST_PROJECT_NAME "shogun")
-
-set(CTEST_DROP_METHOD "http")
-set(CTEST_DROP_SITE "maeth.com")
-set(CTEST_DROP_LOCATION "/cdash/submit.php?project=Shogun")
-set(CTEST_DROP_SITE_CDASH TRUE)
\ No newline at end of file
diff --git a/cmake/Findrxcpp.cmake b/cmake/Findrxcpp.cmake
index 2577e3b6a35..87a09dc87a5 100644
--- a/cmake/Findrxcpp.cmake
+++ b/cmake/Findrxcpp.cmake
@@ -21,10 +21,4 @@ if(NOT EXISTS "${rxcpp_INCLUDE_DIR}")
 endif()
 
 include(FindPackageHandleStandardArgs)
-mark_as_advanced(rxcpp_INCLUDE_DIR)
-
-if(EXISTS "${rxcpp_INCLUDE_DIR}")
-    set(rxcpp_FOUND 1)
-else()
-    set(rxcpp_FOUND 0)
-endif()
\ No newline at end of file
+find_package_handle_standard_args(rxcpp REQUIRED_VARS rxcpp_INCLUDE_DIR)
diff --git a/cmake/external/Eigen3.cmake b/cmake/external/Eigen3.cmake
index 9900bb398a1..106e90180e7 100644
--- a/cmake/external/Eigen3.cmake
+++ b/cmake/external/Eigen3.cmake
@@ -7,11 +7,12 @@ ExternalProject_Add(
 	DOWNLOAD_DIR ${THIRD_PARTY_DIR}/Eigen3
 	URL https://bitbucket.org/eigen/eigen/get/${EIGEN_VERSION}.tar.bz2
 	URL_MD5 a7aab9f758249b86c93221ad417fbe18
-	CMAKE_ARGS -DEIGEN_INCLUDE_INSTALL_DIR:PATH=${EIGEN_INCLUDE_DIR}
+	CMAKE_ARGS -DINCLUDE_INSTALL_DIR:STRING=${EIGEN_INCLUDE_DIR}
 		-DCMAKE_C_FLAGS:STRING=${CMAKE_C_FLAGS}${CMAKE_DEFINITIONS}
 		-DCMAKE_CXX_FLAGS:STRING=${CMAKE_CXX_FLAGS}${CMAKE_DEFINITIONS}
-        -DCMAKE_C_COMPILER:STRING=${CMAKE_C_COMPILER}
-        -DCMAKE_CXX_COMPILER:STRING=${CMAKE_CXX_COMPILER}
+		-DCMAKE_C_COMPILER:STRING=${CMAKE_C_COMPILER}
+		-DCMAKE_CXX_COMPILER:STRING=${CMAKE_CXX_COMPILER}
+		-DCMAKE_BUILD_TYPE:STRING=Release
 		-DEIGEN_BUILD_PKGCONFIG=0
 	)
 
diff --git a/cmake/external/rxcpp.cmake b/cmake/external/rxcpp.cmake
index 0b39994d9a5..d5f00e37a6d 100644
--- a/cmake/external/rxcpp.cmake
+++ b/cmake/external/rxcpp.cmake
@@ -7,7 +7,7 @@ ExternalProject_Add(
         URL_MD5 feb89934f465bb5ac513c9adce8d3b1b
         CMAKE_ARGS
         -DCMAKE_BUILD_TYPE:STRING=RelWithDebInfo
-        -DCMAKE_INSTALL_PREFIX:STRING=${CMAKE_BINARY_DIR}/src/shogun/lib/external
+        -DCMAKE_INSTALL_PREFIX:STRING=${THIRD_PARTY_INCLUDE_DIR}
         -DCMAKE_C_COMPILER:STRING=${CMAKE_C_COMPILER}
         -DCMAKE_CXX_COMPILER:STRING=${CMAKE_CXX_COMPILER}
         BUILD_COMMAND ""
diff --git a/src/shogun/CMakeLists.txt b/src/shogun/CMakeLists.txt
index eb38bb67cc7..c8f79ac75d9 100644
--- a/src/shogun/CMakeLists.txt
+++ b/src/shogun/CMakeLists.txt
@@ -22,7 +22,7 @@ ENDIF()
 OPTION(LIBSHOGUN_BUILD_STATIC "Build libshogun static library")
 OPTION(DISABLE_SSE "Disable SSE and SSE2 features.")
 set(INCLUDE_INSTALL_DIR include)
-set(THIRD_PARTY_INCLUDE_DIR ${CMAKE_CURRENT_BINARY_DIR}/lib/external)
+set(THIRD_PARTY_INCLUDE_DIR ${CMAKE_CURRENT_BINARY_DIR}/third_party)
 
 if (MSVC)
   SET(LIBSHOGUN_BUILD_STATIC ON
@@ -164,7 +164,7 @@ IF(MSVC)
     include(external/MSIntTypes)
     SHOGUN_INCLUDE_DIRS(SCOPE PUBLIC
       $<BUILD_INTERFACE:${MSINTTYPES_INCLUDE_DIR}>
-      $<INSTALL_INTERFACE:include/shogun/lib/external/MSIntTypes>
+      $<INSTALL_INTERFACE:include/shogun/third_party/MSIntTypes>
     )
   ENDIF()
 
@@ -172,7 +172,7 @@ IF(MSVC)
   include(external/MSDirent)
   SHOGUN_INCLUDE_DIRS(SCOPE PUBLIC
     $<BUILD_INTERFACE:${MSDIRENT_INCLUDE_DIR}>
-    $<INSTALL_INTERFACE:include/shogun/lib/external/MSDirent>
+    $<INSTALL_INTERFACE:include/shogun/third_party/MSDirent>
   )
 
   target_link_libraries(shogun PUBLIC winmm)
@@ -223,7 +223,7 @@ IF(NOT EIGEN3_FOUND)
   include(external/Eigen3)
   SHOGUN_INCLUDE_DIRS(SCOPE PUBLIC SYSTEM
     $<BUILD_INTERFACE:${EIGEN_INCLUDE_DIR}>
-    $<INSTALL_INTERFACE:include/shogun/lib/external/eigen>
+    $<INSTALL_INTERFACE:include/shogun/third_party/eigen>
   )
 ELSE()
   # eigen3 bug related to aliasing operators and self-storing.
@@ -247,7 +247,7 @@ IF(NOT rxcpp_FOUND)
     include(external/rxcpp)
     SHOGUN_INCLUDE_DIRS(SCOPE PUBLIC SYSTEM
             $<BUILD_INTERFACE:${rxcpp_INCLUDE_DIR}>
-            $<INSTALL_INTERFACE:include/shogun/lib/external/rxcpp>
+            $<INSTALL_INTERFACE:include/shogun/third_party/rxcpp>
             )
 ELSE()
     SHOGUN_INCLUDE_DIRS(SCOPE PUBLIC SYSTEM ${rxcpp_INCLUDE_DIR})