-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
cmd/cgo/internal/testsanitizers: check for go build and cgo in fuzzer and msan tests #64640
Conversation
… and msan tests Check if the platform can compile programs and has cgo support as a requirement to run the fuzzer and msan tests. Fixes [reserved]
This PR (HEAD: 9fae697) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/548715. Important tips:
|
Message from Bryan Mills: Patch Set 1: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
Message from Bryan Mills: Patch Set 1: Auto-Submit+1 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
Message from Go LUCI: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2023-12-11T19:37:28Z","revision":"1b5f177b630490b15050cbf5bbcee2d81f13952c"} Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
Message from Bryan Mills: Patch Set 1: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
Message from Go LUCI: Patch Set 1: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
Message from Go LUCI: Patch Set 1: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/548715. |
… and msan tests Make sure the platform we are running the tests on can compile programs and has cgo support in order to run the fuzzer and msan tests. This is the same approach used by the asan tests, which share the same requirements. Fixes #64626 Change-Id: I7c0b912dabdd1b7d7d44437e4ade5e5994994796 GitHub-Last-Rev: 9fae697 GitHub-Pull-Request: #64640 Reviewed-on: https://go-review.googlesource.com/c/go/+/548715 LUCI-TryBot-Result: Go LUCI <[email protected]> Auto-Submit: Bryan Mills <[email protected]> Reviewed-by: Matthew Dempsky <[email protected]> Reviewed-by: Bryan Mills <[email protected]>
This PR is being closed because golang.org/cl/548715 has been merged. |
… and msan tests Make sure the platform we are running the tests on can compile programs and has cgo support in order to run the fuzzer and msan tests. This is the same approach used by the asan tests, which share the same requirements. Fixes golang#64626 Change-Id: I7c0b912dabdd1b7d7d44437e4ade5e5994994796 GitHub-Last-Rev: 9fae697 GitHub-Pull-Request: golang#64640 Reviewed-on: https://go-review.googlesource.com/c/go/+/548715 LUCI-TryBot-Result: Go LUCI <[email protected]> Auto-Submit: Bryan Mills <[email protected]> Reviewed-by: Matthew Dempsky <[email protected]> Reviewed-by: Bryan Mills <[email protected]>
Make sure the platform we are running the tests on can compile programs
and has cgo support in order to run the fuzzer and msan tests. This is the
same approach used by the asan tests, which share the same requirements.
Fixes #64626