Update cmake --install support

1. Bug fix 2. Add a small project for testing installation
HenryAWE · May 21, 2024 · debdac6 · debdac6
1 parent baac147
commit debdac6
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 1 deletion.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -38,6 +38,16 @@ jobs:
           working-directory: ${{github.workspace}}/build/example
           run: ./info
 
+        - name: Install
+          working-directory: ${{github.workspace}}/build
+          run: sudo cmake --install .
+
+        - name: Test Installation
+          run: |
+            cmake -GNinja -S test/test_install/ -B build/papilio_test_install -DCMAKE_CXX_COMPILER=g++-12
+            cmake --build build/papilio_test_install
+            build/papilio_test_install/papilio_test_install
+
     Clang:
         runs-on: ubuntu-22.04
         strategy:

diff --git a/cmake/PapilioConfig.cmake.in b/cmake/PapilioConfig.cmake.in
@@ -1,4 +1,4 @@
 @PACKAGE_INIT@
 
 include("${CMAKE_CURRENT_LIST_DIR}/PapilioTargets.cmake")
-check_required_components(papilio::papilio)
+check_required_components(papilio)
diff --git a/test/test_install/CMakeLists.txt b/test/test_install/CMakeLists.txt
@@ -0,0 +1,11 @@
+cmake_minimum_required(VERSION 3.20)
+
+set(CMAKE_CXX_STANDARD 20)
+set(CMAKE_CXX_STANDARD_REQUIRED)
+
+project(papilio_test_install LANGUAGES CXX)
+
+find_package(Papilio REQUIRED)
+
+add_executable(papilio_test_install "test_install.cpp")
+target_link_libraries(papilio_test_install PRIVATE papilio::papilio)
diff --git a/test/test_install/test_install.cpp b/test/test_install/test_install.cpp
@@ -0,0 +1,10 @@
+#include <papilio/papilio.hpp>
+
+int main()
+{
+    papilio::println("papilio installed");
+    papilio::println();
+    papilio::println("{} {} {}", "hello", 42, 3.14f);
+
+    return 0;
+}