Skip to content

Commit

Permalink
Auto merge of #1126 - akiss77:pr-test_cargo_compile, r=alexcrichton
Browse files Browse the repository at this point in the history
For two tests, the `Cargo.toml`s and the `cfg`s don't match. E.g., the first test expects a successful build on all linux platforms but its `toml` is prepared for Intel linuxes only. The second test is very similar.

Here, the `cfg`s are amended to match the `toml`s.
  • Loading branch information
bors committed Jan 5, 2015
2 parents fc2e15e + c8f4ecc commit 56e9314
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/test_cargo_compile.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1357,7 +1357,7 @@ Caused by:
"));
});

#[cfg(target_os = "linux")]
#[cfg(all(any(target_arch = "x86", target_arch = "x86_64"), target_os = "linux"))]
test!(cargo_platform_specific_dependency {
let p = project("foo")
.file("Cargo.toml", r#"
Expand Down Expand Up @@ -1398,7 +1398,7 @@ test!(cargo_platform_specific_dependency {
execs().with_stdout("test passed\n"));
});

#[cfg(not(target_os = "linux"))]
#[cfg(not(all(any(target_arch = "x86", target_arch = "x86_64"), target_os = "linux")))]
test!(cargo_platform_specific_dependency {
let p = project("foo")
.file("Cargo.toml", r#"
Expand Down

0 comments on commit 56e9314

Please sign in to comment.