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.

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

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

Open
sphaero opened this issue May 15, 2024 · 1 comment
Open

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

sphaero opened this issue May 15, 2024 · 1 comment

Comments

@sphaero
Copy link

sphaero commented May 15, 2024

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
Copy link
Member

cklosters commented May 22, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants