Skip to content

fix(xtask): answer PowerShell compatibility question for cmd.exe /c spawn fix#80

Closed
mohamedmansour with Copilot wants to merge 1 commit into
fix_command_windowsfrom
copilot/sub-pr-79
Closed

fix(xtask): answer PowerShell compatibility question for cmd.exe /c spawn fix#80
mohamedmansour with Copilot wants to merge 1 commit into
fix_command_windowsfrom
copilot/sub-pr-79

Conversation

Copilot AI commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

A reviewer asked whether the cmd.exe /c wrapping fix works when cargo xtask is invoked from a PowerShell terminal.

Clarification

The fix is terminal-agnostic: the cmd.exe /c wrapper is applied by the compiled Rust binary at process-spawn time, not by the user's shell. Whether the user runs from CMD, PowerShell, or Windows Terminal, the Rust xtask binary is what calls Command::new("cmd").arg("/c").arg(cmd) — the invoking shell is irrelevant.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix launching non-exe commands through cmd.exe on Windows fix(xtask): answer PowerShell compatibility question for cmd.exe /c spawn fix Mar 6, 2026
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