Skip to content

build: use Ninja for workspace builds#1021

Draft
jaagut wants to merge 2 commits into
agent/build-benchmark-baselinefrom
agent/build-benchmark-ninja
Draft

build: use Ninja for workspace builds#1021
jaagut wants to merge 2 commits into
agent/build-benchmark-baselinefrom
agent/build-benchmark-ninja

Conversation

@jaagut

@jaagut jaagut commented Jul 17, 2026

Copy link
Copy Markdown
Member

Summary

Switch the default and robot CMake generator from Unix Makefiles to Ninja.

Proposed changes

  • Use the already locked Ninja dependency for default workspace builds.
  • Use the same generator for robot workspace builds.

Impact

Ninja reduces build orchestration overhead and can improve parallel scheduling without changing the compiler or generated runtime code. Existing Makefile build directories must be cleaned once before using this branch.

Stack

This PR is based on agent/build-benchmark-baseline so its CI timings can be compared with the baseline PR.

Validation

  • Parsed pixi.toml with Python's TOML parser.
  • Listed Pixi tasks successfully.
  • Built bitbots_head_mover successfully with the Ninja generator.
  • git diff --check

@Flova

Flova commented Jul 18, 2026

Copy link
Copy Markdown
Member

We intentionally opted out of ninja as there where some bugs and due to a controversial opinion of its creator all logs are written to stderr which makes the build really verbose if we want to see build errors at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

2 participants