Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Parsing] MONKEY1-AMIGA: LFLF 61 differently indexed from one file to another #56

Open
1 task done
dwatteau opened this issue Mar 26, 2022 · 6 comments
Open
1 task done
Assignees
Labels
bug Something isn't working

Comments

@dwatteau
Copy link
Owner

Summary

With the Amiga version of Monkey1, from LRG's Monkey Island Anthology.

Trying to reimport the original text content, but with an added -A ao flag so that extra padding is inserted (and so that some bytes differ) will result in the following fatal error:

$ scummtr -g monkey -A ao -of test.txt
$ scummtr -g monkey -A ao -if test.txt
ERROR: LFLF 61 differently indexed from one file to another

This looks similar to issue #54, but the error message is different. As far as I can say, the Amiga version is the closest version to the VGA-DOS version.

This error also happens with the original 2003 scummtr.exe, so this isn't a regression.

Impacted games

The Secret of Monkey Island

ScummTR versions

v0.5.1, v0.4.0

I own a legitimate game

  • I promise that my game does NOT come from an illegal source, such as "abandonware" websites
@dwatteau dwatteau added the bug Something isn't working label Mar 26, 2022
@dwatteau dwatteau self-assigned this Mar 26, 2022
@dwatteau
Copy link
Owner Author

dwatteau commented May 4, 2022

Since this appears related to the #54 issue, the fix will probably be the same

#54 (comment)

i.e., work around the (official) invalid resources of Monkey1 VGA.

@gabberhead
Copy link

i have the same problem with the german amiga version of the game. i did a retranslation like i did for the vga verman version. i can extract the text, but the import gives me the same error.

@dwatteau
Copy link
Owner Author

Ah yes that problem. Some releases of Monkey1 have broken resources, that would require a different approach to SCUMMv4 parsing.

Something like this:

https://bitbucket.org/jestar_jokin/scummpacker/commits/96f34dc6a2b947dea259382387c574329c819efe

or the ScummVM equivalent

It's not very hard to do, but it requires a good understanding of the resources, which I don't really have.

I had interest and time in fixing this some years ago when I opened this issue, but nowadays I don't know if it's going to happen from me. The NUTCracker project is supporting more and more SCUMM games and is more actively maintained, with a newer and cleaner architecture, so the efforts and time are maybe better spent in this sister project instead.

Anyway: maybe it's going to be fixed in scummtr some day, but I doubt I will doing it myself or soon, sorry…

@gabberhead
Copy link

I know nutcracker also. But it works only with scumm v5+ ganes. Till now :) Monkey Island 1 is v4. And I dont know other Tools, for editing the Text.

@dwatteau
Copy link
Owner Author

I know nutcracker also. But it works only with scumm v5+ ganes. Till now :) Monkey Island 1 is v4. And I dont know other Tools, for editing the Text.

Sure, but nutcracker had WIP code for v4 I think – at least Loom or something like that. You can reach out 'rzil' on the ScummVM Discord for this. Monkey1 is maybe on his roadmap; I'd suggest mentioning the 'scummpacker' link above to him, so that he remains aware of some v4 parsing particularities.

(Of course he's also doing this work on his free time and may choose not to work on this now etc.)

@gabberhead
Copy link

german indy atlantis amiga version, has the same problem like monkey 1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants