Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

set CMAKE_PREFIX_PATH#31

Open
djsutherland wants to merge 1 commit intoconda-forge:mainfrom
djsutherland:cmake-prefix
Open

set CMAKE_PREFIX_PATH#31
djsutherland wants to merge 1 commit intoconda-forge:mainfrom
djsutherland:cmake-prefix

Conversation

@djsutherland
Copy link
Contributor

Not totally sure if this is a good idea or not, but this might make cmake builds more likely to work with less fiddling.

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@djsutherland
Copy link
Contributor Author

Hmm. Actually, maybe CMAKE_SYSTEM_PREFIX_PATH is more meant for this, and CMAKE_PREFIX_PATH "is intended to be set by the project." And the system prefix path already includes CMAKE_INSTALL_PREFIX which is always set by the build script, but I think that it maybe includes it after the standard system directories. I'll investigate further later.

@jakirkham jakirkham requested a review from isuruf November 2, 2017 03:34
@jakirkham
Copy link
Member

Something like this sounds like a good idea to me. Though am a bit fuzzy on which CMake paths need to be set ATM.

Also would bump the minor version number once we settle this as this adds functionality without removing any from the toolchain.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants