search

whoenig / crazyflie_ros

ROS Driver for Bitcraze Crazyflie
MIT License
192 stars 205 forks source link

Update crazyflie_tools, disabling safelink #186

Closed acxz closed 4 years ago

acxz commented 4 years ago

Resolves #185 add updated crazyflie_tools with updated crazyflie_cpp (disabling safelink)

whoenig commented 4 years ago

I am fine with merging this, but technically I am not sure why this solves #185. In Crazyflie_ros, crazyflie_cpp is already a submodule (which seems to have the latest version) and if you build using catkin_make it should use the crazyflie_cpp that is directly a submodule in this repository, not the one that is a submodule in crazyflie_tools.

acxz commented 4 years ago

Yeah I found it a bit odd as well. The build itself works fine, but somewhere in the logic the crazyflie_cpp in crazyflie_tools gets run and that is causing the error I face.

acxz commented 4 years ago

Thanks for the real fast responses @whoenig!