From 4c2b90f2c23aeaa05a50774d615877d6ecad5561 Mon Sep 17 00:00:00 2001 From: Alex Crichton Date: Fri, 16 Dec 2016 16:42:31 -0800 Subject: [PATCH] rustbuild: Delete extraneous pretty test suites The pretty printer in the compiler isn't really heavily used, and there's not really many stability guarantees around it. In that sense we don't need to exhaustively test it on every single build of Rust. Additionally these test suites typically take around 10-15 minutes of each CI run, which certainly adds up over time. This commit deletes the pretty passes over the standard test suites, e.g. run-pass, run-pass-fulldeps, etc. The `src/test/pretty` folder is still kept and we can retain test cases there, but this should cut down on the vast majority of pretty test case related test time. --- src/bootstrap/step.rs | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/bootstrap/step.rs b/src/bootstrap/step.rs index 35eb98e4723f2..1b0e3e351ef69 100644 --- a/src/bootstrap/step.rs +++ b/src/bootstrap/step.rs @@ -258,12 +258,6 @@ pub fn build_rules(build: &Build) -> Rules { "incremental"); suite("check-ui", "src/test/ui", "ui", "ui"); suite("check-pretty", "src/test/pretty", "pretty", "pretty"); - suite("check-pretty-rpass", "src/test/run-pass/pretty", "pretty", - "run-pass"); - suite("check-pretty-rfail", "src/test/run-pass/pretty", "pretty", - "run-fail"); - suite("check-pretty-valgrind", "src/test/run-pass-valgrind", "pretty", - "run-pass-valgrind"); } if build.config.build.contains("msvc") { @@ -308,10 +302,6 @@ pub fn build_rules(build: &Build) -> Rules { "compile-fail", "compile-fail-fulldeps"); suite("check-rmake", "src/test/run-make", "run-make", "run-make"); suite("check-rustdoc", "src/test/rustdoc", "rustdoc", "rustdoc"); - suite("check-pretty-rpass-full", "src/test/run-pass-fulldeps", - "pretty", "run-pass-fulldeps"); - suite("check-pretty-rfail-full", "src/test/run-fail-fulldeps", - "pretty", "run-fail-fulldeps"); } for (krate, path, _default) in krates("std_shim") {