Fix custom hook output and return values

Author: Drew Blessing

lib/gitlab_custom_hook.rb

@@ -22,7 +22,8 @@ def post_receive(changes, repo_path)
   def update(ref_name, old_value, new_value, repo_path)
     hook = hook_file('update', repo_path)
     return true if hook.nil?
-    system(hook, ref_name, old_value, new_value) ? true : false
+    system(hook, ref_name, old_value, new_value)
+    $?.exitstatus == 0 ? true : false
   end
 
   private
@@ -51,13 +52,12 @@ def call_receive_hook(hook, changes)
       # need to close stdin before reading stdout
       stdin.close
 
-      # only output stdut_stderr if scripts doesn't return 0
       unless wait_thr.value == 0
         exit_status = false
-        stdout_stderr.each_line { |line| puts line }
       end
     end
 
+    stdout_stderr.each_line { |line| puts line }
     exit_status
   end