Remove source files from INTERFACE add_library (#210)

* Remove source files from add_library * Enable examples and tests for CMake min-req check. closes #209
Thalhammer · Feb 11, 2022 · 9d1a010 · 9d1a010
1 parent 4a537e9
commit 9d1a010
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/cmake.yml b/.github/workflows/cmake.yml
@@ -66,11 +66,13 @@ jobs:
           make -j $(nproc)
           sudo make install
 
+      - uses: ./.github/actions/install/gtest
+
       - name: setup
         run: |
           mkdir build
           cd build
-          cmake .. -DJWT_BUILD_EXAMPLES=OFF
+          cmake .. -DJWT_BUILD_EXAMPLES=ON -DJWT_BUILD_TESTS=ON
           sudo make install
 
       - name: test

diff --git a/cmake/private-find-kazuho-picojson.cmake b/cmake/private-find-kazuho-picojson.cmake
@@ -5,6 +5,6 @@ endif()
 unset(PICOJSON_INCLUDE_DIR CACHE)
 find_path(PICOJSON_INCLUDE_DIR "picojson/picojson.h")
 if(EXISTS "${PICOJSON_INCLUDE_DIR}/picojson/picojson.h")
-  add_library(kazuho_picojson INTERFACE "${PICOJSON_INCLUDE_DIR}/picojson/picojson.h")
+  add_library(kazuho_picojson INTERFACE)
   target_include_directories(kazuho_picojson INTERFACE ${PICOJSON_INCLUDE_DIR})
 endif()