search

ubi-agni / mujoco_ros_pkgs

Wrappers, tools and additional API's for using ROS with MuJoCo
68 stars 12 forks source link

Fix findmujoco.cmake for build-installed mujoco #13

Closed DavidPL1 closed 1 year ago

DavidPL1 commented 1 year ago

With mujoco 2.2.2 built from scratch find_package(mujoco REQUIRED) does not return any errors, but no mujoco properties get set and building mujoco_ros fails.

Edit: My mistake, this actually just ensures the find_package call fails and triggers setting the properties manually, which does work in this case. Is there any reason to keep the first line and check for mujoco_FOUND or is there a case when this fails to resolve the properties correctly?

rhaschke commented 1 year ago

Have a look at #14 for a cleaner solution.