orocos / soem

ROS package wrapping the Simple Open EtherCAT Master (SOEM) from https://github.com/OpenEtherCATsociety/SOEM
http://wiki.ros.org/soem
77 stars 62 forks source link

Possible Improvement: Using SOEM as Submodule #11

Closed rkeatin3 closed 5 years ago

rkeatin3 commented 7 years ago

I was having trouble compiling this package in ROS to work with robotiq_ethercat, so I ended up starting from scratch with the base SOEM repo. I got it to compile with minimal work by creating a ROS package with the following files and SOEM as a submodule:

CMakeLists.txt package.xml

Could this help resolve some of the other issues for this package? Any other feedback?

mgruhler commented 5 years ago

@rkeatin3 thanks for the suggestion. There is a problem in the directory layout of the soem headers, though, see #17, especially with how to install it. How to use the latest soem release is tracked in #16, i linked to this issue therein, as I like the submodule idea.

Without knowing the details, I think that your problems could have been resolved using the workaround described in the Usage section of the README.md.

I'll close this, but feel free to reopen if you feel your issue is not tracked enough in the linked issues above.