FIX: properly detect the git directory to ensure the git commands will use the right git info

Author: royriojas

hook/pre-commit.js

@@ -223,11 +223,11 @@ var spawnCmd = function ( cmd ) {
 var doExec = function ( cmd, cb ) {
   var _exec = require( 'child_process' ).exec;
   return new Promise( function ( resolve, reject ) {
-    _exec( cmd, function ( err, stdout, stderr ) {
+    _exec( cmd, { maxBuffer: Infinity }, function ( err, stdout, stderr ) {
       if ( !err ) {
         resolve( stdout );
       } else {
-        reject( stderr );
+        reject( { err: err, stderr: stderr } );
       }
       cb && cb( err, stdout, stderr );
     } );
@@ -519,6 +519,8 @@ var ignoreBranchFn = function ( branch ) {
     } ).length > 0;
 };
 
+nodeProcess.chdir( opts.gitDirectory );
+
 // check if can be applied to the given branch
 doExec( 'git name-rev --name-only HEAD', function ( err, stdout /*, stderr*/ ) {
   if ( err ) {

lib/hook.js

@@ -7,7 +7,10 @@ module.exports = {
     var p = findGitDir();
 
     return p.then( function ( gitDir ) {
-      return path.resolve( gitDir, './.git/hooks/pre-commit' );
+      return {
+        hookFile: path.resolve( gitDir, './.git/hooks/pre-commit' ),
+        gitDir: gitDir
+      };
     } );
   },
   install: function ( cfg ) {
@@ -27,17 +30,22 @@ module.exports = {
 
     prepushSource = prepushSource.replace( /require\(\s*'process'\s*\);/g, 'process;' )
       .replace( /require\(\s*'console'\s*\);/g, 'console;' )
-      .replace( /require\(\s*'\.\/Promise'\s*\);/g, 'global.Promise;' )
-      .replace( /require\(\s*'precommit-config'\s*\);/g, JSON.stringify( cfg, null, 2 ) + ';' );
+      .replace( /require\(\s*'\.\/Promise'\s*\);/g, 'global.Promise;' );
+
+
+    return this._pathToFile().then( function ( args ) {
+      var pathToFile = args.hookFile;
+      cfg.gitDirectory = args.gitDir;
+      prepushSource = prepushSource.replace( /require\(\s*'precommit-config'\s*\);/g, JSON.stringify( cfg, null, 2 ) + ';' );
 
-    return this._pathToFile().then( function ( pathToFile ) {
       write( pathToFile, prepushSource );
       fs.chmodSync( pathToFile, '755' );
       return pathToFile;
     } );
   },
   remove: function () {
-    return this._pathToFile().then( function ( pathToFile ) {
+    return this._pathToFile().then( function ( args ) {
+      var pathToFile = args.hookFile;
       var del = require( 'del' );
       del.sync( pathToFile, { force: true } );