ok/jj
1
0
Fork 0
forked from mirrors/jj
Code Pull requests Activity

tests: use if cfg!(..) instead of #[cfg(..)] to insert Windows-specific config

Browse source 
The code compiles on all platforms, so I think `if cfg!(..)` is better.
This commit is contained in:
Yuya Nishihara 2023-10-07 22:22:37 +09:00
parent aea135cb8f
commit d94c325d6c
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
cli/tests/common/mod.rs
View file

@ -16,6 +16,7 @@ use std::cell::RefCell;
use std::collections::HashMap;
use std::path::{Path, PathBuf};
use itertools::Itertools as _;
use regex::{Captures, Regex};
use tempfile::TempDir;
use testutils;
@ -106,10 +107,7 @@ impl TestEnvironment {
cmd.env("JJ_TIMESTAMP", timestamp.to_rfc3339());
cmd.env("JJ_OP_TIMESTAMP", timestamp.to_rfc3339());
#[cfg(all(windows, target_env = "gnu"))]
{
use itertools::Itertools as _;
if cfg!(all(windows, target_env = "gnu")) {
// MinGW executables cannot run without `mingw\bin` in the PATH (which we're
// clearing above), so we add it again here.
if let Ok(path_var) = std::env::var("PATH").or_else(|_| std::env::var("Path")) {