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

Write AndroidManifest.xml for Gradle project #40609

Merged

Conversation

amanj120
Copy link
Contributor

Writing to the AndroidManifest.xml file is a necessary step in moving towards a gradle-based build system that can export Android App Bundles. To do this, we will use manifest merging so we can avoid parsing the already existing manifest file. For more details on the final scope of this project, read this comment.

We are working on this PR with @m4gr3d, as explained here

This PR builds on top of many previous PRs I've submitted. There are a lot of changes that need to be made to move towards a Gradle-based build system, so I have broken these changes into smaller, more understandable steps.

@Calinou Calinou added this to the 3.2 milestone Jul 22, 2020
@amanj120 amanj120 force-pushed the bundle_pr_manifest branch 2 times, most recently from ab09b9e to 183fd7f Compare July 23, 2020 16:17
@m4gr3d m4gr3d self-requested a review July 23, 2020 19:34
@amanj120 amanj120 force-pushed the bundle_pr_manifest branch 2 times, most recently from 8dcb2e1 to 70c45c6 Compare July 24, 2020 16:50
Copy link
Contributor

@m4gr3d m4gr3d left a comment

Choose a reason for hiding this comment

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

Great job!

@m4gr3d m4gr3d requested review from akien-mga and a team July 24, 2020 17:24
@akien-mga akien-mga merged commit 1f67117 into godotengine:3.2-android-app-bundle Jul 24, 2020
@akien-mga
Copy link
Member

Thanks!

@amanj120 amanj120 deleted the bundle_pr_manifest branch July 24, 2020 23:56
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.

4 participants