Skip to content

[KGP] Default classpath metadata to the common sources IC property - #7638

Open
Iveta Kovalenko (ywett02) wants to merge 1 commit into
ikvl/osip-75/property-exportfrom
ikvl/osip-75/metadata-defauls
Open

[KGP] Default classpath metadata to the common sources IC property#7638
Iveta Kovalenko (ywett02) wants to merge 1 commit into
ikvl/osip-75/property-exportfrom
ikvl/osip-75/metadata-defauls

Conversation

@ywett02

Copy link
Copy Markdown
Collaborator

enableJvmClasspathMetadata defaulted to false, so opting into kotlin.jvm.enableIncrementalCompilationOfCommonSources alone left common sources compiled incrementally without the isolated fragment dependency view that keeps them from resolving against platform declarations - the KT-62686 problem the classpath metadata exists to prevent.

Default it to the value of the common sources IC property instead, so that enabling common sources IC also enables the mechanism that makes it correct. The internal property still wins when set explicitly, so it remains a kill switch for turning the metadata off on its own.

^KT-88763 Fixed

`enableJvmClasspathMetadata` defaulted to `false`, so opting into
`kotlin.jvm.enableIncrementalCompilationOfCommonSources` alone left
common sources compiled incrementally without the isolated fragment
dependency view that keeps them from resolving against platform
declarations - the KT-62686 problem the classpath metadata exists to
prevent.

Default it to the value of the common sources IC property instead, so
that enabling common sources IC also enables the mechanism that makes
it correct. The internal property still wins when set explicitly, so
it remains a kill switch for turning the metadata off on its own.

^KT-88763 Fixed
@kotlin-safemerge

kotlin-safemerge Bot commented Aug 21, 2026

Copy link
Copy Markdown

Code Owners

RuleOwnersApproval
/​libraries/​tools/​kotlin-​gradle-​plugin-​integration-​tests/​, /​libraries/​tools/​kotlin-​gradle-​plugin/​
kotlin-build-tools

Egorand 🔒
PR commands for maintainers
CommandDescriptionParameters
/safe-mergeRebase-merges with automatic fixup commit squashing--fixup Autosquash fixup commits (on by default)
/safe-squash-mergeSquash-merges with optional commit title/body override--title Title of the squashed commit
--message Body of the squashed commit
/dry-runRuns the test pipeline with changes rebased on latest master--retry Retry the CI run on failure
/test-publicTriggers the public test suite without rebasing on latest master
/test-privateTriggers the private test suite without rebasing on latest master
/codeownersTriggers code owners check and comment update
/fixupSquashes fixup commits and force pushes the branch
/cancel-coordinatorCancels the merge coordinator currently running for this branch

@Egorand Egor Andreevich (Egorand) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/final

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.

2 participants