Support environment filtering in Casks.

Library/Homebrew/cask/lib/hbc/cli.rb

@@ -96,7 +96,7 @@ module Hbc
       if command.respond_to?(:run)
         # usual case: built-in command verb
         command.run(*args)
-      elsif require?(which("brewcask-#{command}.rb"))
+      elsif require?(which("brewcask-#{command}.rb"), ENV["HOMEBREW_PATH"])
         # external command as Ruby library on PATH, Homebrew-style
       elsif command.to_s.include?("/") && require?(command.to_s)
         # external command as Ruby library with literal path, useful

Library/Homebrew/cask/lib/hbc/container/cab.rb

@@ -10,7 +10,7 @@ module Hbc
       end
 
       def extract
-        if (cabextract = which("cabextract")).nil?
+        unless cabextract = which("cabextract", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
           raise CaskError, "Expected to find cabextract executable. Cask '#{@cask}' must add: depends_on formula: 'cabextract'"
         end
 

Library/Homebrew/cask/lib/hbc/container/generic_unar.rb

@@ -6,14 +6,14 @@ module Hbc
   class Container
     class GenericUnar < Base
       def self.me?(criteria)
-        !(lsar = which("lsar")).nil? &&
+        return false unless lsar = which("lsar", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
-          criteria.command.run(lsar,
+        criteria.command.run(lsar,
-                               args:         ["-l", "-t", "--", criteria.path],
+                             args:         ["-l", "-t", "--", criteria.path],
-                               print_stderr: false).stdout.chomp.end_with?("passed, 0 failed.")
+                             print_stderr: false).stdout.chomp.end_with?("passed, 0 failed.")
       end
 
       def extract
-        if (unar = which("unar")).nil?
+        unless unar = which("unar", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
           raise CaskError, "Expected to find unar executable. Cask #{@cask} must add: depends_on formula: 'unar'"
         end
 

Library/Homebrew/cask/lib/hbc/container/gpg.rb

@@ -24,7 +24,7 @@ module Hbc
       end
 
       def extract
-        if (gpg = which("gpg")).nil?
+        unless gpg = which("gpg", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
           raise CaskError, "Expected to find gpg executable. Cask '#{@cask}' must add: depends_on formula: 'gpg'"
         end
 

Library/Homebrew/cask/lib/hbc/container/lzma.rb

@@ -10,7 +10,7 @@ module Hbc
       end
 
       def extract
-        if (unlzma = which("unlzma")).nil?
+        unless unlzma = which("unlzma", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
           raise CaskError, "Expected to find unlzma executable. Cask '#{@cask}' must add: depends_on formula: 'lzma'"
         end
 

Library/Homebrew/cask/lib/hbc/container/xz.rb

@@ -10,7 +10,7 @@ module Hbc
       end
 
       def extract
-        if (unxz = which("unxz")).nil?
+        unless unxz = which("unxz", PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
           raise CaskError, "Expected to find unxz executable. Cask '#{@cask}' must add: depends_on formula: 'xz'"
         end
 

Library/Homebrew/cask/lib/hbc/system_command.rb

@@ -83,6 +83,10 @@ module Hbc
     def each_output_line(&b)
       executable, *args = expanded_command
 
+      unless File.exist?(executable)
+        executable = which(executable, PATH.new(ENV["PATH"], HOMEBREW_PREFIX/"bin"))
+      end
+
       raw_stdin, raw_stdout, raw_stderr, raw_wait_thr =
         Open3.popen3([executable, executable], *args, **options)
 

Library/Homebrew/compat/hbc.rb

@@ -4,7 +4,6 @@ require "compat/hbc/cache"
 require "compat/hbc/caskroom"
 require "compat/hbc/cli"
 require "compat/hbc/dsl"
-require "compat/hbc/system_command"
 
 module Hbc
   class << self

Library/Homebrew/compat/hbc/system_command.rb

@@ -1,18 +0,0 @@
-require "shellwords"
-
-module SystemCommandCompatibilityLayer
-  def initialize(executable, args: [], **options)
-    if args.empty? && !File.exist?(executable)
-      odeprecated "`system_command` with a shell string", "`system_command` with the `args` parameter"
-      executable, *args = Shellwords.shellsplit(executable)
-    end
-
-    super(executable, args: args, **options)
-  end
-end
-
-module Hbc
-  class SystemCommand
-    prepend SystemCommandCompatibilityLayer
-  end
-end