Opened 12 months ago

Last modified 11 months ago

#9677 new enhancement

SCI: GK2: Subtitle patch doesn't work

Reported by: IntEmu Owned by:
Priority: normal Component: Engine: SCI
Keywords: sci32 Cc:
Game: Gabriel Knight 2

Description

Hi, I can't get this custom subtitle patch to work in ScummVM.
http://www.sierrahelp.com/Patches-Updates/Patches-Updates-Games/GK2SubtitlePatch.html

I installed GK2 for ScummVM by following the ScummVM Datafiles wiki and then manually installed the subtitle patch following the instructions in link below. Then I added 'patchDir = .\SUBPATCH' to RESOURCE.CFG/RESOURCE.WIN.
http://www.gog.com/forum/gabriel_knight_series/q_does_gabriel_knight_2_the_beast_within_feature_subtitles

Also, see this thread, http://forums.scummvm.org/viewtopic.php?t=6873. Thank you.

Change History (11)

comment:1 Changed 12 months ago by m-kiewitz

ScummVM does NOT use resource.cfg nor resource.win. Both of those are configuration files for the original interpreters.

You could just create a sub-directory called "patches" and put the patch files in there. Or you could just copy them into the main game directory.

Please keep in mind, that we will implement basically the same feature as well (minus the additional subtitles probably), so you won't need all of those files later. And this will work directly and on-the-fly.

Our patch will also support localized versions of the game. And it will also work on different platform versions.

comment:2 Changed 12 months ago by IntEmu

Hi, renamed '.\SUBPATCH' to '.\PATCHES' and after the 'Chapter One' opening movie ScummVM crashes.

ERROR: Uninitialized read for temp 1 from method ::export 10 (room 34, script 11, loadCall ffffffff)!

Also, using Version 1.0 of the game and please note the author of the patch suggests this patch only works with the Windows version. Thank you.

comment:3 Changed 12 months ago by m-kiewitz

I will have to look into the script patches.

It seems there is at least one bug inside that fan patch - or maybe a sierra script was modified in a way, that our workarounds do not apply anymore.

ScummVM is checking for quite a few issues like this uninitialized variable error. The original interpreter didn't do that, but such issues may cause other issues down the line.

comment:4 Changed 12 months ago by IntEmu

Hi, when will ScummVM support GK2 subtitles? How are the subtitles stored? Thank you.

comment:5 Changed 12 months ago by IntEmu

I mean, you said ScummWM "will implement basically the same feature as well (minus the additional subtitles probably)", when will this feature be available and what do you mean by "minus the additional subtitles"? Thank you.

comment:6 Changed 12 months ago by m-kiewitz

I can't give you a specific date.
I'm currently trying to figure everything out.

The current subtitle patch is ONLY able to show subtitles and do audio at the same time. You can't switch the subtitles off, at least to my knowledge. And it also seems impossible to run the game in text-only mode.

I'm currently trying to do exactly that. Support text-only. Audio-only and audio+subtitles mode.

But it is complicated and I can't just use the code within the patched scripts.

What I meant with "minus the additional subtitles", is that video subtitles do not exist within the game, but those were added manually by the 2 "hackers" with that patch.

My plan is that you can still add those video subtitles by copying over the .msg files, but we won't patch the game that way automatically, because we would have to add all of those video subtitles inside ScummVM. But you shouldn't need to patch any scripts.

I also already found out that tons of those .scr/.hep files are not needed in any case and those are just 1:1 copies of the scripts used within the game.

I will hopefully also fix the missing inventory audio.

comment:7 Changed 12 months ago by IntEmu

Hi, thanks for the information, I'm interested in audio+subtitles and can you try to get the patch to work? Thank you.

comment:8 Changed 12 months ago by wjp

Just to be clear, we don't officially support GK2 at all yet, let alone any additional subtitle patches. Please be patient.

comment:9 Changed 11 months ago by IntEmu

Hi, with v1.1 of the game and recent daily build, subtitles via ScummVM (rather than the patch) work. Thank you.

comment:10 Changed 11 months ago by m-kiewitz

I don't see any commit that could have solved it.
Maybe you just got a release-build, which disables all sorts of hard script errors.

The subtitle patch scripts introduce at least one script bug, and that will be found by ScummVM all the time. I also do not want to add a workaround for it and instead patch the scripts properly, so that the user is able to enable/disable subtitles however he/she wants to.

I would also like to keep saved game compatibility. It seems when using the original patched subtitle scripts, old saved games also do not work anymore (at least when using ScummVM).

Please keep bug open for now.

comment:11 Changed 11 months ago by IntEmu

The subtitle patch on v1.1 of the game still crashes the game and I was mistaken, there are subtitles for audio voice overs if subtitles is selected in ScummVM settings. There are no subtitles for the beginning/end chapter "Movies" nor is there subtitles for in-chapter movies.

Note: See TracTickets for help on using tickets.