Opened 4 years ago
Closed 4 years ago
#12034 closed defect (fixed)
SCI: LSL1: French fan patch version not detected
Reported by: | MADevil666 | Owned by: | sluicebox |
---|---|---|---|
Priority: | normal | Component: | Engine: SCI |
Version: | Keywords: | detection | |
Cc: | Game: | Leisure Suit Larry 1 |
Description
The game in 'Leisure Suit Larry in the Land of the Lounge Lizards (Floppy DOS VGA Remake)\' seems to be an unknown game variant.
Please report the following data to the ScummVM team at https://bugs.scummvm.org/ along with the name of the game you tried to add and its version, language, etc.:
Matched game IDs for the sci engine: lsl1sci
{"resource.002", 0, "3fe2a3aec0ed53c7d6db1845a67e3aa2", 1095908},
{"resource.003", 0, "ac175df0ea9a2cba57f0248651856d27", 376556},
{"resource.000", 0, "5e501a9bf8c753bf4c96158042422f00", 1938733},
{"resource.001", 0, "112648995dbc194037f1e4ed2e195910", 1063341},
{"resource.map", 0, "bbf22cacfcf6ef7e7578045fafad805d", 3337},
Change History (4)
comment:1 by , 4 years ago
comment:2 by , 4 years ago
Component: | --Other-- → Engine: SCI |
---|---|
Keywords: | detection added; LSL1 FR PATCH removed |
Summary: | Leisure Suit Larry 1 with french fan patch → SCI: LSL1: French fan patch version not detected |
Type: | patch → defect |
comment:3 by , 4 years ago
Thanks for reporting this. It's going to take some work before we can add this though.
First, this patch is distributed as three installers: 2.0A, 2.0B, and 2.1. The installers modify the target LSL1VGA resource volumes instead of adding SCI patch files, so this is would need to be more than just one detection entry for whichever version this is.
The bigger problem is that even if we add these entries, they won't work. If these patches are run as French under ScummVM then they will crash almost immediately with VIEW 1051 NOT FOUND. These patches took the Spanish version and modified its resources, so it only works when run as Spanish. We would have to do something to hack around that, and it's a little more complicated since this is a dual language game that can switch to English.
At the script level, LSL1:printLang must only equal English or Spanish for these patches to run.
comment:4 by , 4 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Fixed in: https://github.com/scummvm/scummvm/commit/21b946645e4488f9dbad9ae655e0fcf4575f2391
I added the 2.1 hashes, which are the ones you provided, and a workaround so that the game works even though it needs to think that it's Spanish. French text and pictures work, and since this version includes English, you can also toggle between the languages.
This change will appear in the next daily build: https://www.scummvm.org/downloads/
Thanks again!
Patch from this page : https://www.obsolete-tears.com/forum/viewtopic.php?t=8726