brew-test-bot: email subjects contain OS version.
This commit is contained in:
parent
1a21754e5c
commit
d87ce4b023
@ -17,7 +17,7 @@ require 'utils'
|
|||||||
require 'date'
|
require 'date'
|
||||||
require 'erb'
|
require 'erb'
|
||||||
|
|
||||||
EMAIL_SUBJECT_FILE = "brew-test-bot.email.txt"
|
EMAIL_SUBJECT_FILE = "brew-test-bot.#{MacOS.cat}.email.txt"
|
||||||
HOMEBREW_CONTRIBUTED_CMDS = HOMEBREW_REPOSITORY + "Library/Contributions/cmd/"
|
HOMEBREW_CONTRIBUTED_CMDS = HOMEBREW_REPOSITORY + "Library/Contributions/cmd/"
|
||||||
|
|
||||||
class Step
|
class Step
|
||||||
@ -381,7 +381,7 @@ if ARGV.include? "--email"
|
|||||||
File.open EMAIL_SUBJECT_FILE, 'w' do |file|
|
File.open EMAIL_SUBJECT_FILE, 'w' do |file|
|
||||||
# The file should be written at the end but in case we don't get to that
|
# The file should be written at the end but in case we don't get to that
|
||||||
# point ensure that we have something valid.
|
# point ensure that we have something valid.
|
||||||
file.write "INTERNAL ERROR"
|
file.write "#{MacOS.version}: internal error."
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -419,9 +419,9 @@ if ARGV.include? "--email"
|
|||||||
end
|
end
|
||||||
|
|
||||||
if failed_steps.empty?
|
if failed_steps.empty?
|
||||||
email_subject = 'PASSED'
|
email_subject = ''
|
||||||
else
|
else
|
||||||
email_subject = "#{failed_steps.join ', '}"
|
email_subject = "#{MacOS.version}: #{failed_steps.join ', '}."
|
||||||
end
|
end
|
||||||
|
|
||||||
File.open EMAIL_SUBJECT_FILE, 'w' do |file|
|
File.open EMAIL_SUBJECT_FILE, 'w' do |file|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user