Problem you are trying to solve
The Toolchain::set_env() function is a providing state sync upon cargo plugin calling rustup recursively.
However, when looking at the tests related to this mechanism, I found no direct test that sets RUST_RECURSION_COUNT to 20/21 and asserts either guard’s error. Thus normal nested recursion has integration coverage; the maximum-count termination behavior does not appear to have explicit coverage.
After I discussed with @rami3l, I think this should be tracked here.
Solution you'd like
Adding tests specifically triggering the maxium recursion crash, preventing further problems.
Notes
No response
Problem you are trying to solve
The
Toolchain::set_env()function is a providing state sync upon cargo plugin calling rustup recursively.However, when looking at the tests related to this mechanism, I found no direct test that sets
RUST_RECURSION_COUNTto 20/21 and asserts either guard’s error. Thus normal nested recursion has integration coverage; the maximum-count termination behavior does not appear to have explicit coverage.After I discussed with @rami3l, I think this should be tracked here.
Solution you'd like
Adding tests specifically triggering the maxium recursion crash, preventing further problems.
Notes
No response