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

tta CUESHEET support not working #56

Open
ghost opened this issue Oct 7, 2015 · 3 comments
Open

tta CUESHEET support not working #56

ghost opened this issue Oct 7, 2015 · 3 comments

Comments

@ghost
Copy link

ghost commented Oct 7, 2015

I have only found one software so far that can produce whole cd rips with TTA. Python audio tools looks for this in the 'Cuesheet' while CUERipper embeds it within the 'CUESHEET' tag. Please fix the tag name. Or is there actually software that embeds it within the tag you are using now?

@ghost ghost changed the title tta CUESHEET support tta CUESHEET support not working Oct 7, 2015
@tuffy
Copy link
Owner

tuffy commented Oct 7, 2015

The cuesheet tag isn't in the APEv2 or True Audio specifications at all, so it's going to have to be some sort of de-facto standard. But WavPack uses APEv2 like TTA and it explicitly uses "Cuesheet" for the key name in its specification. So it makes sense for two lossless formats using the same tagging format to use a non-standard tag like "Cuesheet" consistently. Does CUERipper also use "CUESHEET" when embedding cuesheets in WavPack files? If so, it may be a CUERipper bug.

"CUESHEET" itself is a strange choice considering how all the other APEv2 tags are capitalized. But I might be able devise an alias for it if necessary.

@ghost
Copy link
Author

ghost commented Oct 7, 2015

Yes, it uses "CUESHEET" for both TTA and WV files. It appears to use the name "CUESHEET" for everything. FLAC, TTA, WV, APE, etc. Perhaps you could add a fallback to use "CUESHEET" if "Cuesheet" is not found?

@tuffy
Copy link
Owner

tuffy commented Oct 7, 2015

That sounds good. I've pushed out a fix which treats "CUESHEET" as an alias for "Cuesheet" in APEv2 tags. Hopefully that'll take care of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant