Skip to content

fix(desktop): locate sensing server on Windows - #1570

Draft
alloutflo wants to merge 1 commit into
ruvnet:mainfrom
alloutflo:codex/issue-1517-windows-binary
Draft

fix(desktop): locate sensing server on Windows#1570
alloutflo wants to merge 1 commit into
ruvnet:mainfrom
alloutflo:codex/issue-1517-windows-binary

Conversation

@alloutflo

Copy link
Copy Markdown

Summary

  • use sensing-server.exe and where on Windows while retaining the Unix names elsewhere
  • select the first non-empty result when where returns multiple CRLF-separated matches
  • cover both platform naming and multi-match parsing with regression tests

This repairs the sibling-binary and PATH lookup strategies without changing explicit custom paths.

Validation

  • cargo test --manifest-path v2/Cargo.toml -p wifi-densepose-desktop commands::server::tests --no-default-features
  • git diff --check upstream/main...HEAD

The Windows conventions are exercised platform-independently; no Windows runtime or packaging claim is made.

Closes #1517

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.

Desktop: sensing-server binary lookup can never succeed on Windows (missing .exe suffix, uses 'which')

1 participant