-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Move to catkin build system #76
Comments
Can you please confirm that pull request #70 works for you? |
your fork works for me on 14.04, indego. Thanks! |
@icoderaven, Your fork doesn't compile for me on 14.04, Indigo. I get the error:
|
Hm, this is a common issue with the message headers not compiling before the executables. I have a version of the CMakelists.txt that forces the build of the messages first. A temporary way to get it to build is to use a smaller number of parallel threads while building the package. I'll see if I can push that commit sometime. |
Yeah, googling found me this similar question, which mentions this bug in catkin. I worked around it by running catkin twice like:
which feels a little hacky. Yes, please push your fixed CMakeLists.txt when you're able. |
cd lsd_slam/ git checkout catkin response should : LICENSE lsd_slam lsd_slam_core lsd_slam_viewer README.md Then add "run_depend>cmake_modules</run_depend" in "package.xml" both in the folder "lsd_slam/lsd_slam_viewer" and in the folder "lsd_slam/lsd_slam_core". In "CMakeFiles.txt" both in the folder "lsd_slam/lsd_slam_viewer" and in the folder "lsd_slam/lsd_slam_core" add: check the line below (X11 may be mising ) Then u can build your workspace by Then again if got some error of header file missing then "CMakeFiles.txt" in the folder "lsd_slam/lsd_slam_viewer" add: |
I have the next error: |
@oblivione @JakobEngel @icoderaven following the same steps, it gave this error [Eigen]: Could not find a package configuration file provided by "Eigen" with any EigenConfig.cmake Add the installation prefix of "Eigen" to CMAKE_PREFIX_PATH or set |
You must set a path to Eigen library. |
Hi @oblivione, Thanks |
I'm getting similar error did you fixed it? |
Please port to catkin build system to work with newer ROS installs.
The text was updated successfully, but these errors were encountered: