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

Do not call variant.javaCompile for android 3.3.0+ to get rid of Android API warning #318

Merged
merged 1 commit into from
Jun 18, 2019

Conversation

voidzcy
Copy link
Collaborator

@voidzcy voidzcy commented Jun 17, 2019

Resolves #295

I believe this is just something forgotten in #305. Thanks @kcoppock for point this out in #295 (comment).

@voidzcy voidzcy requested a review from zhangkun83 June 17, 2019 22:33
Copy link

@kcoppock kcoppock left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!

@voidzcy voidzcy removed the request for review from zhangkun83 June 17, 2019 22:35
Copy link
Collaborator

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

Oops

@zhangkun83 zhangkun83 merged commit 50abbc0 into master Jun 18, 2019
@voidzcy
Copy link
Collaborator Author

voidzcy commented Jun 18, 2019

@zhangkun83 I was trying to enable tests for Android plugin 3.4 and Gradle 5.4. But it seems there were things changed in 3.2 and will cause tests fail at mergeArmFreeappDebugAndroidTestResources task.

@kcoppock
Copy link

kcoppock commented Jul 3, 2019

Do you know an ETA when there will be a 0.9.0 release available with this change?

@voidzcy
Copy link
Collaborator Author

voidzcy commented Jul 10, 2019

@kcoppock Since this warning fix isn't a sever bug, we may not have a new release until having other bigger changes.

@UmanShahzad
Copy link

It's breaking Android builds - can we just do a 0.8.10 for this?

@voidzcy
Copy link
Collaborator Author

voidzcy commented Jul 10, 2019

/cc @zhangkun83

@voidzcy voidzcy deleted the bugfix/android_variant_get_java_compile_warning branch July 11, 2019 00:14
@zhangkun83
Copy link
Collaborator

Yeah. 0.8.10 now released!

@UmanShahzad
Copy link

Thanks a lot!

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.

WARNING: API 'variant.getJavaCompile()' is obsolete
5 participants