Skip to content
Snippets Groups Projects
Unverified Commit e008f6e5 authored by Xu Cheng's avatar Xu Cheng
Browse files

remove unnecessary build options manipulation for test and postinstall

We already loaded build options in ARGV.resolved_formulae for test and
postinstall
parent 91a47a9e
No related branches found
No related tags found
No related merge requests found
......@@ -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]
# <pre>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
......
......@@ -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
......
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