Skip to content

Commit

Permalink
This commit changes the order of configurations in the global buildsc…
Browse files Browse the repository at this point in the history
…ript: when atomicfu-gradle-plugin is not added to the classpath first, it is not applied at all.

This behaviour started to reproduce after this change in kotlinx-atomicfu (Kotlin/kotlinx-atomicfu#377).

The problem was revealed while running a train build before kotlinx-atomicfu 0.23.2 release. The coroutines build failed with error: "e: Module kotlinx.atomicfu cannot be found in the module graph".
  • Loading branch information
mvicsokolova committed Dec 11, 2023
1 parent 60e8acb commit c495784
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,11 @@ import static Projects.*
apply plugin: 'jdk-convention'

buildscript {

dependencies {
classpath "org.jetbrains.kotlinx:atomicfu-gradle-plugin:$atomicfu_version"
}

/*
* These property group is used to build kotlinx.coroutines against Kotlin compiler snapshot.
* How does it work:
Expand Down Expand Up @@ -59,11 +64,10 @@ buildscript {
dependencies {
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
classpath "org.jetbrains.dokka:dokka-gradle-plugin:$dokka_version"
classpath "org.jetbrains.kotlinx:atomicfu-gradle-plugin:$atomicfu_version"
classpath "org.jetbrains.kotlin:atomicfu:$kotlin_version"
classpath "org.jetbrains.kotlinx:kotlinx-knit:$knit_version"
classpath "org.jetbrains.kotlinx:binary-compatibility-validator:$binary_compatibility_validator_version"
classpath "ru.vyarus:gradle-animalsniffer-plugin:1.5.4" // Android API check
classpath "org.jetbrains.kotlin:atomicfu:$kotlin_version"
classpath "org.jetbrains.kotlinx:kover-gradle-plugin:$kover_version"

// JMH plugins
Expand Down

0 comments on commit c495784

Please sign in to comment.