Skip to content

Ensure PythonLibsNew_FOUND is set in FindPythonLibsNew module #1373

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

Merged
merged 1 commit into from
May 12, 2019
Merged

Ensure PythonLibsNew_FOUND is set in FindPythonLibsNew module #1373

merged 1 commit into from
May 12, 2019

Conversation

jamiesnape
Copy link
Contributor

Since the module is named FindPythonLibsNew, PythonLibsNew_FOUND should be set appropriately.

@henryiii
Copy link
Collaborator

This is also useful if you use CMake's FeatureSummary module; currently it misreports about PythonLibsNew because PythonLibsNew_FOUND is not set. 👍

Since the module is named FindPythonLibsNew, PythonLibsNew_FOUND should be set appropriately.
@EricCousineau-TRI
Copy link
Collaborator

@wjakob @jagerman Any chance we might be able to land this?

@wjakob
Copy link
Member

wjakob commented May 12, 2019

LGTM!

@wjakob wjakob merged commit a0b8f70 into pybind:master May 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants