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

Account for newly-added entry in the rows of _assetmanifest files #10

Merged
merged 4 commits into from
Apr 19, 2024

Conversation

64BTX
Copy link
Contributor

@64BTX 64BTX commented Apr 18, 2024

I've found that the extractor is failing when it tries to assign a specific entry from any asset manifest's rows to a size variable. This worked fine in the past, but a recent update to Priconne's manifest row format added a new entry before that of the byte size, causing the extractor to throw the following error:
pic

I've added a new allocation to the manifest scripts (the \src\manifests\ ones that assign each row's entries to something else). I've dubbed the entry "evertutocom" since the _assetmanifest files either use 'every', 'tutorial0/1', or 'common', but it could really be anything you want.

@lskyset
Copy link
Owner

lskyset commented Apr 19, 2024

So sad i saw this after fixing it myself but thanks

@lskyset lskyset merged commit be439ee into lskyset:main Apr 19, 2024
@lskyset lskyset linked an issue Apr 19, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compatibility issues due to system changes?
2 participants