From 4a7c68763edb1122d9f498d7ce9dd4c34a4c9956 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Tue, 4 Jun 2013 20:34:34 +0100 Subject: [PATCH] brew-test: add five minute timeout. --- Library/Homebrew/cmd/test.rb | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Library/Homebrew/cmd/test.rb b/Library/Homebrew/cmd/test.rb index 3550727910..24644f41f9 100644 --- a/Library/Homebrew/cmd/test.rb +++ b/Library/Homebrew/cmd/test.rb @@ -1,8 +1,11 @@ require 'extend/ENV' require 'hardware' require 'keg' +require 'timeout' module Homebrew extend self + TEST_TIMEOUT_SECONDS = 5*60 + def test raise FormulaUnspecifiedError if ARGV.named.empty? @@ -25,8 +28,10 @@ module Homebrew extend self puts "Testing #{f.name}" begin # tests can also return false to indicate failure - raise if f.test == false - rescue + Timeout::timeout TEST_TIMEOUT_SECONDS do + raise if f.test == false + end + rescue Exception ofail "#{f.name}: failed" end end