Opened 12 years ago
Closed 12 years ago
#6056 closed defect (fixed)
SCI: KQ5 Floppy English detected as Polish
| Reported by: | misterhands | Owned by: | bluegr |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCI |
| Version: | Keywords: | detection | |
| Cc: | Game: | King's Quest 5 |
Description
SCUMMVM 1.5git win32 KQ V version is 0.000.051 and that's the vanilla floppy English release. No Polish inside. I see there is another 0.000.051 version in the detection tables... must be a variant.
The detection entries have been originally reported in bug #2725722. My guess is that the guy made a mistake and posted the English md5 instead of the Polish ones, as it seems unlikely that the English and Polish versions would have the same exact file sizes and md5sums.
Ticket imported from: #3536863. Ticket imported from: bugs/6056.
Attachments (1)
Change History (16)
comment:1 by , 12 years ago
comment:2 by , 12 years ago
| Summary: | SCI :KQV Floppy English detected as Polish → SCI: KQ5 Floppy English detected as Polish |
|---|
comment:3 by , 12 years ago
Yeah it could be a translation thing, meaning that the md5sums are indeed from the English Floppy release, and the Polish version adds translation patch files on top. Given that the file sizes are identical, that's the most likely.
Here's my md5 file.
by , 12 years ago
comment:4 by , 12 years ago
misterhands: Thank you for providing the request information. I have also asked for the same information for the Polish variant via bug #2725722, to be attached here, so when that occurs, we should have sufficient information to amend the detection entries to prevent this detection collision. If we receive no response on this, we will look at amending or disabling the Polish detection entry anyway, but this could break Polish detection.
comment:6 by , 12 years ago
| Priority: | normal → blocker |
|---|
comment:7 by , 12 years ago
Have checked in the engines/sci/detection_tables.h. This is not a case of duplicate MD5sums, but that the Polish entry is marked as dual language with English. The relevant entry is: // King's Quest 5 - Polish DOS Floppy (supplied by jacek909 in bug report #2725722, includes english language?!) // SCI interpreter version 1.000.060 {"kq5", "", { {"resource.map", 0, "70010c20138541f89013bb5e1b30f16a", 7998}, {"resource.000", 0, "a591bd4b879fc832b8095c0b3befe9e2", 276398}, {"resource.001", 0, "c0f48d4a7ebeaa6aa074fc98d77423e9", 1018560}, {"resource.002", 0, "7f188a95acdb60bbe32a8379ba299393", 1307048}, {"resource.003", 0, "0860785af59518b94d54718dddcd6907", 1348500}, {"resource.004", 0, "c4745dd1e261c22daa6477961d08bf6c", 1239887}, {"resource.005", 0, "6556ff8e7c4d1acf6a78aea154daa76c", 1287869}, {"resource.006", 0, "da82e4beb744731d0a151f1d4922fafa", 1170456}, {"resource.007", 0, "431def14ca29cdb5e6a5e84d3f38f679", 1240176}, AD_LISTEND}, Common::PL_POL, Common::kPlatformPC, ADGF_ADDENGLISH, GUIO4(GUIO_NOSPEECH, GAMEOPTION_PREFER_DIGITAL_SFX, GAMEOPTION_ORIGINAL_SAVELOAD, GAMEOPTION_FB01_MIDI) },
I think misterhands is indicating that this version is not dual language, but English only. Not sure if the Polish language version was supplied as Patch files on top of this base version? Hopefully one of the SCI developers has the same version and can clarify.
comment:8 by , 12 years ago
| Priority: | blocker → normal |
|---|
comment:9 by , 12 years ago
So it seems that the Polish version is incomplete. I've added this version in the detection entries, but we will need a better way of distinguishing the two versions properly.
misterhands: could you please provide a checksum of all of your game files?
Here's information on how to do this: http://wiki.scummvm.org/index.php/Reporting_unknown_MD5_checksums
I'm lowering priority of this one, and I'm reopening the other bug report
comment:10 by , 12 years ago
thebluegr: well I did already as requested by tdhs, see the attached md5sums file.
comment:11 by , 12 years ago
misterhands: The md5sums you did are for the full file, which allows us to be sure of your exact version, but our detector only calculates the md5sum checksum of the first <n> bytes of a file, so the values differ. The link explains how to manually get these "restricted" checksums: http://wiki.scummvm.org/index.php/Reporting_unknown_MD5_checksums
In case you wonder, the reason we don't calculate the checksum over the entire file is that doing this for 100MB+ resource files would slow detection down to painfully slow degree.
comment:12 by , 12 years ago
thebluegr: ok but this wiki page does not specify the value of <n> for SCI... should I assume it's 5000 bytes like most engines ?
comment:14 by , 12 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:15 by , 12 years ago
Fixed in r2c5cf9f (master) and r3af8661 (branch 1.5), via MD5 checksums from the other bug report.
Closing.
The bug referred to is "SCI: Kings Quest V PC Polish version md5's - ID: 2725722" : http://sourceforge.net/tracker/?func=detail&aid=2725722&group_id=37116&atid=418820
This provided md5sums for the resource.nnn and resource.map files, but the official translation may exist only as patch files or may only result in changes to these files after the 5000 byte detection limit.
misterhands: Could you please attach a textfile to this bug with a file listing of your KQ5 Floppy English datafiles along with full file md5sums? The output of a tool such as http://md5summer.org/ would be optimal.