-
Notifications
You must be signed in to change notification settings - Fork 197
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
Compile error while building tf2_kdl
with BUILD_TESTING
on - windows 10
#561
Comments
tf2_kdl
with BUILD_TESTING
ontf2_kdl
with BUILD_TESTING
on - windows 10
The thing is that the error is coming from orocos_kdl, not from geometry2. So it seems like a problem in orocos_kdl (or orocos_kdl_vendor), not in geometry2. That said, we do compile and run these tests nightly on Windows, and we haven't seen the same problem (see https://ci.ros2.org/view/nightly/job/nightly_win_deb/2538/console , for instance). So I'm guessing that somehow your installation differs from ours somehow. How did you install Eigen and orocos_kdl? |
I'm not a CMake expert, but I think the Eigen from choco (3.3.4) doesn't have exported targets. Therefore it might be the reason why it doesn't work. The "C:\ProgramData\chocolatey\lib\eigen\share\cmake" directory contains only:
I see there are no |
Yes, that is definitely true, and a longstanding issue that we haven't gotten around to fixing. But the thing is that we build these tests nightly, so I'm still a bit confused on why it didn't work for you. Probably needs a deeper look into the exports and the problem to figure out exactly what is going on. |
Hi I installed ros2 humble for windows 10 using the binary and orocos_kdl is installed as part of humble. also installed the packages the same way with choco from here: https://docs.ros.org/en/humble/Installation/Windows-Install-Binary.html#install-dependencies
But no matter what I try I keep getting the same error: I did find this from 2013, https://www.orocos.org/forum/orocos/orocos-users/eigen3-include-problem-kdl, I did check that findeigen3.cmake is in kdl and a couple of other places. |
Bug report
Required Info:
BUILD_TESTING
Steps to reproduce issue
Build
geometry2
package withBUILD_TESTING
on.Expected behavior
Build succeeded
Actual behavior
Compile error for
tf2_kdl
package:Additional information
Looks like Eigen is not added to include dirs. Adding it fixes the issue:
The text was updated successfully, but these errors were encountered: