]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: replace the '[exec:git]' condition with a '[git]' condition
authorBryan C. Mills <bcmills@google.com>
Thu, 28 Jul 2022 20:59:02 +0000 (16:59 -0400)
committerGopher Robot <gobot@golang.org>
Mon, 24 Oct 2022 21:18:32 +0000 (21:18 +0000)
This makes it more obvious that the condition is testing for something
beyond just the existence of a 'git' executable.

For #27494.

Change-Id: I7608b6c84f9f373292687b3a2066b0ded7deb6e1
Reviewed-on: https://go-review.googlesource.com/c/go/+/421454
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
Auto-Submit: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

66 files changed:
src/cmd/go/scriptconds_test.go
src/cmd/go/testdata/script/README
src/cmd/go/testdata/script/build_buildvcs_auto.txt
src/cmd/go/testdata/script/get_404_meta.txt
src/cmd/go/testdata/script/get_brace.txt
src/cmd/go/testdata/script/get_custom_domain_wildcard.txt
src/cmd/go/testdata/script/get_dash_t.txt
src/cmd/go/testdata/script/get_domain_root.txt
src/cmd/go/testdata/script/get_dot_slash_download.txt
src/cmd/go/testdata/script/get_dotfiles.txt
src/cmd/go/testdata/script/get_insecure.txt
src/cmd/go/testdata/script/get_insecure_custom_domain.txt
src/cmd/go/testdata/script/get_insecure_env.txt
src/cmd/go/testdata/script/get_insecure_redirect.txt
src/cmd/go/testdata/script/get_insecure_update.txt
src/cmd/go/testdata/script/get_internal_wildcard.txt
src/cmd/go/testdata/script/get_issue11307.txt
src/cmd/go/testdata/script/get_issue16471.txt
src/cmd/go/testdata/script/get_issue22125.txt
src/cmd/go/testdata/script/get_legacy.txt
src/cmd/go/testdata/script/get_non_pkg.txt
src/cmd/go/testdata/script/get_race.txt
src/cmd/go/testdata/script/get_test_only.txt
src/cmd/go/testdata/script/get_update.txt
src/cmd/go/testdata/script/get_update_unknown_protocol.txt
src/cmd/go/testdata/script/get_update_wildcard.txt
src/cmd/go/testdata/script/get_vendor.txt
src/cmd/go/testdata/script/get_with_git_trace.txt
src/cmd/go/testdata/script/gopath_moved_repo.txt
src/cmd/go/testdata/script/govcs.txt
src/cmd/go/testdata/script/list_json_fields.txt
src/cmd/go/testdata/script/mod_convert.txt
src/cmd/go/testdata/script/mod_convert_tsv_insecure.txt
src/cmd/go/testdata/script/mod_download_git_decorate_full.txt
src/cmd/go/testdata/script/mod_download_hash.txt
src/cmd/go/testdata/script/mod_download_insecure_redirect.txt
src/cmd/go/testdata/script/mod_download_issue51114.txt
src/cmd/go/testdata/script/mod_download_private_vcs.txt
src/cmd/go/testdata/script/mod_get_direct.txt
src/cmd/go/testdata/script/mod_get_hash.txt
src/cmd/go/testdata/script/mod_get_major.txt
src/cmd/go/testdata/script/mod_get_pseudo.txt
src/cmd/go/testdata/script/mod_get_pseudo_other_branch.txt
src/cmd/go/testdata/script/mod_get_pseudo_prefix.txt
src/cmd/go/testdata/script/mod_getx.txt
src/cmd/go/testdata/script/mod_git_export_subst.txt
src/cmd/go/testdata/script/mod_gonoproxy.txt
src/cmd/go/testdata/script/mod_gopkg_unstable.txt
src/cmd/go/testdata/script/mod_init_glide.txt
src/cmd/go/testdata/script/mod_invalid_version.txt
src/cmd/go/testdata/script/mod_list_direct.txt
src/cmd/go/testdata/script/mod_list_odd_tags.txt
src/cmd/go/testdata/script/mod_missing_repo.txt
src/cmd/go/testdata/script/mod_prefer_compatible.txt
src/cmd/go/testdata/script/mod_pseudo_cache.txt
src/cmd/go/testdata/script/mod_replace_gopkgin.txt
src/cmd/go/testdata/script/mod_retract_pseudo_base.txt
src/cmd/go/testdata/script/mod_sumdb_golang.txt
src/cmd/go/testdata/script/reuse_git.txt
src/cmd/go/testdata/script/test_buildvcs.txt
src/cmd/go/testdata/script/vendor_list_issue11977.txt
src/cmd/go/testdata/script/vendor_test_issue11864.txt
src/cmd/go/testdata/script/vendor_test_issue14613.txt
src/cmd/go/testdata/script/version_buildvcs_git.txt
src/cmd/go/testdata/script/version_buildvcs_git_gpg.txt
src/cmd/go/testdata/script/version_buildvcs_nested.txt

