diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2016-12-20 11:16:45 -0800 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2016-12-20 12:59:09 -0800 |
| commit | da73b4e40b2c89ca058b57de3a7e6614524ae1a7 (patch) | |
| tree | 0dd6f07cd9ab26b6ef5e258027062221519d5652 | |
| parent | 1509155f4dbca58fb38bd0d8467d041da1622ecc (diff) | |
| parent | 70ef94db2278da1b46d249c6fc6b3eac3dc324ae (diff) | |
| download | rust-da73b4e40b2c89ca058b57de3a7e6614524ae1a7.tar.gz rust-da73b4e40b2c89ca058b57de3a7e6614524ae1a7.zip | |
Rollup merge of #38471 - alexcrichton:run-debuginfo-tests, r=brson
rustbuild: Run debuginfo tests by default This fixes an accidental regression in rustbuild which stopped running debuginfo tests by default. Here we flag the test suites as `default(true)` to ensure that they're run on bots, for example.
| -rw-r--r-- | src/bootstrap/step.rs | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/bootstrap/step.rs b/src/bootstrap/step.rs index 5b3ce2ea5bc..c5898c1119a 100644 --- a/src/bootstrap/step.rs +++ b/src/bootstrap/step.rs @@ -267,16 +267,18 @@ pub fn build_rules(build: &Build) -> Rules { // nothing to do for debuginfo tests } else if build.config.build.contains("apple") { rules.test("check-debuginfo", "src/test/debuginfo") + .default(true) .dep(|s| s.name("libtest")) - .dep(|s| s.name("tool-compiletest").host(s.host)) + .dep(|s| s.name("tool-compiletest").target(s.host)) .dep(|s| s.name("test-helpers")) .dep(|s| s.name("debugger-scripts")) .run(move |s| check::compiletest(build, &s.compiler(), s.target, "debuginfo-lldb", "debuginfo")); } else { rules.test("check-debuginfo", "src/test/debuginfo") + .default(true) .dep(|s| s.name("libtest")) - .dep(|s| s.name("tool-compiletest").host(s.host)) + .dep(|s| s.name("tool-compiletest").target(s.host)) .dep(|s| s.name("test-helpers")) .dep(|s| s.name("debugger-scripts")) .run(move |s| check::compiletest(build, &s.compiler(), s.target, |