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

Can't compile when including gz/sim.hh, not enough include paths exported #1971

Closed
scpeters opened this issue Apr 26, 2023 · 1 comment · Fixed by #1978
Closed

Can't compile when including gz/sim.hh, not enough include paths exported #1971

scpeters opened this issue Apr 26, 2023 · 1 comment · Fixed by #1978
Labels
bug Something isn't working

Comments

@scpeters
Copy link
Member

Environment

  • OS Version: Ubuntu 20.04, 22.04, macOS
  • Source or binary build? source build of gz-sim7, tested with 4c57ea4

Description

  • Expected behavior: I should be able to target_link_libraries against gz-sim7::core and #include <gz/sim.hh> and build successfully
  • Actual behavior: it fails to find some referenced header files

Steps to reproduce

  1. Checkout 4c57ea4
  2. Configure using cmake
  3. Build the INTEGRATION_include_sim_hh target
  4. Observer failure to build (following log from local test on macOS):
[100%] Building CXX object test/integration/CMakeFiles/INTEGRATION_include_sim_hh.dir/include_sim_hh.cc.o
In file included from test/integration/include_sim_hh.cc:20:
In file included from build/include/gz/sim.hh:144:
include/gz/sim/physics/Events.hh:24:10: fatal error: 'gz/physics/ContactProperties.hh' file not found
#include <gz/physics/ContactProperties.hh>
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Output

See Github Actions workflow output: https://github.com/gazebosim/gz-sim/actions/runs/4794159738

@scpeters scpeters added the bug Something isn't working label Apr 26, 2023
@scpeters
Copy link
Member Author

I believe this is causing ABI checker builds targeting the ign-gazebo3 branch to fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant