Closes #196 fixing PATH issue with hook when ran from Git GUI Clients

Also, this fix most likely closes many other issues around the same
problem, will try to verify on various clients.

lib/lolcommits/configuration.rb

@@ -213,15 +213,18 @@ def self.can_animate?
     end
 
     # Cross-platform way of finding an executable in the $PATH.
-    # idea taken from http://bit.ly/qDaTbY
+    # idea taken from http://bit.ly/qDaTbY, if only_path is true, only the path
+    # is returned (not the path and command)
     #
     #   which('ruby') #=> /usr/bin/ruby
-    def self.command_which(cmd)
+    def self.command_which(cmd, only_path = false)
       exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
       ENV['PATH'].split(File::PATH_SEPARATOR).each do |path|
         exts.each do |ext|
           exe = "#{path}/#{cmd}#{ext}"
-          return exe if File.executable? exe
+          if File.executable? exe
+            return only_path ? path : exe
+          end
         end
       end
       nil

lib/lolcommits/installation.rb

@@ -75,8 +75,9 @@ def self.do_disable
     protected
 
     def self.hook_script(add_shebang = true)
-      ruby_path    = Lolcommits::Configuration.command_which('ruby')
-      hook_export  = "export PATH=\"#{ruby_path}:$PATH\"\n" if ruby_path
+      ruby_path    = Lolcommits::Configuration.command_which('ruby', true)
+      imagick_path = Lolcommits::Configuration.command_which('identify', true)
+      hook_export  = "export PATH=\"#{ruby_path}:#{imagick_path}:$PATH\"\n"
       capture_cmd  = 'lolcommits --capture'
       capture_args = " #{ARGV[1..-1].join(' ')}" if ARGV.length > 1
 

rubocop-todo.yml

@@ -63,10 +63,10 @@ UselessAssignment:
 Documentation:
   Enabled: false
 
-# Offence count: 2
+# Offence count: 3
 # Configuration parameters: CountComments.
 ClassLength:
-  Max: 178
+  Max: 179
 
 # Offence count: 73
 LineLength: