Skip to content

Fix invalid project tree copy - #28

Merged
m4gr3d merged 1 commit into
godotengine:mainfrom
m4gr3d:fix_binary_copy
May 26, 2026
Merged

m4gr3d merged 1 commit into
godotengine:mainfrom
m4gr3d:fix_binary_copy

Conversation

@m4gr3d

@m4gr3d m4gr3d commented May 26, 2026

Copy link
Copy Markdown
Contributor

Building a second project after completing the build of the first one without restarting GABE would cause the second build to incorrectly use the project tree uri from the first build, resulting in an invalid binary being generated.
This is resolved by resetting the value of grantedTreeUri after returning it.

In addition, when retrieving a project tree uri from the shared preferences, we validate that it's still valid before using it.

Building a second project after completing the build of the first one without restarting GABE would cause the second build to incorrectly use the project tree uri from the first build, resulting in an invalid binary being generated.
This is resolved by resetting the value of `grantedTreeUri` after returning it.
In addition, when retrieving a projec tree uri from the shared preferences, we validate that it's still valid before using it.
@m4gr3d m4gr3d added this to the 1.0.0 milestone May 26, 2026
@m4gr3d
m4gr3d requested review from dsnopek and syntaxerror247 May 26, 2026 05:16
@m4gr3d m4gr3d added the bug Something isn't working label May 26, 2026
@m4gr3d
m4gr3d merged commit 10b8dee into godotengine:main May 26, 2026
1 check passed
@m4gr3d
m4gr3d deleted the fix_binary_copy branch May 26, 2026 17:03
syntaxerror247 pushed a commit to syntaxerror247/android-editor-buildenv-app that referenced this pull request May 27, 2026
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

Successfully merging this pull request may close these issues.

2 participants