Skip to content

Commit

Permalink
Fixed the CMake install action to actually check tatami_stats.
Browse files Browse the repository at this point in the history
  • Loading branch information
LTLA committed Mar 28, 2024
1 parent 1cd4a0e commit bea126d
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
13 changes: 10 additions & 3 deletions .github/workflows/check-install.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,15 @@ jobs:
- name: Get latest CMake
uses: lukka/get-cmake@latest

- name: Install tatami
run: |
git clone https://github.com/tatami-inc/tatami dep-tatami --depth=1
cd dep-tatami
cmake -S . -B build -DTATAMI_TESTS=OFF
sudo cmake --install build
- name: Configure the build
run: cmake -S . -B build -DTATAMI_TESTS=OFF
run: cmake -S . -B build -DTATAMI_STATS_FETCH_EXTERN=OFF -DTATAMI_STATS_TESTS=OFF

- name: Install the library
run: sudo cmake --install build
Expand All @@ -30,7 +37,7 @@ jobs:
cmake_minimum_required(VERSION 3.24)
project(test_install)
add_executable(whee source.cpp)
find_package(tatami_tatami)
target_link_libraries(whee tatami::tatami)
find_package(tatami_tatami_stats)
target_link_libraries(whee tatami::tatami_stats)
EOF
cd _downstream && cmake -S . -B build
3 changes: 3 additions & 0 deletions cmake/Config.cmake.in
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
@PACKAGE_INIT@

include(CMakeFindDependencyMacro)
find_dependency(tatami_tatami CONFIG REQUIRED)

include("${CMAKE_CURRENT_LIST_DIR}/tatami_tatami_statsTargets.cmake")

0 comments on commit bea126d

Please sign in to comment.