jrl-umi3218 / jrl-cmakemodules

CMake utility toolbox
https://jrl-cmakemodules.readthedocs.io/en/master/
Other
56 stars 46 forks source link

python: include python-helpers #611

Closed nim65s closed 1 year ago

nim65s commented 1 year ago

608 moved some things from python.cmake to python-helpers.cmake.

As a result, projects updating their submodule are now broken with eg.

CMake Error at python/CMakeLists.txt:13 (python_install_on_site):
  Unknown CMake command "python_install_on_site".

Here is a quick fix, unless I misunderstood something.

nim65s commented 1 year ago

Now, another issue is that the new version of python.cmake without some macros is shipped with the latest eigenpy release, and multiple other packages were relying on it x)