Skip to content

Commit 6cf812f

Browse files
authored
Merge pull request #1522 from lesserwhirls/gradle
Better support for gradle composite builds
2 parents 84b63c1 + b3cc05c commit 6cf812f

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+370
-366
lines changed

.github/workflows/style.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jobs:
2020
key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }}
2121
restore-keys: |
2222
${{ runner.os }}-gradle-
23-
- name: build-logic Code Style Check with Gradle and Spotless
24-
run: ./gradlew :build-logic:clean :build-logic:spotlessCheck
23+
- name: build-logic-ncj Code Style Check with Gradle and Spotless
24+
run: ./gradlew :build-logic-ncj:clean :build-logic-ncj:spotlessCheck
2525
- name: netCDF-Java Code Style Check with Gradle and Spotless
2626
run: ./gradlew clean spotlessCheck

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ jobs:
2222
JDK8: /usr/thredds-test-environment/${{ matrix.java-vendor }}8
2323
- name: Prep for artifact upload
2424
if: failure()
25-
run: rm -rf build-logic/build/tmp
25+
run: rm -rf build-logic-ncj/build/tmp
2626
- uses: actions/upload-artifact@v4
2727
if: failure()
2828
with:

bufr/build.gradle.kts

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
*/
55

66
plugins {
7-
id("java-library-conventions")
8-
id("protobuf-conventions")
7+
id("ncj-java-library-conventions")
8+
id("ncj-protobuf-conventions")
99
}
1010

1111
description = "Reading BUFR files with the NetCDF-java library."
@@ -17,22 +17,22 @@ dependencies {
1717

1818
api(project(":cdm-core"))
1919

20-
implementation(libs.beust.jcommander)
21-
implementation(libs.findbugs.jsr305)
22-
implementation(libs.guava)
23-
implementation(libs.jdom2)
24-
implementation(libs.protobuf)
25-
implementation(libs.re2j)
26-
implementation(libs.slf4j.api)
20+
implementation(ncjLibs.beust.jcommander)
21+
implementation(ncjLibs.findbugs.jsr305)
22+
implementation(ncjLibs.guava)
23+
implementation(ncjLibs.jdom2)
24+
implementation(ncjLibs.protobuf)
25+
implementation(ncjLibs.re2j)
26+
implementation(ncjLibs.slf4j.api)
2727

2828
testImplementation(platform(project(":netcdf-java-testing-platform")))
2929

3030
testImplementation(project(":cdm-test-utils"))
3131

32-
testImplementation(libs.google.truth)
32+
testImplementation(ncjLibs.google.truth)
3333

34-
testCompileOnly(libs.junit4)
34+
testCompileOnly(ncjLibs.junit4)
3535

36-
testRuntimeOnly(libs.junit5.platformLauncher)
37-
testRuntimeOnly(libs.junit5.vintageEngine)
36+
testRuntimeOnly(ncjLibs.junit5.platformLauncher)
37+
testRuntimeOnly(ncjLibs.junit5.vintageEngine)
3838
}
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@
55

66
plugins {
77
`kotlin-dsl`
8-
alias(libs.plugins.protobuf)
9-
alias(libs.plugins.spotless)
8+
alias(ncjLibs.plugins.protobuf)
9+
alias(ncjLibs.plugins.spotless)
1010
}
1111

1212
dependencies {
13-
implementation(plugin(libs.plugins.protobuf))
14-
implementation(plugin(libs.plugins.spotless))
13+
implementation(plugin(ncjLibs.plugins.protobuf))
14+
implementation(plugin(ncjLibs.plugins.spotless))
1515
}
1616

1717
spotless {
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ dependencyResolutionManagement {
99
mavenCentral()
1010
gradlePluginPortal()
1111
}
12-
versionCatalogs { create("libs") { from(files("../gradle/libs.versions.toml")) } }
12+
versionCatalogs { create("ncjLibs") { from(files("../gradle/ncj.libs.versions.toml")) } }
1313
}
1414

15-
rootProject.name = "build-logic"
15+
rootProject.name = "build-logic-ncj"

build-logic/src/main/kotlin/artifact-publishing-conventions.gradle.kts renamed to build-logic-ncj/src/main/kotlin/ncj-artifact-publishing-conventions.gradle.kts

File renamed without changes.

build-logic/src/main/kotlin/base-conventions.gradle.kts renamed to build-logic-ncj/src/main/kotlin/ncj-base-conventions.gradle.kts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ val catalogs = extensions.getByType<VersionCatalogsExtension>()
99

1010
group = "edu.ucar"
1111

12-
version = catalogs.named("libs").findVersion("netcdf-java").get().requiredVersion
12+
version = catalogs.named("ncjLibs").findVersion("netcdf-java").get().requiredVersion
1313

1414
description = "A component to the Unidata netCDF-Java library (aka CDM)."
1515

@@ -62,5 +62,5 @@ project.extra["public.artifacts"] = publicArtifacts
6262
// will be the bytecode produced by the project for all java compilation
6363
// will be used to run the tests (test, not testWithJdkX), generate code coverage reports, etc.
6464
// other versions of java can be used to run the tests, but this is configured in
65-
// testing-conventions.gradle.kts
65+
// ncj-testing-conventions.gradle.kts
6666
project.extra["project.minimumJdkVersion"] = "8"

build-logic/src/main/kotlin/java-base-conventions.gradle.kts renamed to build-logic-ncj/src/main/kotlin/ncj-java-base-conventions.gradle.kts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
*/
55

66
plugins {
7-
id("base-conventions")
8-
id("testing-conventions")
7+
id("ncj-base-conventions")
8+
id("ncj-testing-conventions")
99
}
1010

1111
tasks {

build-logic/src/main/kotlin/java-library-conventions.gradle.kts renamed to build-logic-ncj/src/main/kotlin/ncj-java-library-conventions.gradle.kts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44
*/
55

66
plugins {
7-
id("java-base-conventions")
7+
id("ncj-java-base-conventions")
88
`java-library`
9-
id("artifact-publishing-conventions")
9+
id("ncj-artifact-publishing-conventions")
1010
id("com.diffplug.spotless")
1111
}
1212

build-logic/src/main/kotlin/platform-conventions.gradle.kts renamed to build-logic-ncj/src/main/kotlin/ncj-platform-conventions.gradle.kts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ import org.gradle.api.publish.maven.MavenPublication
77
import org.gradle.kotlin.dsl.create
88

99
plugins {
10-
id("base-conventions")
10+
id("ncj-base-conventions")
1111
`java-platform`
12-
id("artifact-publishing-conventions")
12+
id("ncj-artifact-publishing-conventions")
1313
}
1414

1515
publishing {

0 commit comments

Comments
 (0)