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

Complete quoting for parameters of some CMake commands #122

Open
elfring opened this issue Dec 30, 2015 · 6 comments
Open

Complete quoting for parameters of some CMake commands #122

elfring opened this issue Dec 30, 2015 · 6 comments

Comments

@elfring
Copy link

elfring commented Dec 30, 2015

Some parameters (like "${TEST_DIR}") are passed to CMake commands in your build script without enclosing them by quotation marks. I see that these places will result in build difficulties if the contents of the used variables will contain special characters like semicolons.

I would recommend to apply advices from a wiki article.

@poulson
Copy link
Member

poulson commented Dec 30, 2015

While elfring is clearly trolling large numbers of projects (see his public activity) and has been blocked from this project, this is perhaps a valid point if a user is building Elemental from a directory whose full path contains a special character.

@justusc
Copy link
Contributor

justusc commented Dec 30, 2015

The most common problem that I have seen arise from not quoting paths is from paths that contain spaces on OS X. It is treated as a list and generates two (or more) incorrect paths.

@poulson
Copy link
Member

poulson commented Dec 30, 2015

Great point. This should definitely be avoided through introducing quotes then. I'm a bit backed up with other changesets right now so it may take a while to get to this.

@jeffhammond
Copy link
Member

I was burned by this quote issue within the past 48 hours (uwsampa/grappa#257) so it's definitely something to fix.

@poulson
Copy link
Member

poulson commented Dec 31, 2015

In that case, I'll increase its priority in my queue!

@rhl-
Copy link
Member

rhl- commented Nov 30, 2016

Is this still open? it looks like there was a commit addressing it.

benson31 added a commit to benson31/Elemental that referenced this issue Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants