Complete quoting for parameters of some CMake commands

Open Beginner friendly
#22 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
64/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
cmake
Domain
build-system

Research direction

Start in CMakeLists.txt, especially the commands and parameters at lines 33-52, then read the linked CMake list documentation about quoting. Update the affected command arguments so variable values containing special characters remain safe, and verify that the project configures and builds successfully.

Written by the indexing model from the issue text.

Description

Some parameters (like “${CMAKE_CURRENT_SOURCE_DIR}/dfd2vk.inl”) 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 the software documentation.

Dominant language
C++
Stars
8
Forks
9
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from KhronosGroup/dfdutils

All issues in KhronosGroup/dfdutils

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.