refactor: replace subprocess crate with std::process

[Integral-Tech]

Fixes issue #695

[Integral-Tech]

@bee-san I have just fixed the failed workflow in my latest commit, please retrigger it :)

[bee-san]

error[E0599]: no method named `signal` found for struct `ExitStatus` in the current scope
22
   --> src\scripts\mod.rs:294:46
23
    |
24
294 |                 None if cfg!(unix) => status.signal().unwrap(),
25
    |                                              ^^^^^^ method not found in `ExitStatus`

Windows doesn't like this? :(

[Integral-Tech]

error[E0599]: no method named `signal` found for struct `ExitStatus` in the current scope
22
   --> src\scripts\mod.rs:294:46
23
    |
24
294 |                 None if cfg!(unix) => status.signal().unwrap(),
25
    |                                              ^^^^^^ method not found in `ExitStatus`

Windows doesn't like this? :(

Yes, the signal() method is Unix only

[Integral-Tech]

@bee-san I just switched to another method of conditional compilation in my latest commit :)

[bee-san]

thanks so much!!

[bee-san]

Oh I forgot to say! Please don't force push in the future, it is slightly annoying to work out what's changed per commit 😅 (When you make a change to fix something, I go to your latest commit to see what the fix was. Because it was force-pushed, I do not know what specifically changed so I have to read the full change again)

Thank you so much for your contribution!!