Skip to content

Make the runner build on windows#297

Open
GuillaumeLagrange wants to merge 13 commits intomainfrom
cod-2510-make-a-rust-benchmark-run-on-windows-no-wsl
Open

Make the runner build on windows#297
GuillaumeLagrange wants to merge 13 commits intomainfrom
cod-2510-make-a-rust-benchmark-run-on-windows-no-wsl

Conversation

@GuillaumeLagrange
Copy link
Copy Markdown
Contributor

@GuillaumeLagrange GuillaumeLagrange commented Apr 16, 2026

The runner is still very much non functional outside of unix, this PR just does preliminary work and adds safeguards for us for when we do want to spend the time on making it run on windows.

Lower prio PR compared to the macos ones

On macos a symlink breaks the existing test creating an infinite loop
where `instropected_go` detects itself as the real `go` executable.

Fixed by changing the behavior to what instrospected_node does.
Fixes compilation warnings in exec-harness's build script.
…tor support gates

Replaces the loose `os` + `os_version` strings on `SystemInfo` with an
`Os` enum.

Adds `Executor::support_level(&Os)` so each executor declares
its own OS matrix: `run_executor` bails on `Unsupported`, skips `setup()`
on `RequiresManualSetup`, and runs it on `FullySupported`.
These tests panic on macOS, and for now are not relevant since we do not
support profiling on macOS.
When we do support it, we should have os-specific tests
Had a lot of issues building directly within the shared folder, this
script can be thrown away if it's no longer relevant.
`pid_t` is not available on msvc in the libc crate.
This aligns the function signature with the actual implementation.
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 16, 2026

Merging this PR will not alter performance

✅ 4 untouched benchmarks


Comparing cod-2510-make-a-rust-benchmark-run-on-windows-no-wsl (a2f1fbb) with cod-2459-be-able-to-run-all-integrations-on-macos (2fdce4e)

Open in CodSpeed

Note: the runner is still absolutely not functional, we just have this
test to avoid adding more debt passively before making it functional.
@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2510-make-a-rust-benchmark-run-on-windows-no-wsl branch from 453175f to a2f1fbb Compare April 16, 2026 09:07
@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2459-be-able-to-run-all-integrations-on-macos branch 4 times, most recently from 3c21514 to 25ee001 Compare April 16, 2026 14:43
@GuillaumeLagrange GuillaumeLagrange removed the request for review from not-matthias April 16, 2026 15:02
Base automatically changed from cod-2459-be-able-to-run-all-integrations-on-macos to main April 16, 2026 15:50
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.

1 participant