Merge branch 'master' of github.com:customink/strainer

customink · Jan 24, 2013 · 8a7d7f0 · 8a7d7f0
2 parents 6361053 + d4f703f
commit 8a7d7f0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@ v1.0.0
 - Moved entirely to Berkshelf integration
 - Moved entirely to thor
 - **Breaking** - new command `strainer`, old command is deprecated and will be removed in `2.0.0`
+- Sync output
 
 v0.2.1
 ------

diff --git a/lib/strainer/command.rb b/lib/strainer/command.rb
@@ -1,3 +1,5 @@
+require 'pty'
+
 module Strainer
   # The Command class is responsible for a command (test) against a cookbook.
   #
@@ -33,7 +35,17 @@ def run!
 
       Dir.chdir Strainer.sandbox_path do
         speak command
-        speak `#{command}`
+        PTY.spawn command do |r, _, pid|
+          begin
+            r.sync
+            r.each_line { |line| speak line }
+          rescue Errno::EIO => e
+            # Ignore this. Otherwise errors will be thrown whenever
+            # the process is closed
+          ensure
+            ::Process.wait pid
+          end
+        end
 
         unless $?.success?
           Strainer.ui.error label_with_padding + Strainer.ui.set_color('Terminated with a non-zero exit status. Strainer assumes this is a failure.', :red)