napframework / nap

NAP Framework source code
https://nap-framework.tech
Mozilla Public License 2.0
410 stars 23 forks source link

string sub-command REPLACE requires at least four arguments #25

Closed sphaero closed 2 months ago

sphaero commented 5 months ago
CMake Error at cmake/nap_module.cmake:120 (string):
  string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
  system_modules/napsequenceaudiogui/CMakeLists.txt:23 (include)

with cmake 3.29

Fix

diff --git a/cmake/nap_module.cmake b/cmake/nap_module.cmake
index 981f14245..92e54fe90 100644
--- a/cmake/nap_module.cmake
+++ b/cmake/nap_module.cmake
@@ -117,7 +117,7 @@ if(APPLE)
 elseif(UNIX)
     set(LINUX_EXTRA_RPATH "")
     foreach(rpath ${DEEP_DEPENDENT_RPATHS})
-        string(REPLACE "{BUILD_TYPE}" ${CMAKE_BUILD_TYPE} rpath "${rpath}")
+        string(REPLACE "{BUILD_TYPE}" "${CMAKE_BUILD_TYPE}" rpath "${rpath}")
         list(APPEND LINUX_EXTRA_RPATH ${rpath})
     endforeach()
 endif()
cklosters commented 4 months ago

Could you submit this as a PR instead of logging it as a bug? NAP ships with it's own version of CMAKE (cmake version 3.21.3) in thirdparty and that's the one we support. Your change is compiled and validated, ensuring the change doesn't break existing builds.

cklosters commented 2 months ago

Not an issue with current supported cmake