If GOBIN is set when installing, prefer that over GOPATH (#232)

Author: justinclift

magefile.go

@@ -27,12 +27,20 @@ func Install() error {
 	}
 
 	gocmd := mg.GoCmd()
-	gopath, err := sh.Output(gocmd, "env", "GOPATH")
+	// use GOBIN if set in the environment, otherwise fall back to first path
+	// in GOPATH environment string
+	bin, err := sh.Output(gocmd, "env", "GOBIN")
 	if err != nil {
-		return fmt.Errorf("can't determine GOPATH: %v", err)
+		return fmt.Errorf("can't determine GOBIN: %v", err)
+	}
+	if bin == "" {
+		gopath, err := sh.Output(gocmd, "env", "GOPATH")
+		if err != nil {
+			return fmt.Errorf("can't determine GOPATH: %v", err)
+		}
+		paths := strings.Split(gopath, string([]rune{os.PathListSeparator}))
+		bin = filepath.Join(paths[0], "bin")
 	}
-	paths := strings.Split(gopath, string([]rune{os.PathListSeparator}))
-	bin := filepath.Join(paths[0], "bin")
 	// specifically don't mkdirall, if you have an invalid gopath in the first
 	// place, that's not on us to fix.
 	if err := os.Mkdir(bin, 0700); err != nil && !os.IsExist(err) {