Only add explicit -mod=vendor if needed

-mod=vendor is not needed if it's already set in GOFLAGS. So lets not
set it unless it's needed.

This is an issue for commands such as `go test` and `go vet` (
see: golang/go#32471 ). This avois such an
issue.

Author: JAORMX

internal/util/projutil/exec.go

@@ -124,7 +124,7 @@ func (opts GoCmdOptions) getGeneralArgsWithCmd(cmd string) ([]string, error) {
 		}
 		// Does the first "go" subcommand accept -mod=vendor?
 		_, ok := validVendorCmds[bargs[0]]
-		if err == nil && info.IsDir() && ok {
+		if err == nil && info.IsDir() && ok && needsModVendor() {
 			bargs = append(bargs, "-mod=vendor")
 		}
 	}
@@ -135,6 +135,10 @@ func (opts GoCmdOptions) getGeneralArgsWithCmd(cmd string) ([]string, error) {
 	return bargs, nil
 }
 
+func needsModVendor() bool {
+	return !strings.Contains(os.Getenv("GOFLAGS"), "-mod=vendor")
+}
+
 func (opts GoCmdOptions) setCmdFields(c *exec.Cmd) {
 	c.Env = append(c.Env, os.Environ()...)
 	if len(opts.Env) != 0 {