Fix: ulimit and odin test must be on the same justfile line
just runs each recipe line in its own shell invocation (no shebang on this recipe), so the previous commit's separate ulimit -s 8192 line never actually affected the odin test line - CI segfaulted again the same way. Confirmed fixed in an ubuntu:24.04 container running the actual `just test` recipe with ulimit -s unlimited set beforehand. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
1 parent
06f4e9ed11
commit
f53ec2d71d
1 file changed
+3
-2
@@ -32,8 +32,9 @@ test:
|
||||
# Odin's core:thread passes that straight to pthread_attr_setstacksize, pthread_create
|
||||
# then fails, and thread.create() returns nil - which core:testing's pool_init
|
||||
# dereferences unconditionally, segfaulting the whole test binary. Clamp it.
|
||||
ulimit -s 8192
|
||||
odin test tests -vet -define:ODIN_TEST_THREADS=1 -out:{{build_dir}}/tests
|
||||
# (One line: just runs each recipe line in its own shell, so a separate
|
||||
# `ulimit` line wouldn't carry over to the `odin test` line.)
|
||||
ulimit -s 8192 && odin test tests -vet -define:ODIN_TEST_THREADS=1 -out:{{build_dir}}/tests
|
||||
|
||||
# Optimized native build into build/<os>/, with the assets it needs at runtime
|
||||
release:
|
||||
|
||||
Reference in new issue
Block a user