Merge pull request #7090 from EricFromCanada/bump-url-escaping

bump-formula-pr: add missing URL escapes
This commit is contained in:
Mike McQuaid 2020-03-02 13:02:52 +00:00 committed by GitHub
commit 557cc1523f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -282,12 +282,12 @@ module Homebrew
] ]
elsif new_mirrors elsif new_mirrors
replacement_pairs << [ replacement_pairs << [
/^( +)(mirror \"#{new_mirrors.last}\"\n)/m, /^( +)(mirror \"#{Regexp.escape(new_mirrors.last)}\"\n)/m,
"\\1\\2\\1version \"#{forced_version}\"\n", "\\1\\2\\1version \"#{forced_version}\"\n",
] ]
else else
replacement_pairs << [ replacement_pairs << [
/^( +)(url \"#{new_url}\"\n)/m, /^( +)(url \"#{Regexp.escape(new_url)}\"\n)/m,
"\\1\\2\\1version \"#{forced_version}\"\n", "\\1\\2\\1version \"#{forced_version}\"\n",
] ]
end end