Add use of compile time option

aminya · Sep 4, 2024 · 6287b4c · 6287b4c
1 parent 5b95e86
commit 6287b4c
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 15 deletions.
diff --git a/tests/myproj/CMakeLists.txt b/tests/myproj/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.16...3.21)
+cmake_minimum_required(VERSION 3.16...3.30)
 
 # set a default CXX standard used by the external tools like clang-tidy, cppcheck, etc.
 # You can later set fine-grained standards for each target using `target_compile_features`
@@ -55,7 +55,7 @@ if(APPLE)
   if(apple_version VERSION_GREATER_EQUAL 13)
     # workaround for linkage error as described in https://github.com/Homebrew/homebrew-core/issues/145991
     # although fixed, this problem still exists in github actions
-    add_link_options(-Wl,-ld_classic)
+    #XXX add_link_options(-Wl,-ld_classic)
     set(ENABLE_INTERPROCEDURAL_OPTIMIZATION "")
   endif()
 endif()
@@ -71,7 +71,7 @@ project_options(
   ENABLE_CLANG_TIDY
   # ENABLE_INCLUDE_WHAT_YOU_USE
   # ENABLE_GCC_ANALYZER
-  ENABLE_COVERAGE
+  #XXX ENABLE_COVERAGE
   # ENABLE_PCH
   # PCH_HEADERS
   # ${PCH_HEADERS}
@@ -124,6 +124,13 @@ target_include_system_directories( # Test the fix of semicolon in genex issue
 target_link_libraries(lib INTERFACE myproj_project_options myproj_project_warnings)
 target_link_system_libraries(lib INTERFACE fmt::fmt Eigen3::Eigen)
 
+option(LIB_WITH_EIGEN "Enable Eigen" OFF)
+if(LIB_WITH_EIGEN)
+  target_compile_definitions(lib INTERFACE HAS_EIGEN)
+endif()
+
+target_link_libraries(main PRIVATE lib)
+
 # Library
 add_library(lib2 "./src/mylib2/lib.cpp")
 set(lib2_INCLUDE_DIR2 "include")
@@ -178,3 +185,6 @@ package_project(
 )
 
 package_project(NAME myproj_main TARGETS main)
+
+set(CPACK_GENERATOR TGZ)
+include(CPack)
diff --git a/tests/myproj/include/mylib/lib.hpp b/tests/myproj/include/mylib/lib.hpp
@@ -1,7 +1,12 @@
 #pragma once
 
 // test external pac
-#include <Eigen/Dense>
+#ifdef HAS_EIGEN
+#    include <Eigen/Dense>
+#else
+#    include <vector>
+#endif
+
 #include <fmt/core.h>
 #include <fmt/ranges.h>
 
@@ -17,14 +22,19 @@
 #include <cstdint>
 #include <cstring>
 
-int some_fun() {
-    fmt::print("Hello from fmt{}", "!");
+int some_fun()
+{
+    fmt::print("Hello from lib{}\n", "!");
 
+#ifdef HAS_EIGEN
     // populate an Eigen vector with the values
     auto eigen_vec = Eigen::VectorXd::LinSpaced(10, 0, 1);
+#else
+    auto eigen_vec = std::vector<int>() = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
+#endif
 
     // print the vector
-    fmt::print("[{}]", fmt::join(eigen_vec, ", "));
+    fmt::print("[{}]\n", fmt::join(eigen_vec, ", "));
 
     return 0;
 }
diff --git a/tests/myproj/src/main/main.cpp b/tests/myproj/src/main/main.cpp
@@ -1,12 +1,18 @@
+#include "mylib/lib.hpp"
+
 // test external pac
-#include <Eigen/Dense>
 #include <fmt/core.h>
 #include <fmt/ranges.h>
 
+#ifdef HAS_EIGEN
+#    include <Eigen/Dense>
+#endif
+
 // test std libraries
 #include <iostream>
 #include <string>
 #include <string_view>
+#include <vector>
 
 // test c libraries
 #include <cassert>
@@ -15,21 +21,22 @@
 #include <cstdint>
 #include <cstring>
 
-int main() {
-    fmt::print("Hello from fmt{}", "!");
+int main()
+{
+    fmt::print("Hello from main{}\n", "!");
 
-    Eigen::VectorXd eigen_vec = Eigen::Vector3d(1, 2, 3);
-    fmt::print("[{}]", fmt::join(eigen_vec, ", "));
+    auto eigen_vec = std::vector<int>() = {1, 2, 3};
+    fmt::print("[{}]\n", fmt::join(eigen_vec, ", "));
 
-#if !defined(__MINGW32__) && !defined(__MSYS__)// TODO fails
+#if defined(HAS_EIGEN) && !defined(__MINGW32__) && !defined(__MSYS__) // TODO fails
     Eigen::VectorXd eigen_vec2 = Eigen::VectorXd::LinSpaced(10, 0, 1);
-    fmt::print("[{}]", fmt::join(eigen_vec2, ", "));
+    fmt::print("[{}]\n", fmt::join(eigen_vec2, ", "));
 #endif
 
     // trigger address sanitizer
     // int *p = nullptr;
     // *p = 1;
 
     // trigger compiler warnings, clang-tidy, and cppcheck
-    int a;
+    int a = some_fun();
 }