Opened 5 years ago
Closed 4 years ago
#10981 closed feature request (fixed)
Unknown game variant for The Journeyman Project: Pegasus Prime
Reported by: | Duffadash | Owned by: | sev- |
---|---|---|---|
Priority: | normal | Component: | Engine: Pegasus |
Version: | Keywords: | unknown-game | |
Cc: | Game: | Journeyman Project Pegasus Prime |
Description
The game in 'PP Data' 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 The Journeyman Project: Pegasus Prime engine: pegasus-mac
{"JMP PP Resources", 0, "d13a602d2498010d720a6534f097f88b", 2075337},
This is from a GOG installer
Attachments (1)
Change History (14)
comment:1 by , 5 years ago
Owner: | set to |
---|
follow-up: 3 comment:2 by , 5 years ago
by , 5 years ago
Attachment: | full_patch.diff added |
---|
comment:3 by , 5 years ago
Replying to lotharsm:
Can you confirm the integrity of the data/files contained in the GoG installer? I wonder why the game is not detected since according to https://www.gog.com/game/journeyman_project_1_pegasus_prime_the the GoG release is already powered by ScummVM.
The game launches and plays fine when using the executable. It is indeed using ScummVM, but a modified copy. Do note that this copy of Pegasus Prime is from an old (2014/2015'ish) installer that I no longer have lying around.
Here's the info from the provided "./docs/ScummVM Documentation/Windows/Special Build Info.txt" file:
This project uses a custom build of ScummVM licensed under GNU GPLv2 using revision d04667e58a889e4744f36fb5797a69fa437a1abf. You may obtain the sources from GitHub: https://github.com/scummvm/scummvm (direct link: https://github.com/scummvm/scummvm/archive/d04667e58a889e4744f36fb5797a69fa437a1abf.zip). On top of that revision, it has been patched with the "full_patch.diff" (included in this directory's parent) The entire source directory is also included in "Source" in the parent directory, including the dists/pegasus directory missing from the patch. This is a cutdown ScummVM build because Pegasus Prime does not need many of the usual features of ScummVM.
I've attached a copy of the full_patch.diff file to the post for good measure.
Also I notice with some curiosity that the provided Read Me lists Linux as a supported OS, which no longer seems to be the case.
comment:4 by , 5 years ago
Thanks a lot for the confirmation and for the additional information in form of the full patch file!
comment:5 by , 5 years ago
I see that they not just modified the detection tables, but also added some new parts to the engine itself. I'll see how this can be backported to the main scummvm tree, but it might take some time to get resolved.
comment:6 by , 5 years ago
This is a duplicate of these issues: https://bugs.scummvm.org/ticket/10347 and https://bugs.scummvm.org/ticket/10622.
This branch should be mostly ready to merge, provided nobody involved disagrees: https://github.com/bgK/scummvm/tree/pegasus-dvd.
comment:7 by , 5 years ago
Thanks for porting the DVD version patch bgK! I remember that clone2727 was working on the changes of the DVD version, which look to be substantial.
The patch does have a nasty hack (marked accordingly), and it includes CoreServices.h directly for OS X. Other than these two, the patch looks good :)
comment:8 by , 5 years ago
I removed the leftover macos include. I'd prefer not to touch to the hack as I wouldn't know what to do with it.
I did some engine-wide fixes a while back that are already in master. This branch contains also contains the DVD version specific corresponding code patches.
comment:10 by , 5 years ago
To be fair, I'm not sure creating a pull request straight away is the way to go. I would like to avoid repeating what happened to the buried engine, as the situation is similar.
comment:11 by , 5 years ago
Creating a pull request is fine. The code is published and the author (Keith) even present in our credits, so including this patch if we want is exactly in the spirit of the GPLv2, and the reason why our project is under this license.
comment:12 by , 5 years ago
Game: | → Journeyman Project Pegasus Prime |
---|
comment:13 by , 4 years ago
Owner: | changed from | to
---|---|
Resolution: | → fixed |
Status: | new → closed |
This is now supported.
Can you confirm the integrity of the data/files contained in the GoG installer? I wonder why the game is not detected since according to https://www.gog.com/game/journeyman_project_1_pegasus_prime_the the GoG release is already powered by ScummVM.