diff options
author | Max Lincoln <max@devopsy.com> | 2014-07-17 13:00:39 -0400 |
---|---|---|
committer | Max Lincoln <max@devopsy.com> | 2014-07-17 13:12:42 -0400 |
commit | ef704af76fa6d2512782824c7c3ba52a9deac3ab (patch) | |
tree | 6dcd987d970cf6521ba64bbc6ced950cd6966163 /lib/mixlib/shellout.rb | |
parent | 5ce6ead83be1a46241db53e401f44c9a40f598ce (diff) | |
download | mixlib-shellout-ef704af76fa6d2512782824c7c3ba52a9deac3ab.tar.gz |
Simplify live_stream vs live_stdout vs live_stderr based on PR comments
Diffstat (limited to 'lib/mixlib/shellout.rb')
-rw-r--r-- | lib/mixlib/shellout.rb | 45 |
1 files changed, 22 insertions, 23 deletions
diff --git a/lib/mixlib/shellout.rb b/lib/mixlib/shellout.rb index a633819..6261c21 100644 --- a/lib/mixlib/shellout.rb +++ b/lib/mixlib/shellout.rb @@ -53,13 +53,13 @@ module Mixlib # to determine if the command was successful. Normally set via options to new attr_accessor :valid_exit_codes - # When live_stream is set, stdout and stderr of the subprocess will be - # copied to it as the subprocess is running. The stderr will also be copied, - # unless live_stderr_stream is set to nil or a different object. For example, - # if live_stream is set to STDOUT, the command's output will be echoed to STDOUT. - attr_accessor :live_stream + # When live_stdout is set, the stdout of the subprocess will be copied to it + # as the subprocess is running. + attr_accessor :live_stdout - attr_writer :live_stderr_stream + # When live_stderr is set, the stderr of the subprocess will be copied to it + # as the subprocess is running. + attr_accessor :live_stderr # ShellOut will push data from :input down the stdin of the subprocss. # Normally set via options passed to new. @@ -150,7 +150,7 @@ module Mixlib # cmd.run_command # etc. def initialize(*command_args) @stdout, @stderr = '', '' - @live_stream = nil + @live_stdout = @live_stderr = nil @input = nil @log_level = :debug @log_tag = nil @@ -166,18 +166,16 @@ module Mixlib @command = command_args.size == 1 ? command_args.first : command_args end - # When live_stderr_stream is set, the stderr of the subprocess will be - # copied to it as the subprocess is running. For example, if live_stderr_stream is - # set to STDERR, the command's output will be echoed to STDERR. The default - # value is to match live_stream, so setting live_stream to STDOUT will also - # set live_stderr_stream to STDOUT. If you only want the stdout of the subprocess - # copied, then you should explicitly set live_stderr_stream to nil. - def live_stderr_stream - # We can't use ||= because it would override an explicit nil - unless defined?(@live_stderr_stream) - @live_stderr_stream = live_stream - end - @live_stderr_stream + # Returns the stream that both is being used by both live_stdout and live_stderr, or nil + def live_stream + live_stdout == live_stderr ? live_stdout : nil + end + + # A shortcut for setting both live_stdout and live_stderr, so that both the + # stdout and stderr from the subprocess will be copied to the same stream as + # the subprocess is running. + def live_stream=(stream) + @live_stdout = @live_stderr = stream end # Set the umask that the subprocess will have. If given as a string, it @@ -303,10 +301,11 @@ module Mixlib when 'returns' self.valid_exit_codes = Array(setting) when 'live_stream' - self.live_stream = setting - when 'live_stderr_stream' - puts "Setting to #{setting}" - self.live_stderr_stream = setting + self.live_stdout = self.live_stderr = setting + when 'live_stdout' + self.live_stdout = setting + when 'live_stderr' + self.live_stderr = setting when 'input' self.input = setting when 'logger' |