opendylan
opendylan copied to clipboard
Make run-application API consistent on Windows
In b190938ddf200c926067bc676d2fdaef71542586 I changed run-application on unix system to not require a limited collection of strings because it makes the API more difficult to use. Do the same for Windows.
It looks like run-application on Windows doesn't work with a sequence of strings at all, which makes sense since neither does the win32 CreateProcessA API as far as I can see.
Assuming my reading of the above is correct, we have choices:
- Change
run-applicationon Windows to accept<sequence>and raise a friendly error if a non-<string>argument contain any non-<string>elements. Convert the sequence to a string withjoin(seq, " "). (Let's assume we'll ignore issues of ensuring command arguments are quoted.) - Change
run-applicationon Windows to accept<string>only and document that in order to be cross-platform users should pass a string torun-application.