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

Fix manifest merging task to work with AGP 4.0.0 #151

Merged
merged 2 commits into from
Jun 5, 2020

Conversation

JohnnyMLa
Copy link
Contributor

AGP 4.0.0 brings in a change which wipes out the output files whenever a task gets rerun
See https://android.googlesource.com/platform/tools/base/+/studio-master-dev/build-system/gradle-core/src/main/java/com/android/build/gradle/internal/tasks/NonIncrementalTask.kt
The manifest merging task tries to update the manifest in place, by setting input == output, resulting in the input file being explicitly deleted just before the task gets run
Fix by making input != output, by setting input to some arbitrary copy location in the fat-R directory

AGP 4.0.0 brings in a change which wipes out the output files whenever a task gets rerun
See https://android.googlesource.com/platform/tools/base/+/studio-master-dev/build-system/gradle-core/src/main/java/com/android/build/gradle/internal/tasks/NonIncrementalTask.kt
The manifest merging task tries to update the manifest in place, by setting input == output, resulting in the input file being explicitly deleted just before the task gets run
Fix by making input != output, by setting input to some arbitrary copy location in the fat-R directory
@JohnnyMLa
Copy link
Contributor Author

Fixes #147

@JohnnyMLa
Copy link
Contributor Author

JohnnyMLa commented Jun 4, 2020

Also, this is an awesome project @kezong!
Still don't understand why it's not supported out-of-the-box by Google, but this is the next best thing.

@kezong
Copy link
Owner

kezong commented Jun 5, 2020

nice!
I will check this commit soon.

@kezong kezong merged commit 4f16a30 into kezong:master Jun 5, 2020
@achmyr
Copy link

achmyr commented Jun 5, 2020

@kezong are you planning to do a release with this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants