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

Quote CMAKE_COMMAND and CMAKE_CTEST_COMMAND #74

Merged
merged 1 commit into from Aug 11, 2019

Conversation

@myd7349
Copy link
Contributor

commented Aug 11, 2019

Thanks @spinda for your advice in #72 .

This patch also specifies VERBATIM for add_custom_command and add_custom_target as recommanded in https://stackoverflow.com/questions/35847655/when-should-i-quote-variables .

Quote CMAKE_COMMAND and CMAKE_CTEST_COMMAND
Also specify VERBATIM for add_custom_command and add_custom_target

@jedisct1 jedisct1 merged commit 46f083e into jedisct1:master Aug 11, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@myd7349 myd7349 deleted the myd7349:quote-cmake-paths branch Aug 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.