Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove comment on how to use different kotlin-dsl plugin versions

    • -8
    • +0
    ./codegen/GenerateKotlinDependencyExtensions.kt
Revert "Revert "Take advantage of type-safe accessors in precompiled script plugins""

This reverts commit cbe1b9ed3f28e6fb39feaba56e8140f490dc5a6b.

    • -3
    • +2
    ./plugins/kotlin-dsl-module.gradle.kts
    • -1
    • +1
    ./plugins/kotlin-dsl-plugin-bundle.gradle.kts
  1. … 3 more files in changeset.
Revert "Take advantage of type-safe accessors in precompiled script plugins"

    • -2
    • +3
    ./plugins/kotlin-dsl-module.gradle.kts
    • -1
    • +1
    ./plugins/kotlin-dsl-plugin-bundle.gradle.kts
  1. … 3 more files in changeset.
Exclude :kotlinDslPlugins DSL types for now

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

Include the Kotlin DSL public API in checkBinaryCompatibility

excluding Kotlin classes for inlined functions

and ignoring checks that require parsing the Kotlin sources

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

    • -0
    • +29
    ./PublicKotlinDslApi.kt
  1. … 5 more files in changeset.
Remove the API parameter names JARs

In order to enable code navigation and refactoring across Kotlin DSL

project boundaries.

The need for parameter names JARs has been mitigated by removing

usages of Kotlin interface delegation from the public API and

implementing the required interfaces explicitly.

Resolves #1360

    • -238
    • +0
    ./build/AddGradleApiParameterNames.kt
  1. … 23 more files in changeset.
Take advantage of type-safe accessors in precompiled script plugins

    • -3
    • +2
    ./plugins/kotlin-dsl-module.gradle.kts
    • -1
    • +1
    ./plugins/kotlin-dsl-plugin-bundle.gradle.kts
Take advantage of type-safe accessors in precompiled script plugins

    • -3
    • +2
    ./plugins/kotlin-dsl-module.gradle.kts
    • -1
    • +1
    ./plugins/kotlin-dsl-plugin-bundle.gradle.kts
Polish `GenerateKotlinDependencyExtensions`

- Make all task inputs `Property<T>`

- Let Gradle provide the properties

    • -4
    • +8
    ./codegen/GenerateKotlinDependencyExtensions.kt
  1. … 1 more file in changeset.
Prefer abstract task properties where applicable

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

    • -2
    • +3
    ./build/PatchKotlinCompilerEmbeddable.kt
Prefer abstract task properties where applicable

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

    • -10
    • +13
    ./build/PatchKotlinCompilerEmbeddable.kt
Refine :kCE:patchKotlinCompilerEmbeddable task

by turning additionnalFiles into a FileCollection

renamed to additionnalRootFiles for clarity

using @PathSensitive(PathSensitivity.NAME_ONLY)

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

    • -20
    • +19
    ./build/PatchKotlinCompilerEmbeddable.kt
  1. … 1 more file in changeset.
Move `kotlinDsl` generated sources to `$buildDir/generated-sources`

And remove no longer necessary `clean` task configuration.

    • -4
    • +3
    ./codegen/GenerateKotlinDependencyExtensions.kt
  1. … 1 more file in changeset.
Infer parameters from action

    • -14
    • +7
    ./build/AddGradleApiParameterNames.kt
  1. … 10 more files in changeset.
Rename all registered types to be without suffix

    • -0
    • +245
    ./build/AddGradleApiParameterNames.kt
    • -245
    • +0
    ./build/GradleApiParameterNamesTransform.kt
  1. … 10 more files in changeset.
Migrate api parameter names transform to new API

    • -23
    • +43
    ./build/GradleApiParameterNamesTransform.kt
Tighten KotlinSourceParser

removing all unneeded stuff

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

First cut of Kotlin support in incubation report

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

    • -0
    • +120
    ./parser/KotlinSourceParser.kt
  1. … 3 more files in changeset.
Add a build step which reruns previous failed test classes (#8470)

This closes https://github.com/gradle/gradle-private/issues/1774

We've been bitten by flaky tests for a long time. This PR adds an extra build step which reruns failed test classes again.

  1. … 3 more files in changeset.
Fix and re-enable :kotlinCompilerEmbeddable:clean

The CheckKotlinCompilerEmbeddableDependencies task had a property

getter calling DefaultTask.getTemporaryDir() which has the side effect

of creating that directory on each call. It happened very often that

this getter was called concurrently to the clean task, by overlapping

outputs detection maybe, causing the clean task to fail with "unable to

delete file".

This commit changes that task property to a simple field with no side

effects, moving the directory creation to the task action.

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

    • -5
    • +6
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
  1. … 1 more file in changeset.
Cleanup

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

    • -4
    • +2
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
Fix mistake

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

    • -0
    • +1
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
Don't use DefaultTask.getTemporaryDir() in a task property

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

    • -1
    • +1
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
Remove IJ workaround and rework :kotlinCompilerEmbeddable build logic

See 35948c40b8a

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

    • -33
    • +34
    ./build/PatchKotlinCompilerEmbeddable.kt
  1. … 1 more file in changeset.
Fix :kotlinDslPlugins dependency on the Gradle API

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

    • -0
    • +10
    ./plugins/kotlin-dsl-plugin-bundle.gradle.kts
  1. … 1 more file in changeset.
Refine ParameterNamesIndex task

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

Rename task field for clarity

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

    • -3
    • +3
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
Validate :kotlinCompilerEmbeddable deps are aligned with the original

In order to ease Kotlin upgrades by failing the build if the

dependencies aren't aligned.

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

    • -0
    • +62
    ./build/CheckKotlinCompilerEmbeddableDependencies.kt
  1. … 1 more file in changeset.
Refine PatchKotlinCompilerEmbeddable task

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

    • -1
    • +4
    ./build/PatchKotlinCompilerEmbeddable.kt
  1. … 1 more file 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>

    • -0
    • +158
    ./build/PatchKotlinCompilerEmbeddable.kt
  1. … 10 more files in changeset.