Skip to content

OPTIONS in CPMAddPackage does not like multiple whitespace #635

@tksuoran

Description

@tksuoran

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?

Activity

dwuertz

dwuertz commented on Apr 8, 2025

@dwuertz

Just got bitten by this and lost several hours...

added a commit that references this issue on Apr 8, 2025
3588035
linked a pull request that will close this issue on Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @tksuoran@dwuertz

      Issue actions

        OPTIONS in CPMAddPackage does not like multiple whitespace · Issue #635 · cpm-cmake/CPM.cmake