Opened 14 months ago
Closed 6 months ago
#13283 closed defect (fixed)
KYRA: Legend of Kyrandia (CD/DOS/English) also matches (Demo/DOS/English)
Reported by: | i30817 | Owned by: | athrxx |
---|---|---|---|
Priority: | normal | Component: | Engine: Kyra |
Version: | Keywords: | detection | |
Cc: | Game: | Kyrandia 1 |
Description (last modified by )
I feel like this could be a scanner bug or it could be intentional if the implementation of a 'demo' ends up in the executable and all the data is the same. I haven't had time to play through the 'demo'.
[kyra1-cd] platform=pc gameid=kyra1 description=The Legend of Kyrandia (CD/DOS/English) language=en extra=CD path=/media/i3/Mordred/Games/ScummVM/legend_of_kyrandia_1 engineid=kyra guioptions=midiPCSpk midiAdLib midiMt32 vga lang_English [kyra1-demo-cd] platform=pc gameid=kyra1 description=The Legend of Kyrandia (Demo/CD/DOS/English) language=en extra=Demo/CD path=/media/i3/Mordred/Games/ScummVM/legend_of_kyrandia_1 engineid=kyra guioptions=midiPCSpk midiAdLib midiMt32 vga lang_English
Change History (17)
comment:1 by , 14 months ago
Keywords: | Launcher added |
---|
comment:2 by , 14 months ago
Description: | modified (diff) |
---|
comment:3 by , 14 months ago
Thanks for your report. I don't quite understand your last comment. Is the bug report still valid?
comment:4 by , 14 months ago
Yes, but i had a additional comment about 'not finding the kyra.dat file'. That was bogus and edited out.
comment:5 by , 14 months ago
Summary: | Legend of Kyrandia (CD/DOS/English) also adds (Demo/DOS/English) → Legend of Kyrandia (CD/DOS/English) also adds (Demo/DOS/English) in mass add |
---|
comment:6 by , 11 months ago
Component: | --Unset-- → Engine: Kyra |
---|---|
Game: | → Kyrandia 1 |
Keywords: | detection added; Launcher removed |
Summary: | Legend of Kyrandia (CD/DOS/English) also adds (Demo/DOS/English) in mass add → KYRA: Legend of Kyrandia (CD/DOS/English) also adds (Demo/DOS/English) in mass add |
comment:7 by , 7 months ago
Could you please check whether you find this particular bug still valid? There have been improvements on the mass add function, but I am not sure if they address this one.
comment:9 by , 6 months ago
I think it's a bit weird the game and the 'demo' being on the same cd, but i guess it's possible it's just a property of the engine where all the data of the demo is in the game and it chose depending on the executable... which is not there, because i deleted unneeded files. It's not a big deal in this game i guess.
comment:10 by , 6 months ago
athrxx: Close this as WONTFIX? Or does this need a patch to KYRA detection?
comment:11 by , 6 months ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → pending |
comment:12 by , 6 months ago
I would assume that it requires further fixing of the mass add function.
Unless it also detects the demo when adding the game "normally". Then it would probably be something that can be fixed in the Kyra detection.
comment:13 by , 6 months ago
The culprit here is the actual kyra detection, not the mass add.
There is an entry called "Special Kyrandia 1 CD demo" that uses the file INTRO.VRM for detection which is also present on the full cd release. That's why scummvm finds two entries (and it happens also with the standard add, I tested the GOG release).
We could add an additional file for detection, but unfortunately I have no idea where to find this demo.
comment:14 by , 6 months ago
Resolution: | fixed |
---|---|
Status: | pending → new |
Summary: | KYRA: Legend of Kyrandia (CD/DOS/English) also adds (Demo/DOS/English) in mass add → KYRA: Legend of Kyrandia (CD/DOS/English) also matches (Demo/DOS/English) |
Changing title and reopening
comment:15 by , 6 months ago
I have found the demo (on my Lands of Lore CD) and made a fix that seems to resolve the ambiguity issue with the detection.
comment:17 by , 6 months ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Turns out i hadn't yet configured the extra path, i forgot about it.