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

Use implicit launch when running a Godot Android project from the editor #101498

Merged

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Jan 13, 2025

This allows developers to override the default launching activity (com.godot.game.GodotApp) with a custom one if desired (Should unblock godot-sdk-integrations/godot-meta-toolkit#1).

Logic is added to fallback to the default launching activity if the implicit launch fails.
Also did some misc cleanups by adding warnings for the export options that depends on the gradle build.

This allows developers to override the default launching activity (`com.godot.game.GodotApp`) with a custom one if desired.
Logic is added to fallback to the default launching activity if the implicit launch fails.
@m4gr3d m4gr3d added this to the 4.4 milestone Jan 13, 2025
@m4gr3d m4gr3d requested a review from dsnopek January 13, 2025 16:55
@m4gr3d m4gr3d requested a review from a team as a code owner January 13, 2025 16:55
Copy link
Contributor

@dsnopek dsnopek left a comment

Choose a reason for hiding this comment

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

Thanks!

This works great in my testing with PR godot-sdk-integrations/godot-meta-toolkit#1, swapping which activity should be launched by default.

I also tested it with a normal Android smartphone app, both with and without "Show In App Library" enabled, in order to excercise the fallback where it gives the default activity name.

The code looks good to me as well!

@akien-mga akien-mga merged commit e898130 into godotengine:master Jan 13, 2025
20 checks passed
@akien-mga
Copy link
Member

Thanks!

@m4gr3d m4gr3d deleted the run_launcher_godot_app_from_editor branch January 13, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants