search

Novum / vkQuake

Vulkan Quake port based on QuakeSpasm
GNU General Public License v2.0
1.84k stars 223 forks source link

No support for CD Audio? #80

Closed Svetomech closed 7 years ago

Svetomech commented 7 years ago

CDAudio disabled at compile time Do I have to compile vkQuake manually to enable CD Audio support?

mdeguzis commented 7 years ago

Did you try adding the audio tracks per the readme? Or is this just a message you noticed and are concerned about?

Svetomech commented 7 years ago

I tried .mp3 and .ogg formats - no noticeable changes. Music should begin when a level starts, correct? Never actually played the game before.

mdeguzis commented 7 years ago

Are you sure you installed all the dependencies? My build logs do show the text "SDL CDAudio support disabled", but this should not affect the tracks ability to play. If you have followed the readme exactly as written for CD Audio, you could try seeing if the files are owned by another user (say root), or are not readable.

Novum commented 7 years ago

SDL2 has no support for CD audio. MP3, Ogg etc. should work like in QuakeSpasm.

Svetomech commented 7 years ago

Are you sure you installed all the dependencies?

I was, but it never hurts to double check. What is the name of the package I should install for vkQuake to read .ogg files? There's only a library name in the readme.

And no, all files are owned by the current user and are perfectly readable (e.g. in a music player).

UPD: @Novum, don't mind me asking, but why are you always so rushy about closing issues? I haven't even finished writing an answer and boom closed.

MP3, Ogg etc. should work like in QuakeSpasm.

But they don't. At least, not for me. I will solve this issue on my own eventually (if it is an issue on my side) like I did with all the other issues that had popped up, but a little help is really appreciated and could possibly save me lots of time.

Novum commented 7 years ago

It works perfectly fine, I just checked. GitHub issues are for actual bugs, not for tech support.