Skip to content

feat: add runner to build configuration - #2702

Open
baszalmstra wants to merge 1 commit into
prefix-dev:mainfrom
baszalmstra:feat/configuration-runner
Open

feat: add runner to build configuration#2702
baszalmstra wants to merge 1 commit into
prefix-dev:mainfrom
baszalmstra:feat/configuration-runner

Conversation

@baszalmstra

Copy link
Copy Markdown
Contributor

The Runner API now exists in rattler_build_script, but the build configuration has no place to carry a runner. Library integrations that already pass Configuration through the build and test code would otherwise need to keep the execution backend as separate state.

This adds an Arc<dyn Runner> to Configuration and a with_runner builder method. The default stays LocalRunner, so existing callers keep the current behavior. CLI builds construct the local runner from the selected environment isolation mode, keeping those settings together. The runner types are also re-exported through the core script API.

This is configuration and API plumbing only. Existing script execution is unchanged.

Testing

  • pixi run cargo-fmt-check
  • pixi run cargo check --workspace --all-targets
  • pixi run cargo clippy -p rattler_build_core --all-targets --all-features -- -D warnings -Dclippy::dbg_macro
  • pixi run cargo check --manifest-path py-rattler-build/rust/Cargo.toml
  • pixi run test

@baszalmstra
baszalmstra requested a review from wolfv July 30, 2026 08:51
@baszalmstra
baszalmstra marked this pull request as ready for review July 31, 2026 07:21
}

#[cfg(test)]
mod tests {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure if these tests add a lot of value lol

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

Successfully merging this pull request may close these issues.

2 participants