Skip to content
Snippets Groups Projects
Unverified Commit dc9fe597 authored by Mike McQuaid's avatar Mike McQuaid Committed by GitHub
Browse files

Merge pull request #6580 from middagj/feature/fix-frozen-string

Fix frozen string bug
parents a1a0569f 0e3ab6e6
No related branches found
No related tags found
No related merge requests found
......@@ -466,15 +466,17 @@ class BuildFlagsError < RuntimeError
require_text = "requires"
end
bottle_text = if bottled
<<~EOS
Alternatively, remove the #{flag_text} to attempt bottle installation.
EOS
end
message = <<~EOS
The following #{flag_text}:
#{flags.join(", ")}
#{require_text} building tools, but none are installed.
#{DevelopmentTools.installation_instructions}
EOS
message << <<~EOS if bottled
Alternatively, remove the #{flag_text} to attempt bottle installation.
#{DevelopmentTools.installation_instructions}#{bottle_text}
EOS
super message
......
......@@ -225,3 +225,9 @@ describe BottleFormulaUnavailableError do
its(:to_s) { is_expected.to match(/This bottle does not contain the formula file/) }
end
describe BuildFlagsError do
subject { described_class.new(["-s"]) }
its(:to_s) { is_expected.to match(/flag:\s+-s\nrequires building tools/) }
end
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment