Support color = auto

This is done using the test command as the stdlib lacks any way to do this without using syscalls directly
jdholtz · Sep 27, 2018 · e78070e · e78070e
1 parent 9ac4ab6
commit e78070e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/exec.go b/exec.go
@@ -121,3 +121,10 @@ func passToGit(dir string, _args ...string) *exec.Cmd {
 	cmd := exec.Command(config.GitBin, args...)
 	return cmd
 }
+
+func isTty() bool {
+	cmd := exec.Command("test", "-t", "1")
+	cmd.Stdout = os.Stdout
+	err := cmd.Run()
+	return err == nil
+}
diff --git a/main.go b/main.go
@@ -146,12 +146,14 @@ func initAlpm() error {
 		return err
 	}
 
-	if value, _, _ := cmdArgs.getArg("color"); value == "always" || value == "auto" {
+	if value, _, _ := cmdArgs.getArg("color"); value == "always" {
 		useColor = true
+	} else if value == "auto" {
+		useColor = isTty()
 	} else if value == "never" {
 		useColor = false
 	} else {
-		useColor = pacmanConf.Color
+		useColor = pacmanConf.Color && isTty()
 	}
 
 	return nil