robotology-legacy / codyco-modules

Whole-body Compliant Dynamical Contacts in Cognitive Humanoids
www.codyco.eu
Other
19 stars 13 forks source link

torqueBalancing ini files not being installed #249

Closed fjandrad closed 6 years ago

fjandrad commented 7 years ago

The CMakeList in

codyco-modules/src/modules/

contains the following condition

if(CODYCO_USES_KDL)
add_subdirectory(torqueBalancing)

Since KDL is being deprecated by default it wont compile the torqueBalancing folder. This is an issue since the ini files from torqueBalancing are commonly used for the starting positions. As a result the ini files are not installed by default.

fjandrad commented 7 years ago

cc @diegoferigo @traversaro @gabrielenava

traversaro commented 7 years ago

It was my fault, sorry guys. It can be easily fixed by moving the if(CODYCO_USES_KDL) inside the torqueBalancing directory, guarding everything except the add_subdirectory(app) call.

traversaro commented 6 years ago

Apparently I forgot to fix this issue. Solution provided in https://github.com/robotology/codyco-modules/commit/5442a76c1a4c2bdb6bf4382432171423d6c81075 . cc @diegoferigo @gabrielenava