You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In b190938 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.
The text was updated successfully, but these errors were encountered:
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-application on 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 with join(seq, " "). (Let's assume we'll ignore issues of ensuring command arguments are quoted.)
Change run-application on Windows to accept <string> only and document that in order to be cross-platform users should pass a string to run-application.
cgay
added a commit
to cgay/opendylan
that referenced
this issue
Jun 24, 2023
In b190938 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.The text was updated successfully, but these errors were encountered: