about summary refs log tree commit diff
path: root/compiler/rustc_interface/src
diff options
context:
space:
mode:
authorMatthias Krüger <476013+matthiaskrgr@users.noreply.github.com>2025-04-10 11:10:15 +0200
committerGitHub <noreply@github.com>2025-04-10 11:10:15 +0200
commit2b0e47eb4c9d27fc9b0f54608c1e44300d84223e (patch)
tree083b2ede49f9f753cb12bbb81e3dc6a0ae6a6fa7 /compiler/rustc_interface/src
parent7f08e638c496a91410d78d58253108f46f1ea604 (diff)
parent34e97592f4dc8f4e44a309e3bf69cdcea22fc047 (diff)
downloadrust-2b0e47eb4c9d27fc9b0f54608c1e44300d84223e.tar.gz
rust-2b0e47eb4c9d27fc9b0f54608c1e44300d84223e.zip
Rollup merge of #139507 - Zalathar:trim-env-name, r=jieyouxu
compiletest: Trim whitespace from environment variable names

When a test contains a directive like `//@ exec-env: FOO=bar`, compiletest currently includes that leading space in the name of the environment variable, so it is defined as ` FOO` instead of `FOO`.

This is an annoying footgun that is pretty much never intended, especially since most other directives *do* trim whitespace. So let's get rid of it by trimming the environment variable name.

Values remain untrimmed, since there could conceivably be a use-case for values with leading space, but perhaps we'll end up trimming values too in the future.

Recently observed in https://github.com/rust-lang/rust/pull/138603#issuecomment-2783709359.

Fixes #132990.
Supersedes #133148.

---

try-job: test-various
Diffstat (limited to 'compiler/rustc_interface/src')
0 files changed, 0 insertions, 0 deletions