about summary refs log tree commit diff
path: root/compiler/rustc_codegen_llvm/src
diff options
context:
space:
mode:
author许杰友 Jieyou Xu (Joe) <39484203+jieyouxu@users.noreply.github.com>2025-02-28 22:29:53 +0800
committerGitHub <noreply@github.com>2025-02-28 22:29:53 +0800
commit4606610b2136cbaba4ab58658042c643044e3511 (patch)
tree47ddebe4bb67c0f4ec9ad39b01a7339068f27dc4 /compiler/rustc_codegen_llvm/src
parent50ed7f974b167ddaed825db269698f3c134db474 (diff)
parent4fcebee60a1f8fc069dd546aa8d7c93983c57ada (diff)
downloadrust-4606610b2136cbaba4ab58658042c643044e3511.tar.gz
rust-4606610b2136cbaba4ab58658042c643044e3511.zip
Rollup merge of #137673 - ChrisDenton:search-path-bug, r=dtolnay
Fix Windows `Command` search path bug

Currently `Command::new` on Windows works differently depending on whether any environment variable is set. For example,

```rust
// Searches for "myapp" in the application and system paths first (aka Windows native behaviour).
Command::new("myapp").spawn();

// Search for "myapp" in `PATH` first
Command::new("myapp").env("a", "b").spawn();
```

This is a bug because the search path should only change if `PATH` is changed for the child (i.e. `.env("PATH", "...")`).

This was discussed in a libs-api meeting where the exact semantics of `Command::new` was not decided but there seemed to be broad agreement that this particular thing is just a bug that can be fixed.

r? libs-api
Diffstat (limited to 'compiler/rustc_codegen_llvm/src')
0 files changed, 0 insertions, 0 deletions