Skip to content

Commit

Permalink
Add missing REQUIRED in cmake (#212)
Browse files Browse the repository at this point in the history
Signed-off-by: Lucas Heitzmann Gabrielli <[email protected]>
  • Loading branch information
heitzmann committed Nov 21, 2023
1 parent 5469296 commit a73f605
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- name: Install dependencies (Linux)
run: sudo apt-get install -y libqhull-dev
if: runner.os == 'Linux'
- name: Install dependencies (macOS)
- name: Install dependencies (MacOS)
run: brew install qhull
if: runner.os == 'macOS'
- name: Install python dependencies
Expand Down Expand Up @@ -69,7 +69,7 @@ jobs:
- name: Install build dependencies (conan)
run: |
conan install . -s compiler.version=16
- name: Build Python ${{ matrix.python-version }} wheel
- name: Build Python ${{ matrix.python-version }}
run: python setup.py install --user
env:
CMAKE_GENERATOR: "Ninja"
Expand Down
2 changes: 1 addition & 1 deletion src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ if(NOT ZLIB_FOUND)
endif()
endif(NOT ZLIB_FOUND)

find_package(Qhull 8)
find_package(Qhull 8 REQUIRED)

set(HEADER_LIST
"${gdstk_SOURCE_DIR}/include/gdstk/allocator.hpp"
Expand Down

0 comments on commit a73f605

Please sign in to comment.