-
Notifications
You must be signed in to change notification settings - Fork 18k
runtime: Goexit on C-created thread report more useful error message #69126
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
Conversation
For golang#68275 Change-Id: Ic213a4e59aa24f125895946bf0001b82e55567bd
Change-Id: I641ccc367a3d63a2d695c3d98961911f7b5c9a3c
This PR (HEAD: fb82b6f) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Ian Lance Taylor: Patch Set 2: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: 767bb61) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from Ian Lance Taylor: Patch Set 3: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: 982c9aa) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from Ian Lance Taylor: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 4: Commit-Queue+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 4: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-07T07:13:56Z","revision":"9cde381f8699ea7f1466bf8687da41aad8ac570a"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 4: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 4: This CL has failed the run. Reason: Tryjob golang/try/gotip-windows-amd64 has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 4: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: c7bdb13) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from qiu laidongfeng2: Patch Set 5: Commit-Queue+1 (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 5: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-07T07:57:52Z","revision":"c7892afb59ad3ecf87347a678a96bbb3769bc9eb"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 5: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 5: This CL has failed the run. Reason: Tryjob golang/try/gotip-windows-386 has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 5: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: 068b3cf) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from qiu laidongfeng2: Patch Set 6: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 6: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-07T08:23:48Z","revision":"bcbc41eeffd470ad4da6f73bced91408b11a54ef"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 6: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 6: This CL has failed the run. Reason: Tryjob golang/try/gotip-windows-386 has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 6: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 9: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 9: This CL has failed the run. Reason: Failed Tryjobs:
Build or test failure, click here for results. To reproduce, try Additional links for debugging:
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 9: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 10: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 10: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-08T05:18:26Z","revision":"e4c6c99b7d5a0a51599bf69bffcf9f6448105d8e"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 10: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 10: This CL has failed the run. Reason: Tryjob golang/try/gotip-linux-arm64-boringcrypto has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 10: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: 9a8fd85) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from qiu laidongfeng2: Patch Set 11: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 11: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-08T05:57:58Z","revision":"d7c08983c11aa2feb3d4a8e0a803550b4e9891d8"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 11: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 11: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 11: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Ian Lance Taylor: Patch Set 11: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
This PR (HEAD: b89bf3c) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/609296. Important tips:
|
Message from qiu laidongfeng2: Patch Set 12: Commit-Queue+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 12: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-09T09:01:57Z","revision":"4e89a4acc785a10bc7394b70c445af1ff3ce5fa1"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from qiu laidongfeng2: Patch Set 12: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 12: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 12: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Ian Lance Taylor: Patch Set 12: Auto-Submit+1 Code-Review+2 Commit-Queue+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 12: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2024-09-10T20:44:04Z","revision":"4e89a4acc785a10bc7394b70c445af1ff3ce5fa1"} Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Ian Lance Taylor: Patch Set 12: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Message from Go LUCI: Patch Set 12: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/609296. |
Fixes #68275 Change-Id: I47b7a2092f1b4d48aebf437db4e329815c956bb9 GitHub-Last-Rev: b89bf3c GitHub-Pull-Request: #69126 Reviewed-on: https://go-review.googlesource.com/c/go/+/609296 Reviewed-by: Tim King <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Auto-Submit: Ian Lance Taylor <[email protected]> Reviewed-by: Ian Lance Taylor <[email protected]>
This PR is being closed because golang.org/cl/609296 has been merged. |
Fixes #68275