about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorEric Huss <eric@huss.org>2018-05-15 16:39:21 -0700
committerEric Huss <eric@huss.org>2018-05-15 16:39:21 -0700
commita90a9632659a11264cf9a27273ecc83d52cb4e9f (patch)
treed91eea1a08b7069fc1f1e4929e5e826ae6a6908a /src
parent27acb9b3c033e1dd74cc72310a5a346527d3e704 (diff)
downloadrust-a90a9632659a11264cf9a27273ecc83d52cb4e9f.tar.gz
rust-a90a9632659a11264cf9a27273ecc83d52cb4e9f.zip
Fix run-make wasm tests
Fixes #50711
Diffstat (limited to 'src')
-rw-r--r--src/bootstrap/test.rs8
-rw-r--r--src/test/run-make-fulldeps/cross-lang-lto/Makefile (renamed from src/test/run-make/cross-lang-lto/Makefile)2
-rw-r--r--src/test/run-make-fulldeps/cross-lang-lto/lib.rs (renamed from src/test/run-make/cross-lang-lto/lib.rs)0
-rw-r--r--src/test/run-make-fulldeps/cross-lang-lto/main.rs (renamed from src/test/run-make/cross-lang-lto/main.rs)0
-rw-r--r--src/tools/compiletest/src/main.rs2
5 files changed, 6 insertions, 6 deletions
diff --git a/src/bootstrap/test.rs b/src/bootstrap/test.rs
index fb8ea5fb341..1f81a617237 100644
--- a/src/bootstrap/test.rs
+++ b/src/bootstrap/test.rs
@@ -838,7 +838,7 @@ test!(RunFailFullDepsPretty {
     host: true
 });
 
-host_test!(RunMake {
+default_test!(RunMake {
     path: "src/test/run-make",
     mode: "run-make",
     suite: "run-make"
@@ -1041,7 +1041,7 @@ impl Step for Compiletest {
 
             // Only pass correct values for these flags for the `run-make` suite as it
             // requires that a C++ compiler was configured which isn't always the case.
-            if !builder.config.dry_run && mode == "run-make" {
+            if !builder.config.dry_run && suite == "run-make-fulldeps" {
                 let llvm_components = output(Command::new(&llvm_config).arg("--components"));
                 let llvm_cxxflags = output(Command::new(&llvm_config).arg("--cxxflags"));
                 cmd.arg("--cc").arg(builder.cc(target))
@@ -1054,13 +1054,13 @@ impl Step for Compiletest {
                 }
             }
         }
-        if mode == "run-make" && !builder.config.llvm_enabled {
+        if suite == "run-make-fulldeps" && !builder.config.llvm_enabled {
             builder.info(
                 &format!("Ignoring run-make test suite as they generally don't work without LLVM"));
             return;
         }
 
-        if mode != "run-make" {
+        if suite != "run-make-fulldeps" {
             cmd.arg("--cc").arg("")
                .arg("--cxx").arg("")
                .arg("--cflags").arg("")
diff --git a/src/test/run-make/cross-lang-lto/Makefile b/src/test/run-make-fulldeps/cross-lang-lto/Makefile
index 925f686fe11..f5480178db0 100644
--- a/src/test/run-make/cross-lang-lto/Makefile
+++ b/src/test/run-make-fulldeps/cross-lang-lto/Makefile
@@ -2,7 +2,7 @@
 # min-llvm-version 4.0
 # ignore-mingw
 
--include ../../run-make-fulldeps/tools.mk
+-include ../tools.mk
 
 # This test makes sure that the expected .llvmbc sections for use by
 # linker-based LTO are available in object files when compiling with
diff --git a/src/test/run-make/cross-lang-lto/lib.rs b/src/test/run-make-fulldeps/cross-lang-lto/lib.rs
index b2a5b946160..b2a5b946160 100644
--- a/src/test/run-make/cross-lang-lto/lib.rs
+++ b/src/test/run-make-fulldeps/cross-lang-lto/lib.rs
diff --git a/src/test/run-make/cross-lang-lto/main.rs b/src/test/run-make-fulldeps/cross-lang-lto/main.rs
index ccd34c9e4db..ccd34c9e4db 100644
--- a/src/test/run-make/cross-lang-lto/main.rs
+++ b/src/test/run-make-fulldeps/cross-lang-lto/main.rs
diff --git a/src/tools/compiletest/src/main.rs b/src/tools/compiletest/src/main.rs
index e8b140978b5..0899ebd5d80 100644
--- a/src/tools/compiletest/src/main.rs
+++ b/src/tools/compiletest/src/main.rs
@@ -614,8 +614,8 @@ pub fn is_test(file_name: &OsString) -> bool {
 }
 
 pub fn make_test(config: &Config, testpaths: &TestPaths) -> test::TestDescAndFn {
-
     let early_props = if config.mode == Mode::RunMake {
+        // Allow `ignore` directives to be in the Makefile.
         EarlyProps::from_file(config, &testpaths.file.join("Makefile"))
     } else {
         EarlyProps::from_file(config, &testpaths.file)