Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows path names need fix in dev build #6241

Open
sadguitarius opened this issue Mar 20, 2024 · 0 comments
Open

Windows path names need fix in dev build #6241

sadguitarius opened this issue Mar 20, 2024 · 0 comments
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs.

Comments

@sadguitarius
Copy link

Environment

  • SuperCollider version: 3.14.0-dev
  • Operating system: Windows 11
  • Other details (Qt version, audio driver, etc.): 5.15.2

Steps to reproduce

s.boot;

Expected vs. actual behavior

When booting the server from the command line in the build folder, this error appears:

Booting server 'localhost' on address 127.0.0.1:57110.
'C:\Users\sadguitarius\dev\source\supercollider_build\supercollider\build\Install\SuperCollider\scsynth.exe" -u 57110 -a 1024 -i 2 -o 2 -S 48000 -H
"" "ASIO' is not recognized as an internal or external command,
operable program or batch file.
Server 'localhost' exited with exit code 1.

When starting from the installation folder, I get:

'C:\Program' is not recognized as an internal or external command,
operable program or batch file.
Server 'localhost' exited with exit code 1.

Apparently there's been some back and forth over the last few years about how to deal with Windows path names in server commands, especially re: quoting and spaces. I see that #5990 was an attempt to fix this and I followed the discussion in #2719. At least for me, I cannot boot the server in either ScIDE, sclang.exe on the command line, or SCNvim whenever there is a space in the runtime path or in the name of the selected sound device.

Changing the default command to Server.program = "start /B /HIGH /D % scsynth.exe".format(Platform.resourceDir.quote); seems to work fine in all of these scenarios. Unfortunately, I haven't yet figured out why the dev build doesn't work with the default settings and the release build does, but are there any objections to going ahead and changing the default command?

@sadguitarius sadguitarius added the bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs. label Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs.
Projects
None yet
Development

No branches or pull requests

1 participant