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 AndroidJavaCompile not detecting incremental changes #11331

Merged
merged 4 commits into from Nov 13, 2019

Conversation

wolfs
Copy link
Member

@wolfs wolfs commented Nov 12, 2019

The method fun getSources(): FileTree does not override the method
protected FileCollection getSources(), since the AGP plugin 3.5.1 has
not been compiled against Gradle 6.0, and therefore the necessary bridge
method is not generated.

Fixes #11330.

@wolfs wolfs added this to the 6.0.1 milestone Nov 12, 2019
@wolfs wolfs self-assigned this Nov 12, 2019
@wolfs wolfs changed the title Fix Android not compiling the necessary classes Fix AndroidJavaCompile not detecting incremental changes Nov 12, 2019
The method `fun getSources(): FileTree` does not override the method
`protected FileCollection getSources()`, since the AGP plugin 3.5.1 has
not been compiled against Gradle 6.0, and therefore the necessary bridge
method is not generated.
@wolfs wolfs force-pushed the wolfs/java-compile/fix-android branch from d11af34 to 2e9e6f8 Compare November 12, 2019 14:10
@wolfs wolfs changed the base branch from master to release November 12, 2019 14:10
@wolfs wolfs requested a review from lptr November 12, 2019 15:07
@wolfs
Copy link
Member Author

wolfs commented Nov 12, 2019

@lptr Please take a look!

@big-guy
Copy link
Member

big-guy commented Nov 12, 2019

Can we update one of the Android smoke tests to do an incremental compile too?

@wolfs
Copy link
Member Author

wolfs commented Nov 13, 2019

@big-guy We can and we should.

The breaking change actually fixes the problem.
@wolfs wolfs merged commit 5fae21e into release Nov 13, 2019
@wolfs wolfs deleted the wolfs/java-compile/fix-android branch November 13, 2019 14:12
@wolfs
Copy link
Member Author

wolfs commented Nov 13, 2019

I'll do the smoke test(s) on master, given we changed quite a bit in those test already.

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