Complete quoting for parameters of some CMake commands about magnum HOT 2 CLOSED

Sorry, but can't reproduce the issue. Actually I did start to quote every variable which might contain a path, but soon realized that none of that is necessary. I tried doing build, running tests and installing in path with spaces and it ran flawlessly. I also did some testing:

set(prefix "/path with spaces")
set(file ${prefix}/file.txt)

foreach(f ${file})
    message(STATUS ${f})
endforeach()

I assumed this would print three lines (containing /path, with, spaces/file.txt), but it actually behaved correctly and printed just one line containing /path with spaces/file.txt. So I conclude as long as the original variable (prefix) is created as string (quoted) and not as a list of strings (unquoted), all following variable replacements will treat it as whole string and not split it again. And all CMAKE_*_PREFIX and CMAKE_*_DIR are presented to the scripts as string values.

This is actually pretty awesome (unlike in shell, which would go haywire in this case).

from magnum.

Would you also like to consider the handling of semicolons?

from magnum.