about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2019-11-05 20:57:00 +0000
committerbors <bors@rust-lang.org>2019-11-05 20:57:00 +0000
commitd1e04902396e2cdcca884c716b0b2db89f0b4b5a (patch)
tree3df837df24050c1248aafbccf3f515e42eaa9b51
parent000c3ff278852788bc88ce6a911050d02d39af93 (diff)
parentb8c2cb17e6136e45d61802dc16e33fb5d2427e24 (diff)
downloadrust-d1e04902396e2cdcca884c716b0b2db89f0b4b5a.tar.gz
rust-d1e04902396e2cdcca884c716b0b2db89f0b4b5a.zip
Auto merge of #4773 - lzutao:rls/print-something, r=Manishearth
build: avoid timing out in Travis

cc #4770
changelog: none
-rw-r--r--.travis.yml1
-rwxr-xr-xci/integration-tests.sh5
-rw-r--r--clippy_lints/src/mutex_atomic.rs2
-rw-r--r--clippy_lints/src/utils/mod.rs2
4 files changed, 6 insertions, 4 deletions
diff --git a/.travis.yml b/.travis.yml
index 7be9174843b..6ab711b723b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -90,6 +90,7 @@ matrix:
   allow_failures:
     - os: windows
       env: CARGO_INCREMENTAL=0 OS_WINDOWS=true
+    - env: INTEGRATION=rust-lang-nursery/stdsimd
 
 before_script:
   - |
diff --git a/ci/integration-tests.sh b/ci/integration-tests.sh
index 6d33224a59b..f6540769bb0 100755
--- a/ci/integration-tests.sh
+++ b/ci/integration-tests.sh
@@ -18,6 +18,8 @@ cd checkout
 
 # run clippy on a project, try to be verbose and trigger as many warnings
 # as possible for greater coverage
+# NOTE: we use `tee` to print any warnings and errors to stdout
+#       to avoid build timeout in Travis
 RUST_BACKTRACE=full \
 cargo clippy \
     --all-targets \
@@ -26,10 +28,9 @@ cargo clippy \
     --cap-lints warn \
     -W clippy::pedantic \
     -W clippy::nursery \
-    > clippy_output 2>&1 || true
+    2>&1 | tee clippy_output
 
 cargo uninstall clippy
-cat clippy_output
 
 if grep -q "internal compiler error\|query stack during panic\|E0463" clippy_output; then
     exit 1
diff --git a/clippy_lints/src/mutex_atomic.rs b/clippy_lints/src/mutex_atomic.rs
index d4194b0ca41..a8c953b880b 100644
--- a/clippy_lints/src/mutex_atomic.rs
+++ b/clippy_lints/src/mutex_atomic.rs
@@ -77,7 +77,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Mutex {
     }
 }
 
-fn get_atomic_name(ty: Ty<'_>) -> Option<(&'static str)> {
+fn get_atomic_name(ty: Ty<'_>) -> Option<&'static str> {
     match ty.kind {
         ty::Bool => Some("AtomicBool"),
         ty::Uint(_) => Some("AtomicUsize"),
diff --git a/clippy_lints/src/utils/mod.rs b/clippy_lints/src/utils/mod.rs
index 5729c8c2967..bf46a9a8b82 100644
--- a/clippy_lints/src/utils/mod.rs
+++ b/clippy_lints/src/utils/mod.rs
@@ -239,7 +239,7 @@ pub fn match_path_ast(path: &ast::Path, segments: &[&str]) -> bool {
 }
 
 /// Gets the definition associated to a path.
-pub fn path_to_res(cx: &LateContext<'_, '_>, path: &[&str]) -> Option<(def::Res)> {
+pub fn path_to_res(cx: &LateContext<'_, '_>, path: &[&str]) -> Option<def::Res> {
     let crates = cx.tcx.crates();
     let krate = crates
         .iter()