Marc R. Schoolderman
Marc R. Schoolderman
The PR in #854 didn't strike me as particular elegant; and if we move to different OS's we might probably need different kernel checks as well. So this cleans up...
These occur in three places: `system` and `exec`. There has been discussion about these in the past so special attention is needed.
Do a review of places where `//SAFETY` comments are missing.
This is a backport from rust-training material.
**What is the scope of your suggestion?** A new Rust Edition will be coming out next month, and it has some changes (e.g. TIL that 'object safety' is now called...
Several compliance tests have something like ``` assert_not_contains!(stdout, "[sudo] password for"); ``` And obviously sudo-rs output does not contain that. We should probably rewrite that or remove those asserts.
See PR #1073; this requires rust 1.77. At the time of writing we could only bump or MSRV to 1.75 since Ubuntu LTS is on that.