Skip to content

Gradle Plugin: Replace execute with finalizedBy #650

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

Merged
merged 1 commit into from
Jan 18, 2019

Conversation

bretthoerner
Copy link

Fixes #648 for me.

@nesterov-n
Copy link

@bretthoerner Would you mind merging this PR? I believe many teams already use gradle 5.0 and this fix is required

@mitutitu16
Copy link

@bretthoerner Can you please confirm when we should expect this fix to be merged in? We're facing issues with Gradle 5.1.

@nesterov-n
Copy link

nesterov-n commented Jan 18, 2019

By the way name of dex task has changed in recent releases of the android gradle plugin.
was: transformClassesWithDexFor<variant>
now: transformClassesWithDexBuilderFor<variant>

I think we should add the new name in getDexTask method

static Task getDexTask(Project project, ApplicationVariant variant) {

Seems like it fixes #648 too.

@bretthoerner bretthoerner merged commit e885662 into master Jan 18, 2019
@bretthoerner
Copy link
Author

@nesterov-n Good catch, you can check out #665?

Once I get both of these in and try them I can make a release.

@bretthoerner bretthoerner deleted the gradle-replace-execute branch February 4, 2019 15:43
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