Skip to content

Commit

Permalink
Auto merge of #11582 - ehuss:enable-source_config_env, r=weihanglo
Browse files Browse the repository at this point in the history
Enable source_config_env test on Windows

This enables the `advaned_env::source_config_env` test on Windows. The issue with `Command` modifying the case of environment variables was fixed by rust-lang/rust#85270.
  • Loading branch information
bors committed Jan 14, 2023
2 parents 48e0c53 + 73f19a0 commit 88f1429
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions tests/testsuite/advanced_env.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,6 @@
use cargo_test_support::{paths, project, registry::Package};

#[cargo_test]
// I don't know why, but `Command` forces all env keys to be upper case on
// Windows. Seems questionable, since I think Windows is case-preserving.
#[cfg_attr(windows, ignore = "broken due to not preserving case on Windows")]
fn source_config_env() {
// Try to define [source] with environment variables.
let p = project()
Expand Down

0 comments on commit 88f1429

Please sign in to comment.