Skip to content

A couple of fixes to compilation avoidance#2401

Merged
ting-yuan merged 2 commits into
google:mainfrom
ting-yuan:compilation-avoidance
Apr 8, 2025
Merged

A couple of fixes to compilation avoidance#2401
ting-yuan merged 2 commits into
google:mainfrom
ting-yuan:compilation-avoidance

Conversation

@ting-yuan

@ting-yuan ting-yuan commented Apr 7, 2025

Copy link
Copy Markdown
Contributor
  1. Exclude KAPT generated classes from class path snapshots. This fixes Incremental compilation fails with "Number of loaded files in snapshots differs" #1280. Tested locally with a sample project from @luboganev. Not sure why this bug is not caught by KAPT3IT yet and still investigating.
  2. Enable class path snapshots based incremental processing on Android builds, which was accidentally omitted previously. Hopefully this will make Android builds much faster.

@ting-yuan ting-yuan requested review from dx404 and jsjeon April 7, 2025 22:57
@ting-yuan ting-yuan merged commit f244bfc into google:main Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsjeon jsjeon Awaiting requested review from jsjeon

1 more reviewer

@dx404 dx404 dx404 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Incremental compilation fails with "Number of loaded files in snapshots differs"

2 participants

@ting-yuan @dx404