admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #7368 from MikeMcQuaid/test-ensure-kill

Browse Source 
test: ensure pkill is run even without exception.
This commit is contained in:
Mike McQuaid 2020-04-16 08:22:04 +01:00 committed by GitHub
commit 957b404ee7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Library/Homebrew/test.rb
View File

@ -37,6 +37,7 @@ begin
rescue Exception => e # rubocop:disable Lint/RescueException rescue Exception => e # rubocop:disable Lint/RescueException
error_pipe.puts e.to_json error_pipe.puts e.to_json
error_pipe.close error_pipe.close
ensure
pid = Process.pid.to_s pid = Process.pid.to_s
if which("pgrep") && which("pkill") && system("pgrep", "-P", pid, out: :close) if which("pgrep") && which("pkill") && system("pgrep", "-P", pid, out: :close)
$stderr.puts "Killing child processes..." $stderr.puts "Killing child processes..."
@ -44,5 +45,5 @@ rescue Exception => e # rubocop:disable Lint/RescueException
sleep 1 sleep 1
system "pkill", "-9", "-P", pid system "pkill", "-9", "-P", pid
end end
exit! 1 exit! 1 if e
end end