overcommit 0.2.2 → 0.2.3

data/bin/hooks/prepare-commit-msg

@@ -75,7 +75,8 @@ insert_submodule_changes() {
     $done = false
 
     def insert
-      puts "#{ARGV.first}\n\n" unless $done
+      ref = ARGV.first
+      puts `git submodule summary --cached #{ref}` unless $done
       $done = true
     end
 
@@ -96,7 +97,7 @@ insert_submodule_changes() {
     ensure
       insert # just in case no lines matched
     end
-  ' "$SUBMODULE_CHANGES" > $TMP_FILE && cp $TMP_FILE "$FILE"
+  ' "$OTHER" > $TMP_FILE && cp $TMP_FILE "$FILE"
 }
 
 strip_submodule_changes() {
@@ -152,8 +153,6 @@ case "$SOURCE" in
     # do nothing else if there are no submodule changes staged
     git diff-index --cached $OTHER | grep -q -e '^:160000' || exit 0
 
-    SUBMODULE_CHANGES=$(git submodule summary --cached $OTHER)
-
     strip_submodule_changes
     insert_submodule_changes
     ;;

data/lib/overcommit/hook_specific_check.rb

@@ -70,7 +70,14 @@ module Overcommit
       end
 
       def in_path?(cmd)
-        system("which #{cmd} &> /dev/null")
+        exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
+        ENV['PATH'].split(File::PATH_SEPARATOR).each do |path|
+          exts.each do |ext|
+            exe = File.join(path, "#{cmd}#{ext}")
+            return true if File.executable? exe
+          end
+        end
+        false
       end
 
       def commit_message_file

data/lib/overcommit/plugins/pre_commit/coffee_lint.rb

@@ -8,7 +8,8 @@ module Overcommit::GitHook
         return :warn, 'Run `npm install -g coffeelint`'
       end
 
-      output = `coffeelint --quiet #{(staged.join(' '))}`.split("\n")
+      paths  = staged.collect(&:path).join(' ')
+      output = `coffeelint --quiet #{paths}`.split("\n")
       return ($?.success? ? :good : :bad), output
     end
   end

data/lib/overcommit/staged_file.rb

@@ -9,9 +9,8 @@ module Overcommit
     attr_reader :contents
 
     def initialize(path)
-      @original_path  = path
-      @tempfile       = Tempfile.new([path.gsub('/', '_'), File.extname(path)])
-      self.contents   = `git show :#{@original_path}`
+      @original_path = path
+      @contents      = `git show :#{@original_path}`
     end
 
     # Given error output from a syntax checker, replace references to the
@@ -23,7 +22,7 @@ module Overcommit
     # The path of the temporary file on disk, suitable for feeding in to a
     # syntax checker.
     def path
-      @tempfile.path
+      tempfile.path
     end
 
     # Set or overwrite the temporary file's contents.
@@ -32,9 +31,7 @@ module Overcommit
     # the template before checking.
     def contents=(contents)
       @contents = contents
-      @tempfile.seek 0
-      @tempfile.write @contents
-      @tempfile.flush
+      write_tempfile
     end
 
     # Returns the set of line numbers corresponding to the lines that were
@@ -69,5 +66,21 @@ module Overcommit
 
       lines
     end
+
+    def tempfile
+      unless @tempfile
+        basename = [@original_path.gsub('/', '_'), File.extname(@original_path)]
+        @tempfile = Tempfile.new(basename)
+        write_tempfile
+      end
+      @tempfile
+    end
+
+    def write_tempfile
+      tempfile.open if tempfile.closed?
+      tempfile.truncate 0
+      tempfile.write @contents
+      tempfile.close
+    end
   end
 end

data/lib/overcommit/version.rb

@@ -1,3 +1,3 @@
 module Overcommit
-  VERSION = '0.2.2'
+  VERSION = '0.2.3'
 end

metadata

@@ -1,112 +1,129 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: overcommit
-version: !ruby/object:Gem::Version
-  version: 0.2.2
-  prerelease: 
+version: !ruby/object:Gem::Version 
+  hash: 17
+  prerelease: false
+  segments: 
+  - 0
+  - 2
+  - 3
+  version: 0.2.3
 platform: ruby
-authors:
+authors: 
 - Causes Engineering
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-25 00:00:00.000000000 Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2013-08-21 00:00:00 -07:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id001
 description: Overcommit is a utility to install and extend Git hooks
 email: eng@causes.com
-executables:
+executables: 
 - overcommit
 extensions: []
+
 extra_rdoc_files: []
-files:
-- lib/overcommit.rb
-- lib/overcommit/version.rb
-- lib/overcommit/utils.rb
-- lib/overcommit/plugins/commit_msg/trailing_period.rb
-- lib/overcommit/plugins/commit_msg/change_id.rb
+
+files: 
+- lib/overcommit/logger.rb
+- lib/overcommit/installer.rb
+- lib/overcommit/hook_specific_check.rb
+- lib/overcommit/reporter.rb
+- lib/overcommit/staged_file.rb
+- lib/overcommit/plugins/commit_msg/hard_tabs.rb
+- lib/overcommit/plugins/commit_msg/russian_novel.rb
 - lib/overcommit/plugins/commit_msg/release_note.rb
 - lib/overcommit/plugins/commit_msg/single_line_subject.rb
-- lib/overcommit/plugins/commit_msg/russian_novel.rb
-- lib/overcommit/plugins/commit_msg/hard_tabs.rb
+- lib/overcommit/plugins/commit_msg/change_id.rb
 - lib/overcommit/plugins/commit_msg/text_width.rb
-- lib/overcommit/plugins/pre_commit/erb_syntax.rb
-- lib/overcommit/plugins/pre_commit/js_console_log.rb
-- lib/overcommit/plugins/pre_commit/ruby_syntax.rb
+- lib/overcommit/plugins/commit_msg/trailing_period.rb
+- lib/overcommit/plugins/pre_commit/ruby_style.rb
 - lib/overcommit/plugins/pre_commit/js_syntax.rb
+- lib/overcommit/plugins/pre_commit/test_history.rb
 - lib/overcommit/plugins/pre_commit/yaml_syntax.rb
-- lib/overcommit/plugins/pre_commit/haml_syntax.rb
+- lib/overcommit/plugins/pre_commit/python_flake8.rb
 - lib/overcommit/plugins/pre_commit/causes_email.rb
+- lib/overcommit/plugins/pre_commit/js_console_log.rb
 - lib/overcommit/plugins/pre_commit/author_name.rb
-- lib/overcommit/plugins/pre_commit/whitespace.rb
-- lib/overcommit/plugins/pre_commit/python_flake8.rb
+- lib/overcommit/plugins/pre_commit/erb_syntax.rb
+- lib/overcommit/plugins/pre_commit/scss_lint.rb
 - lib/overcommit/plugins/pre_commit/coffee_lint.rb
-- lib/overcommit/plugins/pre_commit/test_history.rb
+- lib/overcommit/plugins/pre_commit/haml_syntax.rb
+- lib/overcommit/plugins/pre_commit/whitespace.rb
 - lib/overcommit/plugins/pre_commit/restricted_paths.rb
-- lib/overcommit/plugins/pre_commit/ruby_style.rb
-- lib/overcommit/plugins/pre_commit/scss_lint.rb
+- lib/overcommit/plugins/pre_commit/ruby_syntax.rb
 - lib/overcommit/plugins/pre_commit/css_linter.rb
-- lib/overcommit/staged_file.rb
-- lib/overcommit/reporter.rb
-- lib/overcommit/hook_specific_check.rb
-- lib/overcommit/installer.rb
-- lib/overcommit/errors.rb
 - lib/overcommit/git_hook.rb
+- lib/overcommit/errors.rb
+- lib/overcommit/cli.rb
 - lib/overcommit/configuration.rb
+- lib/overcommit/version.rb
 - lib/overcommit/hooks/pre_commit.rb
 - lib/overcommit/hooks/commit_msg.rb
-- lib/overcommit/logger.rb
-- lib/overcommit/cli.rb
+- lib/overcommit/utils.rb
+- lib/overcommit.rb
+- bin/overcommit
 - bin/run-hook
-- bin/scripts/index-tags
-- bin/scripts/gerrit-change-id
 - bin/scripts/jshint.js
+- bin/scripts/index-tags
 - bin/scripts/csslint-rhino.js
+- bin/scripts/gerrit-change-id
 - bin/scripts/jshint_runner.js
-- bin/overcommit
 - bin/hooks/commit-msg
-- bin/hooks/post-checkout
-- bin/hooks/post-merge
 - bin/hooks/pre-commit
+- bin/hooks/post-merge
+- bin/hooks/post-checkout
 - bin/hooks/prepare-commit-msg
 - config/templates.yml
+has_rdoc: true
 homepage: http://github.com/causes/overcommit
-licenses:
+licenses: 
 - MIT
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
+required_ruby_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
 summary: Opinionated Git hook manager
 test_files: []
+