relevance-grit 0.8.1 → 0.8.3

data/History.txt

@@ -1,3 +1,9 @@
+== 0.8.3 / 2008-07-07
+  * Capture stderr and log if debug is true (rsanheim)
+  
+== 0.8.2 / 2008-06-27
+  * Allow user provided logger (rsanheim)
+  
 == 0.8.0 / 2008-04-24
   * Lots of fixes and additions
 

data/grit.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = "grit"
-  s.version = "0.8.1"
+  s.version = "0.8.3"
   s.date = "2008-04-24"
   s.summary = "Object model interface to a git repo"
   s.email = "tom@rubyisawesome.com"

data/lib/grit/git.rb

@@ -56,16 +56,18 @@ module Grit
       
       call = "#{prefix}#{Git.git_binary} --git-dir='#{self.git_dir}' #{cmd.to_s.gsub(/_/, '-')} #{(opt_args + ext_args).join(' ')}#{postfix}"
       Grit.log(call) if Grit.debug
-      response = timeout ? sh(call) : wild_sh(call)
+      response, err = timeout ? sh(call) : wild_sh(call)
       Grit.log(response) if Grit.debug
+      Grit.log(err) if Grit.debug
       response
     end
 
     def sh(command)
-      ret, pid = nil, nil
-      Open4.popen4(command) do |id, _, io, _|
+      ret, pid, err = nil, nil, nil
+      Open4.popen4(command) do |id, _, stdout, stderr|
         pid = id
-        ret = Timeout.timeout(self.class.git_timeout) { io.read }
+        ret = Timeout.timeout(self.class.git_timeout) { stdout.read }
+        err = stderr.read
         @bytes_read += ret.size
 
         if @bytes_read > 5242880 # 5.megabytes
@@ -74,9 +76,9 @@ module Grit
           raise GitTimeout.new(command, bytes) 
         end
       end
-      ret
+      [ret, err]
     rescue Errno::ECHILD
-      ret
+      [ret, err]
     rescue Object => e
       Process.kill('KILL', pid) rescue nil
       bytes = @bytes_read
@@ -85,12 +87,14 @@ module Grit
     end
 
     def wild_sh(command)
-      ret = nil
-      Open4.popen4(command) {|pid, _, io, _|
-        ret = io.read
+      ret, err = nil, nil
+      Open4.popen4(command) {|pid, _, stdout, stderr|
+        ret = stdout.read
+        err = stderr.read
       }
+      [ret, err]
     rescue Errno::ECHILD
-      ret
+      [ret, err]
     end
 
     # Transform Ruby style options into git command line options

data/lib/grit.rb

@@ -28,8 +28,11 @@ require 'grit/repo'
 require 'grit/index'
 
 module Grit
+  
   class << self
+    # Set +debug+ to true to log all git calls and responses
     attr_accessor :debug
+    # The standard +logger+ for debugging git calls - this defaults to a plain STDOUT logger
     attr_accessor :logger
     def log(str)
       logger.debug { str }

data/test/test_git.rb

@@ -5,10 +5,28 @@ class TestGit < Test::Unit::TestCase
     @git = Git.new(File.join(File.dirname(__FILE__), *%w[..]))
   end
   
+  def teardown
+    Grit.debug = false
+  end
+  
   def test_method_missing
     assert_match(/^git version [\w\.]*$/, @git.version)
   end
   
+  def test_logs_stderr
+    Grit.debug = true
+    Grit.stubs(:log)
+    Grit.expects(:log).with(includes("git: 'bad' is not a git-command"))
+    @git.bad
+  end
+  
+  def testl_logs_stderr_when_skipping_timeout 
+    Grit.debug = true
+    Grit.stubs(:log)
+    Grit.expects(:log).with(includes("git: 'bad' is not a git-command"))
+    @git.bad :timeout => false
+  end
+  
   def test_transform_options
     assert_equal ["-s"], @git.transform_options({:s => true})
     assert_equal ["-s '5'"], @git.transform_options({:s => 5})
@@ -46,7 +64,7 @@ class TestGit < Test::Unit::TestCase
 
   def test_works_fine_if_quick
     output = 'output'
-    Open4.expects(:popen4).yields( nil, nil, mock(:read => output), nil )
+    Open4.expects(:popen4).yields( nil, nil, mock(:read => output), stub(:read => nil) )
     assert_equal output, @git.something
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: relevance-grit
 version: !ruby/object:Gem::Version 
-  version: 0.8.1
+  version: 0.8.3
 platform: ruby
 authors: 
 - Tom Preston-Werner