Merge pull request #18134 from Homebrew/env-not-superenv

This commit is contained in:
Mike McQuaid 2024-08-23 08:05:54 +01:00 committed by GitHub
commit 32bba8ace1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
8 changed files with 18 additions and 28 deletions

View File

@ -75,9 +75,10 @@ class Build
ENV.activate_extensions!(env: args.env) ENV.activate_extensions!(env: args.env)
if superenv?(args.env) if superenv?(args.env)
ENV.keg_only_deps = keg_only_deps superenv = T.cast(ENV, Superenv)
ENV.deps = formula_deps superenv.keg_only_deps = keg_only_deps
ENV.run_time_deps = run_time_deps superenv.deps = formula_deps
superenv.run_time_deps = run_time_deps
ENV.setup_build_environment( ENV.setup_build_environment(
formula:, formula:,
cc: args.cc, cc: args.cc,

View File

@ -31,7 +31,7 @@ module Homebrew
sig { override.void } sig { override.void }
def run def run
ENV.activate_extensions! ENV.activate_extensions!
ENV.deps = args.named.to_formulae if superenv?(nil) T.cast(ENV, Superenv).deps = args.named.to_formulae if superenv?(nil)
ENV.setup_build_environment ENV.setup_build_environment
shell = if args.plain? shell = if args.plain?

View File

@ -28,7 +28,9 @@ module Homebrew
def run def run
ENV.activate_extensions!(env: args.env) ENV.activate_extensions!(env: args.env)
ENV.deps = Formula.installed.select { |f| f.keg_only? && f.opt_prefix.directory? } if superenv?(args.env) if superenv?(args.env)
T.cast(ENV, Superenv).deps = Formula.installed.select { |f| f.keg_only? && f.opt_prefix.directory? }
end
ENV.setup_build_environment ENV.setup_build_environment
if superenv?(args.env) if superenv?(args.env)
# superenv stopped adding brew's bin but generally users will want it # superenv stopped adding brew's bin but generally users will want it

View File

@ -1,7 +1,7 @@
# typed: strict # typed: strict
module EnvActivation module EnvActivation
include Superenv include SharedEnvExtension
end end
# @!visibility private # @!visibility private

View File

@ -51,8 +51,8 @@ module SharedEnvExtension
@debug_symbols = debug_symbols @debug_symbols = debug_symbols
reset reset
end end
private :setup_build_environment
alias generic_shared_setup_build_environment setup_build_environment alias generic_shared_setup_build_environment setup_build_environment
private :generic_shared_setup_build_environment
sig { void } sig { void }
def reset def reset
@ -316,6 +316,14 @@ module SharedEnvExtension
sig { void } sig { void }
def permit_arch_flags; end def permit_arch_flags; end
sig { returns(Integer) }
def make_jobs
Homebrew::EnvConfig.make_jobs.to_i
end
sig { void }
def refurbish_args; end
private private
sig { params(_flags: T::Array[String], _map: T::Hash[Symbol, String]).void } sig { params(_flags: T::Array[String], _map: T::Hash[Symbol, String]).void }

View File

@ -12,14 +12,3 @@ module SharedEnvExtension
} }
def []=(key, value); end def []=(key, value); end
end end
# @!visibility private
class Sorbet
module Private
module Static
class ENVClass
include SharedEnvExtension
end
end
end
end

View File

@ -149,15 +149,6 @@ module Stdenv
append "CXX", "-stdlib=libc++" if compiler == :clang append "CXX", "-stdlib=libc++" if compiler == :clang
end end
sig { returns(Integer) }
def make_jobs
Homebrew::EnvConfig.make_jobs.to_i
end
# This method does nothing in {Stdenv} since there is no argument refurbishment.
sig { void }
def refurbish_args; end
private private
sig { params(before: Regexp, after: String).void } sig { params(before: Regexp, after: String).void }

View File

@ -21,7 +21,6 @@ module SharedEnvExtension
# Normalise the system Perl version used, where multiple may be available # Normalise the system Perl version used, where multiple may be available
self["VERSIONER_PERL_VERSION"] = MacOS.preferred_perl_version self["VERSIONER_PERL_VERSION"] = MacOS.preferred_perl_version
end end
private :setup_build_environment
sig { returns(T::Boolean) } sig { returns(T::Boolean) }
def no_weak_imports_support? def no_weak_imports_support?