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

Update for Android library changes in Godot 4.4-beta4 #266

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

dsnopek
Copy link
Collaborator

@dsnopek dsnopek commented Feb 11, 2025

These are the changes necessary to make hybrid apps compile and work with the (not-yet-committed) Godot PR godotengine/godot#102669.

It's built on top of #264, with the assumption that we'll merge that soon so our CI will work and we can resume other stuff, and that this PR will be merged after 4.4-beta4 releases (with the changes from godotengine/godot#102669)

@dsnopek dsnopek added the bug Something isn't working label Feb 11, 2025
@dsnopek dsnopek added this to the 4.x milestone Feb 11, 2025
@dsnopek dsnopek requested a review from m4gr3d February 11, 2025 16:59
@dsnopek dsnopek marked this pull request as draft February 11, 2025 16:59
@dsnopek dsnopek force-pushed the hybrid-godot-44b4-android-lib branch from 2b37463 to 4eeaa32 Compare February 11, 2025 21:58
@dsnopek dsnopek force-pushed the hybrid-godot-44b4-android-lib branch from 4eeaa32 to 8ad96bf Compare February 17, 2025 22:17
@dsnopek dsnopek changed the title [DRAFT] Update for Android library changes in Godot 4.4-beta4 (unreleased) Update for Android library changes in Godot 4.4-beta4 Feb 17, 2025
@dsnopek dsnopek marked this pull request as ready for review February 17, 2025 22:23
@dsnopek
Copy link
Collaborator Author

dsnopek commented Feb 17, 2025

This is working for me in the hybrid app sample with the (now released) Godot 4.4-beta4! Taking out of DRAFT status

@dsnopek dsnopek requested a review from m4gr3d February 17, 2025 22:24
@dsnopek
Copy link
Collaborator Author

dsnopek commented Feb 18, 2025

Looks like we're going to be failing CI until this PR is merged - see this recent CI job: https://github.com/GodotVR/godot_openxr_vendors/actions/runs/13393965119/job/37408499157

@dsnopek dsnopek requested a review from devloglogan February 18, 2025 16:03
@m4gr3d m4gr3d merged commit 19a0237 into GodotVR:master Feb 19, 2025
11 checks passed
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