admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #20654 from Homebrew/system_command_parameter

Browse Source 
system_command: add missing `must_succeed`
This commit is contained in:
Bo Anderson 2025-09-09 05:06:42 +00:00 committed by GitHub
commit 80f9e6714f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Library/Homebrew/system_command.rb
View File

@ -112,6 +112,7 @@ class SystemCommand
sudo_as_root: T::Boolean, sudo_as_root: T::Boolean,
env: T::Hash[String, String], env: T::Hash[String, String],
input: T.any(String, T::Array[String]), input: T.any(String, T::Array[String]),
must_succeed: T::Boolean,
print_stdout: T.any(T::Boolean, Symbol), print_stdout: T.any(T::Boolean, Symbol),
print_stderr: T.any(T::Boolean, Symbol), print_stderr: T.any(T::Boolean, Symbol),
debug: T.nilable(T::Boolean), debug: T.nilable(T::Boolean),
@ -122,9 +123,10 @@ class SystemCommand
timeout: T.nilable(T.any(Integer, Float)), timeout: T.nilable(T.any(Integer, Float)),
).returns(SystemCommand::Result) ).returns(SystemCommand::Result)
} }
def self.run!(executable, args: [], sudo: false, sudo_as_root: false, env: {}, input: [], print_stdout: false, def self.run!(executable, args: [], sudo: false, sudo_as_root: false, env: {}, input: [], must_succeed: true,
print_stderr: true, debug: nil, verbose: nil, secrets: [], chdir: nil, reset_uid: false, timeout: nil) print_stdout: false, print_stderr: true, debug: nil, verbose: nil, secrets: [], chdir: nil,
run(executable, args:, sudo:, sudo_as_root:, env:, input:, must_succeed: true, print_stdout:, print_stderr:, reset_uid: false, timeout: nil)
run(executable, args:, sudo:, sudo_as_root:, env:, input:, must_succeed:, print_stdout:, print_stderr:,
debug:, verbose:, secrets:, chdir:, reset_uid:, timeout:) debug:, verbose:, secrets:, chdir:, reset_uid:, timeout:)
end end