Christian Poveda Ruiz
Christian Poveda Ruiz
> > @jbaublitz @ojeda the discussion we've had so far here makes me think that we need to figure out a better way to integrate this feature as it has...
My first impression is that this shouldn't be too difficult to do as we already do some attribute detection for other things. Maybe the hardest part would be to set...
Hmm... I see this existing as a `Builder` method. However the CLI argument discussion would be more tricky as you want to do two selections: - First pick the enum...
I'd lean to use an std function instead (I"m not sure if https://doc.rust-lang.org/std/env/fn.temp_dir.html does what we want). Adding a dependency that has `unsafe` code for a single function feels like...
yeah we would have to come up with a random name for the directory ourselves and I'm not sure what would be the proper requirements for that. Maybe we could...
the issue with `temp_dir` is that it gives us the path to the temporary directory (`/tmp/` in linux) but `mktemp` creates one random directory under the temporary directory. So we...
The advantage of just moving it to `system` is that we have the peace of mind that `sudo` has been using the same function without issues for years.
Yes, I agree that using the same function as `sudo` is no guarantee of absolute safety. But sharing that function means that if either of us gets a vulnerability related...
I don't have a way to debug this as I don't have a windows machine but the [`clang-sys` docs](https://github.com/KyleMayes/clang-sys?tab=readme-ov-file#dynamic) mentions something about windows: > On Windows, running an executable that...