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

Copy project icons to Gradle project directory during Android Custom Build. #39997

Merged

Conversation

amanj120
Copy link
Contributor

Being able to copy the project icons into the Gradle project directory is a necessary step in moving towards a gradle-based build system that can export Android App Bundles. For more details on the final scope of this project, read this comment.

These methods aren't currently called anywhere in the code, but they will be useful in subsequent PR's.

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

This PR builds on top of a previous PR 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.

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.

Since this is dependent on PR #39864, let's review it once that PR is complete.

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.

In a similar way to #40182, integrate the new functions in the export logic behind the use_custom_build flag.

platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Show resolved Hide resolved
@amanj120 amanj120 force-pushed the bundle_pr_icons branch 4 times, most recently from 2212fef to 2bd83ad Compare July 20, 2020 21:05
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.

Nice job, it's almost complete. Just some minor feedback to address.

platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
platform/android/export/export.cpp Outdated Show resolved Hide resolved
@amanj120 amanj120 force-pushed the bundle_pr_icons branch 2 times, most recently from c0f9002 to 84868c4 Compare July 22, 2020 19:52
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.

Looks great!

@m4gr3d m4gr3d requested review from akien-mga and a team July 23, 2020 15:59
@akien-mga akien-mga merged commit f1a8a15 into godotengine:3.2-android-app-bundle Jul 23, 2020
@akien-mga
Copy link
Member

Thanks!

@amanj120 amanj120 deleted the bundle_pr_icons branch July 23, 2020 16:08
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