This revision was automatically updated to reflect the committed changes.
Closed by commit rLLDB357282: [Cmake] Unify python variables (authored by
JDevlieghere, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D59968?vs=192735&id=192858#toc
Repository:
rLLDB LLDB
CHANGES S
labath added inline comments.
Comment at: lldb/cmake/modules/LLDBConfig.cmake:246-247
else()
-find_package(PythonLibs REQUIRED)
+include(FindPythonInterp)
+include(FindPythonLibs)
endif()
Is there any difference between `include(FindPythonInterp
stella.stamenova accepted this revision.
stella.stamenova added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59968/new/
https://reviews.llvm.org/D59968
JDevlieghere created this revision.
JDevlieghere added reviewers: zturner, stella.stamenova, labath, sgraenitz.
Herald added subscribers: lldb-commits, abidh, mgorny.
Herald added a project: LLDB.
Both FindPythonInterp and FindPythonLibs do two things, they'll set some
variables (PYTHON_LIBRARIES