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

Change target sdk to 28 and use latest Gradle plugin #186

Merged
merged 2 commits into from
Oct 3, 2018

Conversation

lbalmaceda
Copy link
Contributor

This is probably the root cause of "androidX" not working properly when this SDK was a dependency

@lbalmaceda lbalmaceda added this to the v1-Next milestone Oct 2, 2018
Copy link
Contributor

@friederbluemle friederbluemle left a comment

Choose a reason for hiding this comment

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

👍

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-bin.zip
Copy link
Contributor

Choose a reason for hiding this comment

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

When using the wrapper task to update the Gradle wrapper version, please consider doing

./gradlew wrapper  --gradle-version 4.10.2 --distribution-type all

The default for Android projects is the all variant, which provides additional documentation and sources helpful when working on Gradle related stuff. Since it's the default it's also much more likely to be already downloaded in a Gradle cache. It is a one-time, machine-wide download.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I'll update it

@lbalmaceda
Copy link
Contributor Author

I'm still fighting against jacoco to get the coverage reports exported as xml. Something on this oss plugin update broke it and I'm trying to figure out if it was because of the gradle version or the android plugin version. Once that's done I'll merge this. cc @friederbluemle

@friederbluemle
Copy link
Contributor

@lbalmaceda The Android Gradle plugin 3.2.0 changed the output directory for javac compiles classes files (required for JaCoCo). It used to be under <build_dir>/intermediates/classes/[...], now it is <build_dir>/intermediates/javac/[...] (directory depends on build variant).

Currently, the path is hard coded inside of AndroidLibraryPlugin.groovy

You probably need to change

dir: "${project.buildDir}/intermediates/classes/${name}",

to

dir: "${project.buildDir}/intermediates/javac/${name}",

@lbalmaceda lbalmaceda merged commit 45f6933 into master Oct 3, 2018
@lbalmaceda lbalmaceda modified the milestones: v1-Next, 1.14.0 Oct 3, 2018
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