Handle quotes in PATH

[segiddins]

Fixes #4323 and adds tests

[indirect]

If I'm understanding this correctly, this change does not handle entries that use quotes around the path separator, which is the specific case that quotes are required for. For example, PATH=C:\ruby\bin;"C:\stuff;other stuff\bin".

[segiddins]

@indirect correct, but it fixes the issue that was reported ;)

[indirect]

@segiddins so now we're just knowingly waiting for the first person who has a path containing File::PATH_SEPARATOR, because it will be completely broken for them. 😂 😭

[segiddins]

@indirect but no more broken than it is now. this is an improvement without regression. feel free to add a disabled test for the other scenario or hold up this PR until someone contributes that other half

[indirect]

@segiddins yeah, yeah. I'm 👍 on this once the tests pass.

[segiddins]

Ought to pass now

[segiddins]

@homu r+

[homu]

📌 Commit bd0f564 has been approved by segiddins

[homu]

⚡ Test exempted - status