Skip to content
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

compilation error #12

Open
qlibp opened this issue Feb 13, 2023 · 5 comments
Open

compilation error #12

qlibp opened this issue Feb 13, 2023 · 5 comments

Comments

@qlibp
Copy link

qlibp commented Feb 13, 2023

The compilation order of CMake's add_subdirectory() may result in compilation error.

image

velodyne_msgs are found by catkin, the cmake invoke compilation on tools folder before common folder, which results in the following error(I guess)

@gaoxiang12
Copy link
Owner

Looks like I should let the sad.common depend on lidar messages. I'll look into it.

@qlibp
Copy link
Author

qlibp commented Feb 13, 2023

Already verified, once I comment out the add_subdirectory(tools) line first, and compile the code, the messages will generate successfully.

@qlibp
Copy link
Author

qlibp commented Feb 13, 2023

Also, can't build objects with g2o successfully. I try two ways to build the entire project.

  1. build at the {project_src}/build, and trigger an error
  2. I run cmake .. & make inside thirdparty/g2o/build and I can't build the whole project still.

@qlibp
Copy link
Author

qlibp commented Feb 13, 2023

Also, can't build objects with g2o successfully. I try two ways to build the entire project.

  1. build at the {project_src}/build, and trigger an error
  2. I run cmake .. & make inside thirdparty/g2o/build and I can't build the whole project still.

I don't know which g2o/config.h am I missing, I grep the entire folder and find out that it's within thirdparty/g2o/build/
So, I add an additional line of include_directory() in cmake/packages.cmake to make it compiled.

If I get it wrong, plz correct me.

@aqiuxx
Copy link

aqiuxx commented Mar 4, 2023


sudo apt install ros-noetic-velodyne-msgs

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

No branches or pull requests

3 participants