-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue when exporting a new package with SOEM #22
Comments
Actually, exporting If your problem is "only" automated tests, you could "silence" What should work is to include find_package(catkin REQUIRED ...)
find_package(soem REQUIRED)
catkin_package(
...
DEPENDS
soem
...
)
include_directories(
...
${soem_INCLUDE_DIRS}
${soem_INCLUDE_DIRS}/soem
)
target_link_libraries(${PROJECT_NAME} ${catkin_LIBRARIES} ${soem_LIBRARIES}) and then only Other than that, I'd like to link to #17 which when (if) a proper solution is available would also solve your issue. |
Thank you for the suggestion. I tried it out and here are my observations:
I got the ROS version of SOEM. Could that be the reason why it is not a system package? I'm not sure how this is handled behind the scene. I will follow #17 too. |
about 1: This seems to be a linker error. Did you add about 2: I'm not sure how This package is a wrapped version of SOEM for the use with ROS. You could also try the original SOEM (linked in the description and the README), but will probably run into a lot of the same issues you are having here :-) As a last resort, I just learned that you can also ignore |
@wongkaiweng I just sent #23 which (among others) includes hopefully a solution to your problem. Actually, it is not even required/possible to specify |
#24 contains a similar solution. |
Hi,
I am trying to set up a new package (
pkg_a
) that depends on SOEM. I got that working properly. The problem appears when I use this new package with another package (pkg_b
), I have to add${soem_INCLUDE_DIRS}/soem
in theinclude_directories
ofpkg_b
again.I am looking to find a solution that this is taken care of in
pkg_a
. Since I cannot add${soem_INCLUDE_DIRS}/soem
incatkin_package INCLUDE_DIRS
ofpkg_a
(I would get a non-package include path error fromcatkin_lint
), I am curious if there is a way to do this.Thanks!
The text was updated successfully, but these errors were encountered: