Skip to content

Conversation

le-horizon
Copy link
Contributor

@le-horizon le-horizon commented Aug 25, 2025

Somehow I need this change to be able to build wx_armor, otherwise, it complains when make.
Also updated README.

[ 30%] Building CXX object CMakeFiles/wx_armor_driver.dir/wx_armor/wx_armor_driver.cc.o
In file included from /home/horizon/interbotix_xs_driver/wx_armor/wx_armor/wx_armor_driver.cc:1:
/home/horizon/interbotix_xs_driver/wx_armor/wx_armor/wx_armor_driver.h:45:10: fatal error: dynamixel_workbench_toolbox/dynamixel_workbench.h: No such file or directory
45 | #include "dynamixel_workbench_toolbox/dynamixel_workbench.h"
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/wx_armor_driver.dir/build.make:76: CMakeFiles/wx_armor_driver.dir/wx_armor/wx_armor_driver.cc.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:117: CMakeFiles/wx_armor_driver.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

@le-horizon le-horizon requested a review from QuantuMope August 25, 2025 19:18
@le-horizon le-horizon changed the title add missed 3rd party dependencies: dynamixel add missed 3rd party dependencies: dynamixel; update README Aug 25, 2025
@QuantuMope
Copy link
Contributor

An older PR includes this information that was never merged.
#25

In that PR, I also created a script that installs all dependencies and builds wx_armor.

Want me to test it on gail8? Should be a quick cherry-pick

@QuantuMope
Copy link
Contributor

QuantuMope commented Aug 25, 2025

Also regarding several of the cmake changes. Those can be removed by setting

export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH

in your .bashrc. This is mentioned in the above PR.

@le-horizon
Copy link
Contributor Author

Also regarding several of the cmake changes. Those can be removed by setting

export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH

in your .bashrc. This is mentioned in the above PR.

Ah, this is not very user friendly. It's perhaps better to make a more robust build script?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants