lib/packer/runner.rb in packer-config-1.6.0 vs lib/packer/runner.rb in packer-config-1.6.1

@@ -15,16 +15,16 @@
         # Run without streaming std* to any screen
         stdout, stderr, status = Open3.capture3(cmd)
       else
         # Run but stream as well as capture stdout to the screen
         # see: http://stackoverflow.com/a/1162850/83386
-        Open3.popen3(cmd) do |std_in, std_out, std_err, thread|
+        Open3.popen3(cmd) do |_std_in, std_out, std_err, thread|
           # read each stream from a new thread
           Thread.new do
             until (raw = std_out.getc).nil? do
               stdout << raw
-              $stdout.write "#{raw}"
+              $stdout.write raw.to_s
             end
           end
           Thread.new do
             until (raw_line = std_err.gets).nil? do
               stderr << raw_line
@@ -33,9 +33,10 @@
 
           thread.join # don't exit until the external process is done
           status = thread.value
         end
       end
+      # rubocop:disable Style/NumericPredicate
       raise CommandExecutionError.new(stderr) unless status == 0
       stdout
     end
 
     def self.exec!(*args)