Replies: 2 comments
-
We discussed this in today's Verus meeting and the consensus seemed to favor this change. I might wait to attempt it until after the interpreter branch is merged, since that would be one deterministic way to fix the |
Beta Was this translation helpful? Give feedback.
0 replies
-
The test suite now uses Verus' global default ( |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
If the user does not specify an rlimit via the
VERIFY_EXTRA_ARGS
environment variable, then the args we pass to the verifier are instantiated via adefault()
call:https://github.com/secure-foundations/verus/blob/b44a565193338c90fa50fe2f0620664cf6f26d0f/source/rust_verify/tests/common/mod.rs#L124
This appears to set the
rlimit
to0
, which means a test can potentially run forever. I might suggest that we add something like:I tried this as an experiment, and on the CI machine, it causes
e18_pass
fromrust_verify/tests/summer_school.rs
to fail, so we may want to tweak that test or introduce the ability to locally increase the rlimit before adopting a test-suite-wide rlimit.Beta Was this translation helpful? Give feedback.
All reactions