index 6eb60b8ca2135e973e3bd390fef060810d450948..6e92cd104b7684d6ec6d56fb10c10f781189ce95 100644 (file)
@@ -23,23 +23,6 @@ import (
 func scriptConditions() map[string]script.Cond {
        conds := scripttest.DefaultConds()
 
-       // Our "exec" has a special case for plan9 git, which does not
-       // behave like git on other platforms.
-       //
-       // TODO(bcmills): replace this special-case "exec" with a more tailored "git"
-       // condition.
-       conds["exec"] = script.CachedCondition(
-               conds["exec"].Usage().Summary,
-               func(name string) (bool, error) {
-                       if runtime.GOOS == "plan9" && name == "git" {
-                               // The Git command is usually not the real Git on Plan 9.
-                               // See https://golang.org/issues/29640.
-                               return false, nil
-                       }
-                       _, err := exec.LookPath(name)
-                       return err == nil, nil
-               })
-
        add := func(name string, cond script.Cond) {
                if _, ok := conds[name]; ok {
                        panic(fmt.Sprintf("condition %q is already registered", name))
@@ -60,6 +43,7 @@ func scriptConditions() map[string]script.Cond {
        add("fuzz-instrumented", sysCondition("-fuzz with instrumentation", platform.FuzzInstrumented))
        add("gc", script.BoolCondition(`runtime.Compiler == "gc"`, runtime.Compiler == "gc"))
        add("gccgo", script.BoolCondition(`runtime.Compiler == "gccgo"`, runtime.Compiler == "gccgo"))
+       add("git", lazyBool("the 'git' executable exists and provides the standard CLI", hasWorkingGit))
        add("GODEBUG", script.PrefixCondition("GODEBUG contains <suffix>", hasGodebug))
        add("GOEXPERIMENT", script.PrefixCondition("GOEXPERIMENT <suffix> is enabled", hasGoexperiment))
        add("link", lazyBool("testenv.HasLink()", testenv.HasLink))
@@ -163,3 +147,13 @@ func isTrimpath() (bool, error) {
        }
        return false, nil
 }
+
+func hasWorkingGit() bool {
+       if runtime.GOOS == "plan9" {
+               // The Git command is usually not the real Git on Plan 9.
+               // See https://golang.org/issues/29640.
+               return false
+       }
+       _, err := exec.LookPath("git")
+       return err == nil
+}
index edbf645476e3d8c79dc8d2d2481bab97983c0886..8724d052e3493d503828de4f9c671c73b98bf164 100644 (file)
@@ -410,6 +410,8 @@ The available conditions are:
        runtime.Compiler == "gc"
 [gccgo]
        runtime.Compiler == "gccgo"
+[git]
+       the 'git' executable exists and provides the standard CLI
 [hurd]
        host GOOS=hurd
 [illumos]
index dd9eef5f8289e52eb64ebf132a0291f0db45623a..cfd5d8243b8e2007f4d32858477e5220d2215819 100644 (file)
@@ -2,7 +2,7 @@
 # not attempt to stamp VCS information when the VCS tool is not present.
 
 [short] skip
-[!exec:git] skip
+[!git] skip
 
 cd sub
 exec git init .
index 29fc5421e12fb9824c62dcec8d4f0bc526d16cf4..553afb9ae105c3960356391c733d0c6eff49c951 100644 (file)
@@ -1,7 +1,7 @@
 # golang.org/issue/13037: 'go get' was not parsing <meta> tags in 404 served over HTTPS.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GONOSUMDB=bazil.org,github.com,golang.org
 env GO111MODULE=off
index 3449a0c2c82995fc54786f672fb2caf1f4a3b8f9..34f66a62a32bc5d2dfab9cff3a509b167082ce76 100644 (file)
@@ -1,6 +1,6 @@
 env GO111MODULE=off
 
-[!exec:git] skip
+[!git] skip
 
 # Set up some empty repositories.
 cd $WORK/_origin/foo
index cda25e12b0e6ea0a2f3ef7a0165fcfca5e8b7df0..32ddd92b59882d16e5d0792c4050bdee19cd282a 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get -u rsc.io/pdf/...
index baac91686868a66a83d51d206993a7df8ca728cb..0a0a6084de56b1e9f7d044f865536a6e6e3bbc58 100644 (file)
@@ -1,7 +1,7 @@
 # Tests issue 8181
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get -v -t github.com/rsc/go-get-issue-8181/a github.com/rsc/go-get-issue-8181/b
index 918784869b312a10b2f0535f2eff8b12a3c7fcb0..250fa6464f518ff0059c4769ca1b3a6f37d977aa 100644 (file)
@@ -2,7 +2,7 @@
 # go get foo.io (not foo.io/subdir) was not working consistently.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # go-get-issue-9357.appspot.com is running
index dbaf46ced363df24a3771014225cd42948878a54..2af9564fe42e30c4d1f1d921df43c7b0f5fc6009 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # Tests Issues #9797 and #19769
index 6757f9dce344f502cc532563ee2b0c954fd653f6..676a044cdc0a86010eba1b0b562d1b6dcbad29dc 100644 (file)
@@ -1,7 +1,7 @@
 env GO111MODULE=off
 [short] skip
 
-[!exec:git] skip
+[!git] skip
 
 # Set up a benign repository and a repository with a dotfile name.
 cd $WORK/_origin/foo
index 69930f7107a58b9ddeb4729b313ccb8cc999969e..0079220e40816fe1c15384f96696e6c3be08e152 100644 (file)
@@ -1,7 +1,7 @@
 # TODO(matloob): Split this test into two? It's one of the slowest tests we have.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env PATH=$WORK/tmp/bin${:}$PATH
 go build -o $WORK/tmp/bin/ssh ssh.go
index 7eba42e873e011a8c7d479e53a2e04e360bbf20e..3a0765f2a1ed6f596342869661055909b453b46d 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 ! go get -d insecure.go-get-issue-15410.appspot.com/pkg/p
index 8d88427c319d22d7990e8664279e39b9f9b178ca..87484404eb67a4495b370faa6480f3a7238d96b1 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # GOPATH: Set up
 env GO111MODULE=off
index fb5f26951cdad9dfdd72f67775db707149f7e9a4..2e5ec4e0d8c6da8b87e80fe5e50449547e8c56fb 100644 (file)
@@ -2,7 +2,7 @@
 # golang.org/issue/34049: 'go get' would panic in case of an insecure redirect in GOPATH mode
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=off
 
index e1a1a23d47e475a34927e55bd570389a0b27dc09..01660d58b4b1753292a211f2673c03b87c19ac7b 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # Clone the repo via HTTP manually.
index ff20d4ba04e64c804c5e707eea85a6e6a5836d69..71ecb8c8fe16028237c0e49dc684b951ea1e6193 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # This used to fail with errors about internal packages
index 9d6b7dde0196653caa333d7dcd48162a7bca32d9..dc46e747123b35c3e394bcc5803f4b1a72dfe15a 100644 (file)
@@ -1,7 +1,7 @@
 # go get -u was not working except in checkout directory
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 env GOPATH=$WORK/tmp/gopath
index 2a2225a444da9787fc3fb97471891e3cf87e0699..60369132f95a6baa34dfe0b3a16db9ca8a4c70a9 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=off
 
index 6fdbe81cd304583cce0f26d99e929f82b5d65234..3418df8429576c21d5d34f37100c815c39979717 100644 (file)
@@ -1,7 +1,7 @@
 # This test verifies a fix for a security issue; see https://go.dev/issue/22125.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 [!exec:svn] skip
 
 env GO111MODULE=off
index 938d42868af714d44405898a14c605083aa1cc6d..2909b736055aa81597c7a5d24d1f4e7a21e704ca 100644 (file)
@@ -4,7 +4,7 @@
 # The test still seems to be useful as a test of direct-mode go get.
 
 [short] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 env GOPATH=$WORK/tmp/d1
index a87853052d75e6e0c179c2dcc26b85174257a0ab..5eac1e337ee6d7f703e93c8a437313e1e156f257 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GOBIN=$WORK/tmp/gobin
 env GO111MODULE=off
index 16a560afcac9b00690fe0e26451c7b9c1c196b47..87fbf62863b037ba28e224a3e4e27dd6b6992ec3 100644 (file)
@@ -1,7 +1,7 @@
 # Tests issue #20502
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 [!race] skip
 env GO111MODULE=off
 
index a3f38ddbab3886b51ece548991200d11167e331d..ec8baf9553f9ab06f6fc47b56e2c71d0ce20a42d 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get golang.org/x/tour/content...
index 9afce6a443babc7ffe5c43d780b72f851ad4be1f..5aeb99000b203a6d9839e6439018fb9678a02e98 100644 (file)
@@ -3,7 +3,7 @@
 # former dependencies, not current ones.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # Rewind
index b00adea70b22a4d7ed1a8f9d3741487bc5227414..12807ad675f7d1b048370899e4c250d8bdad202e 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 # Clone the repo via HTTPS manually.
index 4e66004014a6c842ffbe6fc5dbbb1dcb3e479e39..01e2c374aefea0fda74d4be87c9a39b068cecdb4 100644 (file)
@@ -1,7 +1,7 @@
 # Issue 14450: go get -u .../ tried to import not downloaded package
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get github.com/tmwh/go-get-issue-14450/a
index 4ebb8a26b6dd3047872f6f8ccd566ce09fd50ab3..f9a4a6bf7bff1c7cc52099422d2552642ebd95a6 100644 (file)
@@ -12,7 +12,7 @@ go get -d
 go get -t -d
 
 [!net] stop
-[!exec:git] stop
+[!git] stop
 
 cd $GOPATH/src
 
index 98854c72ad262b3d360760429415000ee2053520..abc7014e45749d079a7db177772e35e7dc1fac67 100644 (file)
@@ -3,7 +3,7 @@ env GO111MODULE=off
 env GIT_TRACE=1
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # go get should be success when GIT_TRACE set
 go get golang.org/x/text
index 99d80bff5d94c324a377acb8b535081167ccdb91..5815d73a4aeafe0873a739fb8fa559a6144c9e69 100644 (file)
@@ -18,14 +18,14 @@ env GIT_ASKPASS=$PWD/replace.exe
 
 # Test that 'go get -u' reports moved git packages.
 
-[exec:git] go get -d rsc.io/pdf
-[exec:git] go get -d -u rsc.io/pdf
-[exec:git] exec ./replace.exe pdf rsc.io/pdf/.git/config
-
-[exec:git] ! go get -d -u rsc.io/pdf
-[exec:git] stderr 'is a custom import path for'
-[exec:git] ! go get -d -f -u rsc.io/pdf
-[exec:git] stderr 'validating server certificate|[nN]ot [fF]ound'
+[git] go get -d rsc.io/pdf
+[git] go get -d -u rsc.io/pdf
+[git] exec ./replace.exe pdf rsc.io/pdf/.git/config
+
+[git] ! go get -d -u rsc.io/pdf
+[git] stderr 'is a custom import path for'
+[git] ! go get -d -f -u rsc.io/pdf
+[git] stderr 'validating server certificate|[nN]ot [fF]ound'
 
 
 # Test that 'go get -u' reports moved Mercurial packages.
index 46f1bd0da2ada67d7cb80664699f6ac3d7e3881c..419a6c5d20dd44c9aa751db60801730cbf1ae24c 100644 (file)
@@ -67,7 +67,7 @@ stderr '^go: rsc.io/nonexist.bzr/hello: GOVCS disallows using bzr for public rsc
 # git is OK by default
 env GOVCS=
 env GONOSUMDB='*'
-[net] [exec:git] [!short] go get rsc.io/sampler
+[net] [git] [!short] go get rsc.io/sampler
 
 # hg is OK by default
 env GOVCS=
@@ -150,7 +150,7 @@ stderr '^package rsc.io/nonexist.bzr/hello: GOVCS disallows using bzr for public
 # git is OK by default
 env GOVCS=
 env GONOSUMDB='*'
-[net] [exec:git] [!short] go get rsc.io/sampler
+[net] [git] [!short] go get rsc.io/sampler
 
 # hg is OK by default
 env GOVCS=
index 5ddbb7385e0d9793875d787ff4e9bce80710e917..54d22201100670377f9d28390ee4e4ac11b612fa 100644 (file)
@@ -26,7 +26,7 @@ go list -json=Deps
 stdout '"Deps": \['
 stdout '"errors",'
 
-[!exec:git] skip
+[!git] skip
 
 # Test -json=<field> without Stale skips computing buildinfo
 cd repo
index f60fe87637cb3e0c4389c3cac54db09b933216c6..1c9d626acdfe8ec41a8df710f7be87295ae8855b 100644 (file)
@@ -1,6 +1,6 @@
 [short] skip
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=
index 283e2d99366bb8c914f9b4428943355fdd286077..9910ce769936d3edf010a6cd3055b9bb8c06aa1d 100644 (file)
@@ -3,7 +3,7 @@ env GOPROXY=direct
 env GOSUMDB=off
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # secure fetch should report insecure warning
 cd $WORK/test
index 3b19acc1b152a0e560b7eec3acaca34c0b910a06..997b502b247b9bc38176fd5ad3ad22573f8e977b 100644 (file)
@@ -1,7 +1,7 @@
 env GO111MODULE=on
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GOPROXY=direct
 env HOME=$WORK/home/gopher
index 5a42c4b0726177cca8cdd18f11018c25c3e4e886..e62a165e75908b48b1a3cfbeaff6b24af8563ac7 100644 (file)
@@ -2,7 +2,7 @@ env GO111MODULE=on
 
 # Testing mod download with non semantic versions; turn off proxy.
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOPROXY=direct
 env GOSUMDB=off
 
index 46eb666686502cafdea600ab9b0ec044bb454212..fed5b8da4bf1ad943178349b39c5f83169a16b39 100644 (file)
@@ -1,7 +1,7 @@
 # golang.org/issue/29591: 'go get' was following plain-HTTP redirects even without -insecure (now replaced by GOINSECURE).
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 92479c6dd3cd0a6db2e9a8fafb1ba14980acad65..2aa3c2d19a61d8c9d3a67b43eb35d1b0dd66cd28 100644 (file)
@@ -1,5 +1,5 @@
 [short] skip
-[!exec:git] skip
+[!git] skip
 [!net] skip
 [!linux] skip  # Uses XDG_CONFIG_HOME
 
index da9fe0290b7a4d955ec22c300a8e0ce7aa772274..7459b80a6cf7db9078fe3040cda3b5b1a343cfa0 100644 (file)
@@ -2,7 +2,7 @@ env GO111MODULE=on
 
 # Testing stderr for git ls-remote; turn off proxy.
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOPROXY=direct
 
 ! go mod download github.com/golang/nonexist@latest
index 856e05bc329328d2fe33771d12e8ac7d7cfb11a2..b7b052984c3ff51ca4d06e09aadc1cf1ec0f2943 100644 (file)
@@ -4,7 +4,7 @@
 
 [short] skip
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 3bb3ee7880948308803d2930aa3c7468cb48a875..63e9e46db1ca60403f7f32e36b96b98d4383c2fe 100644 (file)
@@ -2,7 +2,7 @@ env GO111MODULE=on
 env GOPROXY=direct
 env GOSUMDB=off
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # fetch commit hash reachable from refs/heads/* and refs/tags/* is OK
 go list -m golang.org/x/time@8be79e1e0910c292df4e79c241bb7e8f7e725959 # on master branch
index 2db13180bd79786283919bba2e6e5d61ebe2c5c3..76c9de5cc7e12fce7657b2da9ebb729374504f45 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index b964ae448453b9860fa1d854109110e975649b7c..7b43c69c1d4a612582c0d87cd2ac2ab05c275fd4 100644 (file)
@@ -2,7 +2,7 @@ env GO111MODULE=on
 
 # Testing git->module converter's generation of +incompatible tags; turn off proxy.
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOPROXY=direct
 env GOSUMDB=off
 
index d085f4fa3c8e6b2319bce5ed94c9b569f58d3ded..21f900f303dd9169bab9c8cce51a4fd2ddf77e41 100644 (file)
@@ -7,7 +7,7 @@ env GOSUMDB=off
 # supplied to 'go get', regardless of branches
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # For this test repository:
 #  tag v0.2.1 is most recent tag on master itself
index 8e6cd907f183495842855232179f2ed9f16e3bb1..513450d4a5af3b1f850584d19403854976e683be 100644 (file)
@@ -7,7 +7,7 @@ env GOSUMDB=off
 # to 'go get', when using a repo with go.mod in a sub directory.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # For this test repository go.mod resides in sub/ (only):
 #  master is not tagged
index b3d06c17c85afa0111bdcc10999d5425a2d8deca..dee3f747c6c38bd5449699f9d5cd08f2e68162ce 100644 (file)
@@ -1,6 +1,6 @@
 [short] skip
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index a28b4f2d67b1e858cc56b1d3597f999aefdb088f..44fb501fa85515dac55e9610f7ea9266adfda4e0 100644 (file)
@@ -3,7 +3,7 @@ env GOPROXY=direct
 
 # Testing that git export-subst is disabled
 [!net] skip
-[!exec:git] skip
+[!git] skip
 go build
 
 -- x.go --
index d42d668f679eeaf57a76adcdda70d0f6ee313dca..98a1d28b5620631a24fb9d0bd215a286ffeae15e 100644 (file)
@@ -37,7 +37,7 @@ stderr '^go: golang.org/x/text: module lookup disabled by GOPROXY=off$'
 
 # GONOPROXY bypasses proxy
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOPRIVATE=none
 env GONOPROXY='*/fortune'
 ! go get rsc.io/fortune # does not exist in real world, only on test proxy
index 58bbc7651b3b5351dab83c0ee3dd14d1e9927417..3608bcd79607da3ca28ff2178f8454e700851cce 100644 (file)
@@ -8,7 +8,7 @@ cp go.mod.empty go.mod
 go list
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 skip  # TODO(#54503): redirect gopkg.in requests to a local server and re-enable.
 
index 373810c7687275f39d91a82e2ebc739f9421d64e..2126ae5b48768b5181728ee868a9805c706d3f0d 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 8385b08d95f105c061e2b395705f55cbd2016e7b..d1e1da4492512b98cff5631288f099fb42a69f84 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 3aa1881554edcec6768fa985f81e7b4169a3f7a8..3e7c479d664952d7274353416a9f35ee8d972548 100644 (file)
@@ -3,7 +3,7 @@ env GOPROXY=direct
 env GOSUMDB=off
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 # golang.org/issue/33099: if an import path ends in a major-version suffix,
 # ensure that 'direct' mode can resolve the package to the module.
index c1f40cdf3a312cd2079bd8d73ecb4962f52aa6e0..b413e87e18ac51d796d2323761352647cde3e6d5 100644 (file)
@@ -1,5 +1,5 @@
 [short] skip
-[!exec:git] skip
+[!git] skip
 [!net] skip
 
 env GOPROXY=direct
index b91a8dbedacb52277ef4cf0f9285a341e92e109c..4b403fe51e1dce836096a871748351f60dc2731b 100644 (file)
@@ -3,7 +3,7 @@
 # tags for prefixes of the module path.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 8e88997a3c3089c9360128c1d58dac2cd841e1eb..7ba5eb4fa9cb0b125f04e231c7ddd751cbdbd77e 100644 (file)
@@ -34,7 +34,7 @@ stderr '^go: github.com/russross/blackfriday@patch: can''t query version "patch"
 # order to determine whether it contains a go.mod file, and part of the point of
 # the proxy is to avoid fetching unnecessary data.)
 
-[!exec:git] stop
+[!git] stop
 env GOPROXY=direct
 
 go list -versions -m github.com/russross/blackfriday github.com/russross/blackfriday
index dd89614b9f9f313d935854e7d1ee1957fcad47cd..a7ceac42b4262f96dfa10e825383a137064b0cb6 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index df752d9716e538b4bb89c73679d45df8e1b395ca..996dccd29f47bb965c1fb5acd7d1402fb081e112 100644 (file)
@@ -6,7 +6,7 @@
 
 [short] skip
 [!net] skip
-[!exec:git] skip
+[!git] skip
 
 env GO111MODULE=on
 env GOPROXY=direct
index 27c2b670658e214bd27be9f739da3a6a64560812..c52f0b851e15d24810e64d0b2ba034b1a31c9159 100644 (file)
@@ -3,7 +3,7 @@
 # Verifies golang.org/issue/41700.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOPROXY=direct
 env GOSUMDB=off
 go mod init m
index a48a5ba1b048c038c63d79fec29903cba91f99f5..7dd6cdc06676515d60fbd42b9b2aa0cc81c7b00b 100644 (file)
@@ -12,7 +12,7 @@ stdout '^sum.golang.org$'
 # Download direct from github.
 
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GOSUMDB=sum.golang.org
 env GOPROXY=direct
 
index a5a0c8a9a0bd9078af0cd862db7736377a5c6b46..8c23bde49b6af562f9ec1519342270f12816fd73 100644 (file)
@@ -1,5 +1,5 @@
 [short] skip
-[!exec:git] skip
+[!git] skip
 [!net] skip
 
 env GO111MODULE=on
index 965f76bf0d05df965607cf7fb4581f89a99ac2b5..db844f88b35341114653a36032e2a2efe7efafd2 100644 (file)
@@ -3,7 +3,7 @@
 # test binaries are almost never distributed to users.)
 
 [short] skip
-[!exec:git] skip
+[!git] skip
 
 exec git init
 
index cdab33c0892bff21c93c41a17d155c711be0eb16..35c82c7b2109280ad13edebdcdb0397dc261fb3d 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get github.com/rsc/go-get-issue-11864
index b9ee434c4c2cf90f657728e3348c7ee7ef0a08eb..ff179cbe8b2d4503004633ca510b43e63c362e49 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 go get github.com/rsc/go-get-issue-11864
index 8759445aea48aed41c8f2454e8f3b554da59b5d3..7822deeeda91589e230d723f6a9b34407b54c41f 100644 (file)
@@ -1,5 +1,5 @@
 [!net] skip
-[!exec:git] skip
+[!git] skip
 env GO111MODULE=off
 
 cd $GOPATH
index 44706870e22f22d9113c9ddb6a1e75754ace39db..e7ca06d422f510e42d8a983098688879eda089cd 100644 (file)
@@ -2,7 +2,7 @@
 # controlled with -buildvcs. This test focuses on Git. Other tests focus on
 # other VCS tools but may not cover common functionality.
 
-[!exec:git] skip
+[!git] skip
 [short] skip
 env GOBIN=$WORK/gopath/bin
 env oldpath=$PATH
index dcf97d7c447f5b5d67c9321ff3ed15333122f1fb..c88075c5bc7de0d5698d941f6d6ff1aa6f7728e0 100644 (file)
@@ -2,7 +2,7 @@
 # the current commit is signed and the use has configured git to display commit
 # signatures.
 
-[!exec:git] skip
+[!git] skip
 [!exec:gpg] skip
 [short] skip
 env GOBIN=$GOPATH/bin
index a0c69f9c12acfdf36bdf40898dba0f668eb81025..6dab8474b59d44b862d98f46d8608bf2d56b1b21 100644 (file)
@@ -1,4 +1,4 @@
-[!exec:git] skip
+[!git] skip
 [!exec:hg] skip
 [short] skip
 env GOFLAGS='-n -buildvcs'