Note: GitHub action verify problem with Java 16-ea is not part of this PR.

(@michael-o: Requested changes have been done)

Note: GitHub action verify problem with Java 16-ea is not part of this PR.

See #116

Is this PR (or #109) not relevant 😢 ?

I will try to pick this up next month.

Thank you !

I will try to pick this up next month.

😁 (#FriendlyBump)

Thanks for the note. I think both Bourne Shell and Cmd Shell must go. There is no use for that.

I think both Bourne Shell and Cmd Shell must go. There is no use for that

=> We abandon this PR, and perhaps #109 depending the relevance ?

I first need to understand why this was done at all. Maybe lack of knowledge. Currently, I don't see a reason why we should wrap these command into a shell execution at all.

From what I've always understood only with a new shell you can isolate system properties and environment variables.

System properties are pure args to a process. There is no connection to the parent process. If want to isolate env vars, you can do that easily with the ProcessBuilder. It works just fine. Moreover, you cannot start with a nude environment because some processes expect default env vars and they won't be present. Just set those you need and that's it.