#9841 closed defect (fixed)
SCI: KQ6: No credits music
Reported by: | dafioram | Owned by: | sluicebox |
---|---|---|---|
Priority: | normal | Component: | Engine: SCI |
Version: | Keywords: | ||
Cc: | Game: | King's Quest 6 |
Description
ScummVM: 1.10.0git-3379-g8c31328
Tester OS: Win7-64
Game: KQ6 1.000.00G (Kings quest collection) English
After wedding Cassima when the screen says "thank you for playing" it should play some music during the credits and it is just silent.
The closets save I can get is right before attacking Alhazred. So you will have to click through to get through the wedding.
This is the short ending.
Attachments (3)
Change History (10)
by , 7 years ago
Attachment: | FailTOC.png added |
---|
by , 7 years ago
Attachment: | kq6-cd-win.077 added |
---|
Save during Alhazred final battle. Also my game is cd version no floppy.
comment:1 by , 7 years ago
Depending on subtitle/audio settings, the game will try to play CD-Audio.
In case you have not ripped the audio track (Girl in the tower), ScummVM won't be able to play that music.
I'm not sure, if ScummVM is capable of playing the Audio-CD directly.
comment:2 by , 7 years ago
I was unable to find any audio tracks on disc 1 or 2 of the kings quest collection discs. I used windows media player and cdex. They only found data tracks.
First I tried both discs in (separately) while I played the ending and instead of giving me the error it gave me a no track 1 found.
comment:3 by , 7 years ago
Then I put a misc track01.ogg into the folder and it played the audio, but still gives the "Failed to query the CD TOC: 21" in the status window (see FailTOC.png). So as long as i can get the track it works and now it just reports a disc query error.
Maybe it shouldn't query the disc if it finds the track in the folder? Any idea how many tracks it is looking for? Do you know if the GOG version has the audio tracks?
comment:4 by , 7 years ago
The track is supposed to be called track1.mp3 / .ogg
And I'm not sure if ogg support is built into daily builds, probably it is, but I'm not sure.
comment:5 by , 7 years ago
Keywords: | sci32 removed |
---|
comment:6 by , 7 years ago
Owner: | set to |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
Then I put a misc track01.ogg into the folder and it played the audio, but still gives the "Failed to query the CD TOC: 21" in the status window (see FailTOC.png). So as long as i can get the track it works and now it just reports a disc query error.
This is a debugging message, not a warning/error. You may ignore it.
Maybe it shouldn't query the disc if it finds the track in the folder?
Since Win32AudioCDManager::play
prefers emulated CD (and so does the MacOSXAudioCDManager), the CD manager code could be changed so that AudioCDManager tries to open the real CD itself in play
after emulation fails, instead of right now where users of AudioCDManager must call open
themselves before they can call play
(and then the result of that open
is ignored anyway if emulated files are available). I do not know the reason why open
is its own public API. (If it is because the engine is responsible for cleaning up also with close
, then I guess SCI is leaking resources right now when CD audio tries to be played…)
Any idea how many tracks it is looking for?
It is not looking for more than one track; querying the CD TOC tells the manager how many tracks are available on the CD.
Do you know if the GOG version has the audio tracks?
The track is supposed to be called track1.mp3 / .ogg
Tracks with leading zeroes are supported just fine.
And I'm not sure if ogg support is built into daily builds, probably it is, but I'm not sure.
Yes, it is. (Per the earlier comment, audio from a random track01.ogg played successfully.)
So I will close this ticket now since the root problem is that KQ Collection doesn’t include the CD audio track. If someone wants to have the audio CD manager changed to avoid loading CDs unnecessarily as described above, please go ahead and open a new ticket for that.
comment:7 by , 3 years ago
Owner: | changed from | to
---|---|
Resolution: | wontfix → fixed |
Fixed this a year and a half ago in 3c71df687b462bcceba3c745ef0aaabd3eff322f and f9287b26c8ee453f774ecb556460d46456e9ac49
Status Window (Its the Failed to query...)