Opened 7 years ago

Closed 7 years ago

#6111 closed defect (fixed)

KYRA: LOL Floppy FR version unknown

Reported by: misterhands Owned by: athrxx
Priority: normal Component: Engine: Kyra
Keywords: detection Cc:
Game: Lands of Lore

Description

SCUMMVM 1.5 win32
LOL Floppy FR

The game is not recognized at all by SCUMMVM (could not find any game in the specified directory).
Here is the md5 data:
{"INTROVOC.PAK", 0, "baf2e3dd341e38784d9204c9a5c38033", 1890471},
{"VOC.PAK", 0, "19523182806a7998e087ba4cfb106215", 2276340},
{"INTRO.PAK", 0, "6fc916e9fb55b5699e6a94b36193dbc8", 941898},
{"CHAPTER7.PAK", 0, "733e33c8444c93843dac3b683c283eaa", 325124},
{"GENERAL.PAK", 0, "f4fd14f244bd7c7fa08d026fafe44cc5", 1395249},

Ticket imported from: #3552534. Ticket imported from: bugs/6111.

Change History (7)

comment:1 Changed 7 years ago by lordhoto

It would be nice if you could supply the detection entries for the installer files too. Just point ScummVM at a directory containing all the contents of the floppy files for that. It should result in an output for "WESTWOOD.1" then.

Currently no developer owns a french floppy version as far as I know, thus adding support for it will take some time.

Assigning this to athrxx.

comment:2 Changed 7 years ago by lordhoto

Owner: set to athrxx
Summary: Kyra: LOL Floppy FR version unknownKYRA: LOL Floppy FR version unknown

comment:3 Changed 7 years ago by misterhands

Well I've got no floppy drive in my computer anymore, those big IDE ribbon connectors on motherboards are now ancient history...
Fortunately, I just bought a new shiny USB floppy drive to make up for that ! :-)

Here you go :
{"WESTWOOD.1", 0, "43857e24d1fc6731f3b13d9ed6db8c3a", 1353217},

I should add that the md5sums given in the bug report are those of the v1.23 patched version. A fresh install from the floppies is v1.20 and has different md5 sums.
Perhaps these should be added as well ?

As far as adding support for this version, isn't it as simple as adding the md5sums in the engine detection table as usual ?

comment:4 Changed 7 years ago by athrxx

I have added your md5 data to the code. I cannot test this, however (I don't have a French floppy version myself). So I'll have to ask you to test this and confirm whether it works for you. Please test both the extracted version and the installer files if it isn't too much of a bother.
For testing you'll have to check out the latest code and compile yourself or download the latest nightly build. Changes should be picked up tomorrow or the day after.

comment:5 Changed 7 years ago by athrxx

Resolution: fixed
Status: newpending

comment:6 Changed 7 years ago by misterhands

It works, for both the installer files and the extracted version.

Thank you !

comment:7 Changed 7 years ago by lordhoto

Status: pendingclosed
Note: See TracTickets for help on using tickets.