search

robotology / robotology-superbuild

CMake/YCM-based superbuild to simplify the build process of robotology projects.
48 stars 48 forks source link

Move yarp-matlab-bindings under CORE profile #240

Closed kouroshD closed 4 years ago

kouroshD commented 5 years ago

Currently,yarp-matlab-bindings option is under Dynamics profile here, since it has been used mostly by people who use Dynamics profile. Since, some users may want to use this option but not necessarily Dynamics repositories, I suggest either we move it under CORE profile here, or we make it as cmake_dependent_option in superbuild.

cc @fjandrad @traversaro

diegoferigo commented 5 years ago

:+1: Yarp is in CORE, if Matlab is found in the system I don't see any reason why the bindings should not be in core too.

fjandrad commented 5 years ago

+1 for core xD

traversaro commented 4 years ago

Ok for me, for anyone interested: feel free to open a PR to change the README and the CMake files.

traversaro commented 4 years ago

Fixed by @kouroshD in https://github.com/robotology/robotology-superbuild/pull/297 .