commit 5b5037649093d6e3c6f2ec6fdbfaff769cae6bb5
author Ghislain Antony Vaillant <ghisvail@users.noreply.github.com> Fri Mar 24 22:09:38 2017 +0000
committer Wenzel Jakob <wenzel.jakob@epfl.ch> Fri Mar 24 23:09:38 2017 +0100
tree e981a7ff3815d0197056c28370e767a3d17eb1c4
parent 0d92938f74ea46755294e5b5ab7b6ee81ca2c3b9

Arch-indep CMake packaging (#764)

* Arch-indep CMake packaging

Since pybind11 is a header-only library, the CMake packaging does not have to carry any architecture specific checks. Without this patch, the detection of pybind11 will fail on 32-bit architectures if the project was built on a 64-bit machine and vice-versa. This fix is similar to what is applied to `Eigen` and other header-only C++ libraries.

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 16f1d67..51f9d79 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -105,9 +105,14 @@
   configure_package_config_file(tools/${PROJECT_NAME}Config.cmake.in
                                 "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}Config.cmake"
                                 INSTALL_DESTINATION ${PYBIND11_CMAKECONFIG_INSTALL_DIR})
+  # Remove CMAKE_SIZEOF_VOID_P from ConfigVersion.cmake since the library does
+  # not depend on architecture specific settings or libraries.
+  set(_PYBIND11_CMAKE_SIZEOF_VOID_P ${CMAKE_SIZEOF_VOID_P})
+  unset(CMAKE_SIZEOF_VOID_P)
   write_basic_package_version_file(${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}ConfigVersion.cmake
                                    VERSION ${${PROJECT_NAME}_VERSION}
                                    COMPATIBILITY AnyNewerVersion)
+  set(CMAKE_SIZEOF_VOID_P ${_PYBIND11_CMAKE_SIZEOF_VOID_P})
   install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}Config.cmake
                 ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}ConfigVersion.cmake
                 tools/FindPythonLibsNew.cmake