[stable] etc.linux.memoryerror: Adapt to dynamic SIGSTKSZ since glibc v2.34#22473
[stable] etc.linux.memoryerror: Adapt to dynamic SIGSTKSZ since glibc v2.34#22473kinke merged 1 commit intodlang:stablefrom
Conversation
This fixes segfault-handler tests for LDC Linux CI, apparently caused by an insufficient alternate-stack size. Claude was able to diagnose the problem: ldc-developers/ldc#5041 (comment)
|
Thanks for your pull request and interest in making D better, @kinke! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub run digger -- build "stable + dmd#22473" |
| _SC_THREAD_ROBUST_PRIO_INHERIT, | ||
| _SC_THREAD_ROBUST_PRIO_PROTECT, | ||
| _SC_MINSIGSTKSZ, | ||
| _SC_SIGSTKSZ, |
There was a problem hiding this comment.
These come from my /usr/lib/x86_64-linux-gnu/bits/confname.h, glibc v2.39.
|
Oh come on: I just upgraded macos. |
|
Ah huh, master is all up to date, its just stable that is broken. Heyyy @dkorpel! Stable needs some PR's from master. |
|
Yeah, CI fixes would ideally always target |
|
Let's wait on Dennis to decide. If he doesn't come back in the next day or so with a decision, I'll override the CI and merge. All the linux runners are green, and this shouldn't touch any other OS. |
|
Sorry, decide on what? |
|
Keep in mind I'm not a PR manager, so I'm acting as a backup. |
|
Cheers Dennis! |
This fixes segfault-handler tests for LDC Linux CI, apparently caused by an insufficient alternate-stack size. Claude was able to diagnose the problem: ldc-developers/ldc#5041 (comment)