-
Notifications
You must be signed in to change notification settings - Fork 339
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
Allow cuesheet FILE case-insensitivity #1705
Comments
What does that mean? MPD supports standards-compliant CUE sheets, and they mandate that FILE is all-uppercase. CUE sheets that use lower case are not standards compliant and thus ill-formed, and will not be supported by MPD. |
Do you mean the Over here, I'm experiencing different behavior for .cue files that are embedded in a ZIP file. This doesn't work:
First lines of Foo.cue:
This works:
First lines of Bar.cue:
VersionMPD version: 0.23.11
LogAfter issuing
ConfigurationSnippets below are added to mpd.conf through playlist.conf
resamplers.conf
database.conf
|
What did you mean? Your feature request was ambiguous, but apparently ou meant the value (the actual file name), not the directive. But I don't think MPD should support that; that would require scanning the whole directory each time a CUE sheet gets loaded. Instead of doing that, you can better switch the Linux kernel to be case insensitive. |
I've looked a bit into this and unfortunately only some of the filesystems in Linux can be made case-insensitive (NFS isn't one of them). |
Feature request
Allow cuesheet FILE directive to be case-insensitive, to interoperate with cuesheets that were originally created on case-insensitive systems.
For consistency, FILE should first match with case-sensitivity and then fallback to non case-sensitive match. This feature could be made into a mpd.conf setting under the
cue
playlist plugin to give some control over whether it should be enabled or not.The text was updated successfully, but these errors were encountered: