Skip to content

Add bazel build support#529

Merged
shameekganguly merged 2 commits intogz-sensors9from
shameek/bazel
Jun 25, 2025
Merged

Add bazel build support#529
shameekganguly merged 2 commits intogz-sensors9from
shameek/bazel

Conversation

@shameekganguly
Copy link
Copy Markdown
Contributor

@shameekganguly shameekganguly commented Jun 24, 2025

🎉 New feature

Summary

Adds bazel build support for gz-sensors. The bzlmod setup is similar to sdformat (added in gazebosim/sdformat#1493).

Also adds bazel CI.

Some include paths in test/test_config.hh and src/DopplerVelocityLog.cc had to be adjusted to satisfy layering check.

Test it

$ bazel test :all
$ bazel build --action_env=CC=/usr/bin/clang  :all

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@shameekganguly shameekganguly requested a review from mjcarroll June 25, 2025 00:08
],
)

gz_configure_header(
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not needed in this PR, but moving to @gz-common//testing should help get rid of this header.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call, I'll open a fix.

@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Jun 25, 2025
@shameekganguly shameekganguly merged commit dc4ea44 into gz-sensors9 Jun 25, 2025
14 checks passed
@shameekganguly shameekganguly deleted the shameek/bazel branch June 25, 2025 15:24
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏛️ ionic Gazebo Ionic

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants