From e008f6e53cefb027980b88ba8d3962a14b7fa1e2 Mon Sep 17 00:00:00 2001 From: Xu Cheng Date: Sun, 17 Jul 2016 17:47:05 +0800 Subject: remove unnecessary build options manipulation for test and postinstall We already loaded build options in ARGV.resolved_formulae for test and postinstall --- Library/Homebrew/formula.rb | 10 ---------- Library/Homebrew/postinstall.rb | 2 +- 2 files changed, 1 insertion(+), 11 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index 5e8232d97..23f948196 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -836,14 +836,6 @@ class Formula method(:post_install).owner == self.class end - # @private - def run_post_install - build, self.build = self.build, Tab.for_formula(self) - post_install - ensure - self.build = build - end - # Tell the user about any caveats regarding this package. # @return [String] #
def caveats
@@ -1346,7 +1338,6 @@ class Formula
old_home = ENV["HOME"]
old_curl_home = ENV["CURL_HOME"]
ENV["CURL_HOME"] = old_curl_home || old_home
- build, self.build = self.build, Tab.for_formula(self)
mktemp("#{name}-test") do |staging|
staging.retain! if ARGV.keep_tmp?
@testpath = staging.tmpdir
@@ -1361,7 +1352,6 @@ class Formula
end
ensure
@testpath = nil
- self.build = build
ENV["HOME"] = old_home
ENV["CURL_HOME"] = old_curl_home
end
diff --git a/Library/Homebrew/postinstall.rb b/Library/Homebrew/postinstall.rb
index 0b6d8f6b0..c5c7ace31 100644
--- a/Library/Homebrew/postinstall.rb
+++ b/Library/Homebrew/postinstall.rb
@@ -13,7 +13,7 @@ begin
formula = ARGV.resolved_formulae.first
formula.extend(Debrew::Formula) if ARGV.debug?
- formula.run_post_install
+ formula.post_install
rescue Exception => e
Marshal.dump(e, error_pipe)
error_pipe.close
--
cgit v1.2.3