Build fails on ROS kinetic

[RaduCorcodel]

Hi,

I tried building on Kinetic today and got this error messages:

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp: In member function ‘void generic_control_toolbox::KDLManager::updateSolvers()’:
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1134:25: error: ‘class KDL::ChainIkSolverVel’ has no member named ‘updateInternalDataStructures’
     ikvel_[item.first]->updateInternalDataStructures();
                         ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1135:25: error: ‘class KDL::ChainFkSolverPos_recursive’ has no member named ‘updateInternalDataStructures’
     fkpos_[item.first]->updateInternalDataStructures();
                         ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1136:30: error: ‘class KDL::ChainJntToJacSolver’ has no member named ‘updateInternalDataStructures’
     jac_solver_[item.first]->updateInternalDataStructures();
                              ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1137:33: error: ‘class KDL::ChainDynParam’ has no member named ‘updateInternalDataStructures’
     dynamic_chain_[item.first]->updateInternalDataStructures();

On this system, there is this version of KDL installed.

ros-kinetic-orocos-kdl is already the newest version (1.3.2-1xenial-20201017-070916+0000).

On ROS Melodic it builds just fine. I tried updating all ROS packages but that didn't help. Any ideas on what else I could do?

Radu,

[diogoalmeida]

Might be that the kdl version from kinetic is just not compatible with the current version of the toolbox. The common way to address this would be to have a "kinetic" branch that hosts a version of the code that is compatible with that ros version. I can look into it, but cannot do it in the near future. If you have a version that works, I can create a kinetic branch and you can make a PR into it.

…

________________________________ From: RaduCorcodel <notifications@github.com> Sent: Wednesday, December 16, 2020 5:35:26 PM To: diogoalmeida/generic_control_toolbox <generic_control_toolbox@noreply.github.com> Cc: Subscribed <subscribed@noreply.github.com> Subject: [diogoalmeida/generic_control_toolbox] Build fails on ROS kinetic (#19) Hi, I tried building on Kinetic today and got this error messages: /homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp: In member function ‘void generic_control_toolbox::KDLManager::updateSolvers()’: /homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1134:25: error: ‘class KDL::ChainIkSolverVel’ has no member named ‘updateInternalDataStructures’ ikvel_[item.first]->updateInternalDataStructures(); ^ /homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1135:25: error: ‘class KDL::ChainFkSolverPos_recursive’ has no member named ‘updateInternalDataStructures’ fkpos_[item.first]->updateInternalDataStructures(); ^ /homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1136:30: error: ‘class KDL::ChainJntToJacSolver’ has no member named ‘updateInternalDataStructures’ jac_solver_[item.first]->updateInternalDataStructures(); ^ /homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1137:33: error: ‘class KDL::ChainDynParam’ has no member named ‘updateInternalDataStructures’ dynamic_chain_[item.first]->updateInternalDataStructures(); On this system, there is this version of KDL installed. ros-kinetic-orocos-kdl is already the newest version (1.3.2-1xenial-20201017-070916+0000). On ROS Melodic it builds just fine. I tried updating all ROS packages but that didn't help. Any ideas on what else I could do? Radu, — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#19>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AALOBTNPEOQDS4EWZ7RYYWTSVDVV5ANCNFSM4U6K25DQ>.

[RaduCorcodel]

Sounds good. I will roll back to the newest commit that still runs under kinetic and hopefully that fixes it. Thanks so much, I think that would work just fine.