Skip to content
New issue

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

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

Already on GitHub? # to your account

Project binary injection #89

Merged
merged 6 commits into from
Feb 4, 2023
Merged

Project binary injection #89

merged 6 commits into from
Feb 4, 2023

Conversation

KANAjetzt
Copy link
Member

Project binary injection

Because we cant set the order of autoloads via override.cfg we have to use external tooling ( GodotPckTool ) to inject the modified project settings in to the games .pck file.

Relevant feature proposal:
godotengine/godot-proposals#6137

New functions

  • reorder_autoloads()
    Removes and re adds the autoloads from the ProjectSettings to set the ModLoader at the top.

  • create_project_binary()
    Saves the current ProjectSettings to ./addons/mod_loader/project.binary

  • inject_project_binary()
    Uses godotPckTool to add the custom project.binary to the original game pck file.

  • clean_up_project_binary_file()
    Removes the project.binary from ./addons/mod_loader/project.binary

  • setup_file_data()
    Initializes the 2 dictionary's path and file_name

@KANAjetzt KANAjetzt requested a review from a team February 3, 2023 23:48
@KANAjetzt KANAjetzt mentioned this pull request Feb 4, 2023
Copy link
Collaborator

@ithinkandicode ithinkandicode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

Copy link
Member

@otDan otDan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

everything looks in place

@KANAjetzt KANAjetzt merged commit 87b44cc into GodotModding:develop Feb 4, 2023
@KANAjetzt KANAjetzt mentioned this pull request Feb 4, 2023
@KANAjetzt KANAjetzt deleted the project_binary_injection branch February 8, 2023 18:32
@KANAjetzt KANAjetzt mentioned this pull request Feb 15, 2023
This was referenced Feb 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants