-
Notifications
You must be signed in to change notification settings - Fork 206
Open
Description
This does not work:
CPMAddPackage(
NAME geogram
GIT_TAG cmake-fetchcontent
GITHUB_REPOSITORY tksuoran/geogram
OPTIONS
"GEOGRAM_SUB_BUILD ON"
"GEOGRAM_WITH_GRAPHICS OFF"
"GEOGRAM_WITH_LEGACY_NUMERICS OFF"
"GEOGRAM_WITH_HLBFGS ON"
"GEOGRAM_WITH_TETGEN ON"
"GEOGRAM_WITH_TRIANGLE ON"
"GEOGRAM_WITH_LUA OFF"
"GEOGRAM_LIB_ONLY ON"
"GEOGRAM_WITH_FPG ON"
"GEOGRAM_USE_SYSTEM_GLFW3 OFF"
"GEOGRAM_WITH_GARGANTUA OFF"
"GEOGRAM_WITH_TBB OFF"
)
Value of GEOGRAM_WITH_GRAPHICS
will end up with OFF
and that is not recognized as OFF
.
This works:
CPMAddPackage(
NAME geogram
GIT_TAG cmake-fetchcontent
GITHUB_REPOSITORY tksuoran/geogram
OPTIONS
"GEOGRAM_SUB_BUILD ON"
"GEOGRAM_WITH_GRAPHICS OFF"
"GEOGRAM_WITH_LEGACY_NUMERICS OFF"
"GEOGRAM_WITH_HLBFGS ON"
"GEOGRAM_WITH_TETGEN ON"
"GEOGRAM_WITH_TRIANGLE ON"
"GEOGRAM_WITH_LUA OFF"
"GEOGRAM_LIB_ONLY ON"
"GEOGRAM_WITH_FPG ON"
"GEOGRAM_USE_SYSTEM_GLFW3 OFF"
"GEOGRAM_WITH_GARGANTUA OFF"
"GEOGRAM_WITH_TBB OFF"
)
Would it be possible to allow multiple whitespaces here?
dwuertz
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
dwuertz commentedon Apr 8, 2025
Just got bitten by this and lost several hours...
Strip whitespace from OPTION_VALUE (fixes cpm-cmake#635)