EmbeddedKotlinProvider.kt

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Polish EmbeddedKotlinProvider

Signed-off-by: Paul Merlin <paul@gradle.com>

Pin the whole subgraph of embedded kotlin dependencies

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 3 more files in changeset.
Pin the whole subgraph of embedded kotlin dependencies

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 3 more files in changeset.
Remove the embedded-kotlin-repository

It was based on ClientModule dependencies that caused more troubles than

anything. The main one being that the artificial dependency graph needed

to be kept up to date with each Kotlin version. Moreover since several

Kotlin versions, declaring a repository is necessary in any case. This

change simplifies the code and should make the runtime a bit faster.

However, the Kotlin dependencies when applying the `kotlin-dsl` plugin

are now required to be downloaded. This isn't a big change and should

only impact people using the `kotlin-dsl` plugin without depending on

Kotlin in other parts of their build. This could be alleviated in the

future if dependency resolution considers the Gradle install/distro as a

source of artifacts in the same way it uses maven local.

Along the way, the pinning of Kotlin dependencies to the embedded

version for the build scripts classpath has been moved from a resolution

rule to proper, faster, dependency constraints.

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 6 more files in changeset.
Remove the embedded-kotlin-repository

It was based on ClientModule dependencies that caused more troubles than

anything. The main one being that the artificial dependency graph needed

to be kept up to date with each Kotlin version. Moreover since several

Kotlin versions, declaring a repository is necessary in any case. This

change simplifies the code and should make the runtime a bit faster.

However, the Kotlin dependencies when applying the `kotlin-dsl` plugin

are now required to be downloaded. This isn't a big change and should

only impact people using the `kotlin-dsl` plugin without depending on

Kotlin in other parts of their build. This could be alleviated in the

future if dependency resolution considers the Gradle install/distro as a

source of artifacts in the same way it uses maven local.

Along the way, the pinning of Kotlin dependencies to the embedded

version for the build scripts classpath has been moved from a resolution

rule to proper, faster, dependency constraints.

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 6 more files in changeset.
Remove the embedded-kotlin-repository

It was based on ClientModule dependencies that caused more troubles than

anything. The main one being that the artificial dependency graph needed

to be kept up to date with each Kotlin version. Moreover since several

Kotlin versions, declaring a repository is necessary in any case. This

change simplifies the code and should make the runtime a bit faster.

However, the Kotlin dependencies when applying the `kotlin-dsl` plugin

are now required to be downloaded. This isn't a big change and should

only impact people using the `kotlin-dsl` plugin without depending on

Kotlin in other parts of their build. This could be alleviated in the

future if dependency resolution considers the Gradle install/distro as a

source of artifacts in the same way it uses maven local.

Along the way, the pinning of Kotlin dependencies to the embedded

version for the build scripts classpath has been moved from a resolution

rule to proper, faster, dependency constraints.

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 6 more files in changeset.
Add missing dependency to embedded kotlin

* kotlin-stdlib now depends on kotlin-stdlib-common, this was not

reflected in the setup of the embedded kotlin repo.

* A way to experience this issue was with locking on a kotlin-dsl

project.

Fixes #10697

  1. … 1 more file in changeset.
Add missing dependency to embedded kotlin

* kotlin-stdlib now depends on kotlin-stdlib-common, this was not

reflected in the setup of the embedded kotlin repo.

* A way to experience this issue was with locking on a kotlin-dsl

project.

Add missing dependency to embedded kotlin

* kotlin-stdlib now depends on kotlin-stdlib-common, this was not

reflected in the setup of the embedded kotlin repo.

* A way to experience this issue was with locking on a kotlin-dsl

project.

Revert "Merge pull request #9759 from gradle/eskatos/kotlin/1.3.40"

This reverts commit 2c6fb9495acea6a150ad313ed3474ebc3689b920, reversing

changes made to 8959fc43370bc5141db6f775b333f36d30cc42a1.

  1. … 26 more files in changeset.
Upgrade embedded Kotlin from 1.3.31 to 1.3.40

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 7 more files in changeset.
Upgrade embedded Kotlin from 1.3.31 to 1.3.40

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 7 more files in changeset.
Add `kotlin-scripting-compiler-embeddable` to embedded Kotlin repository

  1. … 1 more file in changeset.
Fix :kCE dependencies

use stdlib instead of stdlib-jdk8

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 2 more files in changeset.
Replace :patchExternalModules with :kotlinCompilerEmbeddable

Only the kotlin-compiler-embeddable jar needs to be patched now.

Some patch actions were not needed anymore.

There's now a :kotlinCompilerEmbeddable subproject responsible for

building the patched kotlin-compiler-embeddable jar.

Only :kotlinDsl depends on it.

This greatly simplified the setup in the root project build script.

Various places required to be adjusted to account for the new setup.

The patching itself was rewritten to work in a streaming fashiong to

reduce i/o operations and disk usage. It was also moved up the

dependency tree.

All in all this should provide with a noticeable build speed up.

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 10 more files in changeset.
Import :kotlin-dsl and :kotlin-dsl-test-fixtures

Both in one go because of their circular dependency

Tests are failing because of the lack of :customInstallation

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -0
    • +194
    ./EmbeddedKotlinProvider.kt
  1. … 536 more files in changeset.