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

Support reading of Godot4 .pck files #32

Merged
merged 2 commits into from
Aug 5, 2023

Conversation

rgson
Copy link
Contributor

@rgson rgson commented Jul 31, 2023

Implements reading of Godot 4 .pck files (i.e. PCK version 2).

Decryption is not implemented. An error is produced if the pack's flags indicate that the content is encrypted.

Closes: #17

@hhyyrylainen
Copy link
Owner

This seems like this implements just reading? Which is fine, but I just want to confirm. I'll need to put a note in the readme / prevent trying to write out a .pck file with version 2 so that no one reports that as a bug.

The PCK version 2 format is implemented for reading but not for writing.
@rgson
Copy link
Contributor Author

rgson commented Aug 1, 2023

That's correct.

I've pushed another commit now to at least block the saving of version 2 .pck files (until someone implements that part).

@hhyyrylainen
Copy link
Owner

The way the blocking is done seems fine to me.
Do you happen to have a Godot 4 .pck file just so I can verify this works correctly now? I tried asking in #17 but no one has replied.

@rgson
Copy link
Contributor Author

rgson commented Aug 2, 2023

Sure. Here's a .pck from godot-demo-projects/2d/platformer exported with Godot 4.1.1.stable.official.bd6af8e0e on Linux.

Platformer 2D.pck.tar.gz

Copy link
Owner

@hhyyrylainen hhyyrylainen left a comment

Choose a reason for hiding this comment

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

Seems fine to me, I tested with the provided Godot 4 pck and didn't find any issues with that or with using a Godot 3 .pck file with these changes.

@hhyyrylainen hhyyrylainen merged commit 117ff02 into hhyyrylainen:master Aug 5, 2023
2 checks passed
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.

Add support for Godot 4 PCK files
2 participants