-
Notifications
You must be signed in to change notification settings - Fork 17.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
os/exec: avoid calling LookPath in cmd.Start for resolved paths #61517
Conversation
This PR (HEAD: 6f84ef7) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/512155. Important tips:
|
Message from Ian Lance Taylor: Patch Set 2: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Gopher Robot: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Gopher Robot: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Gopher Robot: Patch Set 2: TryBot-Result-1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
This PR (HEAD: ad2b483) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/512155. Important tips:
|
Message from qiulaidongfeng: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Bryan Mills: Patch Set 3: (4 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Follow up on CL 511458, see https://go-review.googlesource.com/c/go/+/511458/2..4/src/cmd/go/main.go#b270 . For golang#36768 Change-Id: Icc2a4dbb1219b1d69dd10a900478957b0e975847
This PR (HEAD: bac7e66) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/512155. Important tips:
|
Message from qiulaidongfeng: Patch Set 5: (4 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Bryan Mills: Patch Set 5: Auto-Submit+1 Code-Review+2 Run-TryBot+1 (6 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Gopher Robot: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Message from Gopher Robot: Patch Set 5: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/512155. |
Follow up on CL 511458, see https://go-review.googlesource.com/c/go/+/511458/2..4/src/cmd/go/main.go#b270 . For #36768. Change-Id: Icc2a4dbb1219b1d69dd10a900478957b0e975847 Change-Id: Icc2a4dbb1219b1d69dd10a900478957b0e975847 GitHub-Last-Rev: bac7e66 GitHub-Pull-Request: #61517 Reviewed-on: https://go-review.googlesource.com/c/go/+/512155 Auto-Submit: Bryan Mills <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Matthew Dempsky <[email protected]> Reviewed-by: Bryan Mills <[email protected]> Run-TryBot: Bryan Mills <[email protected]>
This PR is being closed because golang.org/cl/512155 has been merged. |
Follow up on CL 511458, see https://go-review.googlesource.com/c/go/+/511458/2..4/src/cmd/go/main.go#b270 .
For #36768.
Change-Id: Icc2a4dbb1219b1d69dd10a900478957b0e975847