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

[BT] Add support for SourcesChanges.ToBeCalculated #5344

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ywett02
Copy link

@ywett02 ywett02 commented Aug 8, 2024

KT-70556

Currently, the conversion from org.jetbrains.kotlin.buildtools.api.SourcesChanges.ToBeCalculated to org.jetbrains.kotlin.incremental.ChangedFiles results in the use of ChangedFiles.Unknown (codepointer). This configuration triggers a rebuild of the whole module (codepointer).

Instead of using ChangedFiles.Unknown, I propose returning null to prompt the IC to track the file changes.

Testing

./gradlew :compiler:build-tools:kotlin-build-tools-api-tests:check

Reusing configuration cache.

BUILD SUCCESSFUL in 40s
366 actionable tasks: 163 executed, 193 from cache, 10 up-to-date

Publishing build scan...
https://gradle.com/s/eha7w77c4ygvi

Configuration cache entry reused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants