Use bundle install in more places.

This provides a more consistent version for `rubocop` than relying on
`Homebrew.install_gem_setup_path!` (and we really want `brew style` to
provide consistent output).
This commit is contained in:
Mike McQuaid 2019-01-08 15:08:21 +00:00
parent 03afa5a75f
commit efc3c0c4c0
No known key found for this signature in database
GPG Key ID: 48A898132FD8EE70
7 changed files with 29 additions and 17 deletions

View File

@ -20,7 +20,7 @@ module Cask
def install_rubocop
capture_stderr do
begin
Homebrew.install_gem_setup_path! "rubocop"
Homebrew.install_bundler_gems!
rescue SystemExit
raise CaskError, Tty.strip_ansi($stderr.string).chomp.sub(/\AError: /, "")
end

View File

@ -73,7 +73,7 @@ module Homebrew
end
def regenerate_man_pages
Homebrew.install_gem_setup_path! "ronn"
Homebrew.install_bundler_gems!
markup = build_man_page
convert_man_page(markup, TARGET_DOC_PATH/"Manpage.md")

View File

@ -76,8 +76,6 @@ module Homebrew
FileUtils.rm_f "test/coverage/.resultset.json"
end
ENV["BUNDLE_GEMFILE"] = "#{HOMEBREW_LIBRARY_PATH}/test/Gemfile"
# Override author/committer as global settings might be invalid and thus
# will cause silent failure during the setup of dummy Git repositories.
%w[AUTHOR COMMITTER].each do |role|
@ -86,8 +84,7 @@ module Homebrew
ENV["GIT_#{role}_DATE"] = "Sun Jan 22 19:59:13 2017 +0000"
end
Homebrew.install_gem_setup_path! "bundler", "<2"
system "bundle", "install" unless quiet_system("bundle", "check")
Homebrew.install_bundler_gems!
parallel = true

View File

@ -17,7 +17,7 @@ module Homebrew
switch :debug
end.parse
Homebrew.install_gem_setup_path! "bundler", "<2"
Homebrew.install_bundler!
ohai "cd #{HOMEBREW_LIBRARY_PATH}/vendor"
(HOMEBREW_LIBRARY_PATH/"vendor").cd do

View File

@ -17,8 +17,7 @@ module Homebrew
def check_style_impl(files, output_type, options = {})
fix = options[:fix]
Homebrew.install_gem_setup_path! "rubocop"
Homebrew.install_gem! "rubocop-rspec"
Homebrew.install_bundler_gems!
require "rubocop"
require "rubocops"

View File

@ -1,4 +1,6 @@
---
BUNDLE_BIN: "../bin"
BUNDLE_PATH: "../vendor/bundle"
BUNDLE_DISABLE_SHARED_GEMS: "true"
BUNDLE_BIN: "../bin"
BUNDLE_JOBS: "4"
BUNDLE_RETRY: "3"

View File

@ -203,7 +203,7 @@ module Homebrew
_system(cmd, *args, **options)
end
def install_gem!(name, version = nil)
def setup_gem_environment!
# Match where our bundler gems are.
ENV["GEM_HOME"] = "#{ENV["HOMEBREW_LIBRARY"]}/Homebrew/vendor/bundle/ruby/#{RbConfig::CONFIG["ruby_version"]}"
ENV["GEM_PATH"] = ENV["GEM_HOME"]
@ -217,12 +217,15 @@ module Homebrew
path.prepend(RUBY_BIN) if which("ruby") != RUBY_PATH
path.prepend(Gem.bindir)
ENV["PATH"] = path
end
return unless Gem::Specification.find_all_by_name(name, version).empty?
def install_gem!(name)
setup_gem_environment!
return unless Gem::Specification.find_all_by_name(name).empty?
ohai "Installing or updating '#{name}' gem"
install_args = %W[--no-ri --no-rdoc #{name}]
install_args << "--version" << version if version
# Do `gem install [...]` without having to spawn a separate process or
# having to find the right `gem` binary for the running Ruby interpreter.
@ -238,17 +241,28 @@ module Homebrew
odie "Failed to install/update the '#{name}' gem." if exit_code.nonzero?
end
def install_gem_setup_path!(name, version = nil, executable = name)
install_gem!(name, version)
def install_gem_setup_path!(name)
install_gem!(name)
return if which(executable)
return if which(name)
odie <<~EOS
The '#{name}' gem is installed but couldn't find '#{executable}' in the PATH:
The '#{name}' gem is installed but couldn't find '#{name}' in the PATH:
#{ENV["PATH"]}
EOS
end
def install_bundler!
install_gem_setup_path! "bundler"
end
def install_bundler_gems!
install_bundler!
ENV["BUNDLE_GEMFILE"] = "#{HOMEBREW_LIBRARY_PATH}/test/Gemfile"
system "bundle", "install" unless quiet_system("bundle", "check")
setup_gem_environment!
end
# rubocop:disable Style/GlobalVars
def inject_dump_stats!(the_module, pattern)
@injected_dump_stat_modules ||= {}