getGitHooksPath helper function

cmd/add.go

@@ -3,7 +3,6 @@ package cmd
 import (
 	"io/ioutil"
 	"log"
-	"os/exec"
 	"path/filepath"
 	"strings"
 
@@ -94,7 +93,7 @@ else
 fi
 `
 
-	pathToFile := filepath.Join(getGitHooksDir(), hookName)
+	pathToFile := filepath.Join(getGitHooksPath(), hookName)
 
 	if yes, _ := afero.Exists(fs, pathToFile); yes {
 		if isLefthookFile(pathToFile) {
@@ -150,18 +149,6 @@ func addLocalHookDir(hookName string, fs afero.Fs) {
 	check(err)
 }
 
-func getGitHooksDir() string {
-	cmd := exec.Command("git", "rev-parse", "--git-common-dir")
-
-	outputBytes, err := cmd.CombinedOutput()
-	if err != nil {
-		panic(err)
-	}
-
-	gitDir := strings.TrimSpace(string(outputBytes))
-	return filepath.Join(getRootPath(), gitDir, "hooks")
-}
-
 func isLefthookFile(pathFile string) bool {
 	file, err := ioutil.ReadFile(pathFile)
 	if err != nil {

cmd/cmd_test.go

@@ -35,7 +35,7 @@ func TestInstallCmdExecutor(t *testing.T) {
 		"prepare-commit-msg",
 	}
 
-	files, err := afero.ReadDir(fs, getGitHooksDir())
+	files, err := afero.ReadDir(fs, getGitHooksPath())
 	assert.NoError(t, err)
 
 	actualFiles := []string{}
@@ -60,7 +60,7 @@ func TestAddCmdExecutor(t *testing.T) {
 		"pre-commit",
 	}
 
-	files, _ := afero.ReadDir(fs, getGitHooksDir())
+	files, _ := afero.ReadDir(fs, getGitHooksPath())
 	actualFiles := []string{}
 	for _, f := range files {
 		actualFiles = append(actualFiles, f.Name())
@@ -82,7 +82,7 @@ func TestAddCmdExecutor(t *testing.T) {
 		"pre-push.old",
 	}
 
-	files, _ = afero.ReadDir(fs, getGitHooksDir())
+	files, _ = afero.ReadDir(fs, getGitHooksPath())
 	actualFiles = []string{}
 	for _, f := range files {
 		actualFiles = append(actualFiles, f.Name())
@@ -147,7 +147,8 @@ func presetConfig(fs afero.Fs) {
 	fs.Mkdir(filepath.Join(getRootPath(), ".lefthook/commit-msg"), defaultFilePermission)
 	fs.Mkdir(filepath.Join(getRootPath(), ".lefthook/pre-commit"), defaultFilePermission)
 
-	fs.MkdirAll(filepath.Join(getRootPath(), ".git", "hooks"), defaultFilePermission)
+	setGitHooksPath(".git/hooks")
+	fs.MkdirAll(getGitHooksPath(), defaultFilePermission)
 }
 
 func presetExecutable(hookName string, hookGroup string, exitCode string, fs afero.Fs) {

cmd/install.go

@@ -205,7 +205,7 @@ func configChecksum(fs afero.Fs) string {
 func recordedChecksum() string {
 	pattern := regexp.MustCompile(`(?:# lefthook_version: )(\w+)`)
 
-	file, err := ioutil.ReadFile(filepath.Join(getGitHooksDir(), checkSumHook))
+	file, err := ioutil.ReadFile(filepath.Join(getGitHooksPath(), checkSumHook))
 	if err != nil {
 		return ""
 	}

cmd/root.go

@@ -8,8 +8,8 @@ import (
 	"strings"
 
 	"github.com/logrusorgru/aurora"
-	"github.com/spf13/afero"
 	"github.com/mattn/go-isatty"
+	"github.com/spf13/afero"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 )
@@ -28,11 +28,12 @@ const (
 )
 
 var (
-	Verbose      bool
-	NoColors     bool
-	rootPath     string
-	cfgFile      string
-	originConfig *viper.Viper
+	Verbose              bool
+	NoColors             bool
+	rootPath             string
+	gitHooksPath         string
+	cfgFile              string
+	originConfig         *viper.Viper
 	configFileExtensions = []string{".yml", ".yaml"}
 
 	au aurora.Aurora
@@ -57,7 +58,7 @@ lefthook install`,
 		message := `This command must be executed within git repository.
 Change working directory or initialize new repository with 'git init'.`
 
-			log.Fatal(au.Brown(message))
+		log.Fatal(au.Brown(message))
 	},
 }
 
@@ -92,6 +93,7 @@ func initConfig() {
 	log.SetFlags(0)
 
 	setRootPath(rootExecutionRelPath)
+	setGitHooksPath(getHooksPathFromGitConfig())
 
 	// store original config before merge
 	originConfig = viper.New()
@@ -135,6 +137,25 @@ func setRootPath(path string) {
 	rootPath = strings.TrimSpace(string(outputBytes))
 }
 
+func getGitHooksPath() string {
+	return gitHooksPath
+}
+
+func setGitHooksPath(path string) {
+	gitHooksPath = filepath.Join(getRootPath(), path)
+}
+
+func getHooksPathFromGitConfig() string {
+	cmd := exec.Command("git", "rev-parse", "--git-path", "hooks")
+
+	outputBytes, err := cmd.CombinedOutput()
+	if err != nil {
+		panic(err)
+	}
+
+	return strings.TrimSpace(string(outputBytes))
+}
+
 func getSourceDir() string {
 	return filepath.Join(getRootPath(), viper.GetString(configSourceDirKey))
 }

cmd/uninstall.go

@@ -70,13 +70,13 @@ func deleteSourceDirs(fs afero.Fs) {
 
 // DeleteGitHooks read the config and remove all git hooks except
 func DeleteGitHooks(fs afero.Fs) {
-	hooksPath := filepath.Join(getRootPath(), ".git", "hooks")
+	hooksPath := getGitHooksPath()
 
 	hooks, err := afero.ReadDir(fs, hooksPath)
 	if (err != nil) {
-		log.Println("⚠️ ", au.Bold(".git/hooks"), "directory does not exist, creating")
+		log.Println("⚠️ ", au.Bold(hooksPath), "directory does not exist, creating")
 		if err := os.Mkdir(hooksPath, os.ModePerm); err != nil {
-			log.Println(au.Brown("🚨 Failed to create"), au.Bold(".git/hooks"), au.Brown("directory"))
+			log.Println(au.Brown("🚨 Failed to create"), au.Bold(hooksPath), au.Brown("directory"))
 			log.Fatal(err)
 		}
 	}
@@ -99,7 +99,7 @@ func revertOldGitHooks(fs afero.Fs) {
 		return
 	}
 
-	hooksPath := filepath.Join(getRootPath(), ".git", "hooks")
+	hooksPath := getGitHooksPath()
 	for _, file := range hookGroups {
 		hookFilePath := filepath.Join(hooksPath, file.Name()+".old")