Skip to content

Fix an issue where the pybind::embedd target would link to an incorrect python lib #2665

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

Closed
wants to merge 1 commit into from

Conversation

jbigot
Copy link

@jbigot jbigot commented Nov 15, 2020

Fix an issue where the pybind::embedd target would link to an incorrect python lib

  • with find_package(Python3) it would link the wrong Python3::Pyhton3 instead of the correct Python3::Pyhton
  • with find_package(Python2) it would link the wrong Python2::Pyhton2 instead of the correct Python3::Pyhton

Fix #2664

Description

Fix an issue where the pybind::embedd target would link to an incorrect python lib

  • with find_package(Python3) it would link the wrong Python3::Pyhton3 instead of the correct Python3::Pyhton
  • with find_package(Python2) it would link the wrong Python2::Pyhton2 instead of the correct Python3::Pyhton

Changed the target ${_Python}::${_Python} to ${_Python}::Python

Suggested changelog entry:

Fix an issue where the pybind::embedd target would link to an incorrect python lib

…ct python lib

* with find_package(Python3) it would link the wrong Python3::Pyhton3 instead of the correct Python3::Pyhton
* with find_package(Python2) it would link the wrong Python2::Pyhton2 instead of the correct Python3::Pyhton

Fix pybind#2664
@YannickJadoul
Copy link
Collaborator

Very close duplicate of #2662, it seems. But thanks for submitting!

@jbigot jbigot deleted the issue2664_fix branch November 15, 2020 17:08
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.

[BUG] cmake lead to missng symbols when linking against pybind11::embed while preloading python with find_package(Python3)
2 participants