-
Notifications
You must be signed in to change notification settings - Fork 757
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
Improve error handling in Server:prepareForRecord #1580
Comments
Why is this even a bug? I think it would be rather extraordinary to expect the server, or even the language, to create directories automatically. Sorry, but I think this should just be closed. |
Really, well you obviously have some technical insight, I am missing. I naively assume it should be possible to create any path you can write and you have write access to. If it's really absurd, please close. I am on the road. |
What I mean is: Standard behavior, in any programming language, when you try to open a file for writing and the location doesn't exist, is to fail to open the file. One rationale is that you wouldn't want typos to create spurious directories in the file system. In Linux, scsynth posts an error in this case that the path couldn't be opened. To me, this is the expected behavior. If the error doesn't appear in Windows, that would be a bug. |
Well, you get an error, when you run prepareForRecord as an individual command. But in a series of commands the message won't turn up and recording just writes nothing. But it's certainly not the type of bug I believed it to be... |
Can you remove the tags for me. I am on a mobile. |
I'll mark this as an enhancement for future (not 3.7). It's a legitimate problem that the language gives no sort of result to the user that the file-open failed. It can do so (see http://doc.sccode.org/Reference/Server-Command-Reference.html#/fail):
|
Shouldn't be done until #1581 is fixed. |
I'll add it in #1547, which will go into 3.8 |
Addressed in #1655 |
Windows 8.1, SC alpha1, VS build, 64 bit
Files via
s.prepareForRecord("C:\Users\name\Documents\SuperCollider\test\test.wav")
or
s.prepareForRecord("C:/Users/name/Documents/SuperCollider/test/test.wav")
can only be created, if all folders in a path already exist on the system. In any other case file-creation fails.
The text was updated successfully, but these errors were encountered: