Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fetch content #446

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Fetch content #446

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
fb703d8
Add install and FetchContent
flagarde Feb 16, 2023
6b7ae0d
cmake-format
flagarde Feb 16, 2023
eb8461d
Fix
flagarde Feb 16, 2023
0711ac1
cmake-format
flagarde Feb 16, 2023
a70c613
Update CMakeLists.txt
flagarde Feb 16, 2023
8ae0a09
Update cmake/CPM.cmake
flagarde Feb 16, 2023
3e7e6d6
Update CMakeLists.txt
flagarde Feb 16, 2023
d908cfe
cmake-format
flagarde Feb 16, 2023
d2975d0
Update CMakeLists.txt
flagarde Feb 16, 2023
14bb75f
Update CMakeLists.txt
flagarde Feb 16, 2023
a0663be
Update CMakeLists.txt
flagarde Feb 16, 2023
fa5058c
Update CPM.cmake
flagarde Feb 16, 2023
03027e0
Update CMakeLists.txt
flagarde Feb 16, 2023
deeefc3
Updates based on comments
flagarde Mar 2, 2023
f02381c
Updates based on comments
flagarde Mar 2, 2023
fbf5763
Updates based on comments
flagarde Mar 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 5 additions & 0 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,11 @@ if(NOT ${CMAKE_PROJECT_NAME} STREQUAL ${PROJECT_NAME})
else()
set(CPM_RELEASE TRUE)
endif()
flagarde marked this conversation as resolved.
Show resolved Hide resolved
# Trick to use the find_package
configure_file(
"${CMAKE_CURRENT_SOURCE_DIR}/cmake/CPM.cmake" "${CMAKE_CURRENT_BINARY_DIR}/CPM.cmake" COPYONLY
)
set(CPM_DIR "${CMAKE_CURRENT_BINARY_DIR}")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really necessary. In my tests, these are not set upstream. I don't understand how but:

  • CPM_DIR (set in this file) and CURRENT_CPM_VERSION (set in the include(cmake/CPM.cmake)) do not propagate upstream
  • But we can still use the macro/functions defined in cmake/CPM.cmake

include(cmake/CPM.cmake)
endif()

Expand Down