roboticslab-uc3m / questions-and-answers

A place for general debate and question&answer
https://robots.uc3m.es/developer-manual/appendix/repository-index.html
2 stars 0 forks source link

Update CMake/YCM export options #84

Closed PeterBowman closed 4 years ago

PeterBowman commented 5 years ago

YCM 0.11 will introduce a bunch of new features regarding how targets are treated upon export:

PeterBowman commented 4 years ago

Seems like YCM 0.11 might bump minimum required CMake version to 3.12: https://github.com/robotology/ycm/issues/261. Thus, requiring YCM 0.11+ in our repos would break support for Ubuntu 16.04 unless users are instructed to pull the latest (CMake 3.15) release from the new https://apt.kitware.com/ repository.

PeterBowman commented 4 years ago

See announcement and a brief CMake 3.12 migration guide at https://github.com/robotology/QA/issues/364.

PeterBowman commented 4 years ago

YCM 0.11 has been released today: https://github.com/robotology/ycm/releases/tag/v0.11.0.

PeterBowman commented 4 years ago

I'm turning this issue into an announcement. Since YCM already requires CMake 3.12, we strongly suggest to upgrade CMake accordingly. This is best achieved using the new Kitware PPA: ref (CMake 3.16.0). Also, a pip alternative exists: ref (CMake 3.15.3).

cc @roboticslab-uc3m

PeterBowman commented 4 years ago

Also, YCM 0.11 introduces the CMake option YCM_EP_INSTALL_DIR to change where the subprojects will be installed (default: install/). I'd consider this on superbuild repos, e.g. asibot-main.

traversaro commented 4 years ago

Also, YCM 0.11 introduces the CMake option YCM_EP_INSTALL_DIR to change where the subprojects will be installed (default: install/). I'd consider this on superbuild repos, e.g. asibot-main.

Just to let you know, the variable YCM_EP_INSTALL_DIR is quite experimental, and we did not test it extensively. In particular, I am not really sure if it works correctly YCM_EP_INSTALL_DIR is a prefix that also contains some libraries that are used as dependency of the superbuild.

PeterBowman commented 4 years ago

@traversaro that's good to know, thanks!

PeterBowman commented 4 years ago

https://github.com/roboticslab-uc3m/kinematics-dynamics/compare/4b1f60d...ee572ce

PeterBowman commented 4 years ago

Issues with CMake 3.15+: https://github.com/robotology/ycm/issues/262#issuecomment-579840557.

PeterBowman commented 4 years ago

Relevant repositories have been updated. Closing.