Skip to content

Remove PCK encryption note for Android. #11072

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

Merged
merged 1 commit into from
Jul 5, 2025

Conversation

bruvzg
Copy link
Member

@bruvzg bruvzg commented Jul 1, 2025

Since godotengine/godot#105984 was merged, PCK encryption is supported for APK / AAB exports as well.

@bruvzg bruvzg added this to the 4.5 milestone Jul 1, 2025
@bruvzg bruvzg added the area:contributing Issues and PRs related to the Contributing/Development section of the documentation label Jul 1, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Looks good to me.

However, I'm not sure if we should remove the note entirely right now, given how "important" it is. Maybe we should change it to say "Prior to Godot 4.5, ..." instead, so that users on 4.4 who happen to be reading stable once 4.5 is released will still be aware of this limitation.

We can remove the note entirely once 4.6 is out instead.

@bruvzg
Copy link
Member Author

bruvzg commented Jul 1, 2025

Not sure if it's important, encryption has extremely limited use, and this note most likely was not and will not be noticed by most users.

Copy link
Contributor

@skyace65 skyace65 left a comment

Choose a reason for hiding this comment

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

I'm in agreement with bruvzg. I think it's fine to add without specifying the version

@mhilbrunner
Copy link
Member

Same, I think this is fine.

@skyace65 skyace65 merged commit d438b93 into godotengine:master Jul 5, 2025
1 check passed
@skyace65
Copy link
Contributor

skyace65 commented Jul 5, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:contributing Issues and PRs related to the Contributing/Development section of the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants