Build/internal/NesQuEIT: enforce no colon : for nessieProject() (#…

…9842) ... otherwise leads to invalid `:buildName::moduleName` Gradle project references.
projectnessie · Oct 30, 2024 · 3ec9513 · 3ec9513
1 parent 1a1ae4f
commit 3ec9513
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/build-logic/src/main/kotlin/Utilities.kt b/build-logic/src/main/kotlin/Utilities.kt
@@ -195,6 +195,9 @@ fun DependencyHandlerScope.nessieProject(
   artifactId: String,
   configuration: String? = null
 ): ModuleDependency {
+  if (artifactId.startsWith(":")) {
+    throw IllegalArgumentException("artifactId for nessieProject() must not start with ':'")
+  }
   return if (!isIncludedInNesQuEIT(project(":").dependencyProject.gradle)) {
     project(":$artifactId", configuration)
   } else {

diff --git a/integrations/spark-extensions/build.gradle.kts b/integrations/spark-extensions/build.gradle.kts
@@ -30,7 +30,7 @@ dependencies {
   // picks the right dependencies for scala compilation
   forScala(sparkScala.scalaVersion)
 
-  implementation(nessieProject(":nessie-cli-grammar"))
+  implementation(nessieProject("nessie-cli-grammar"))
   implementation(project(":nessie-spark-extensions-base_${sparkScala.scalaMajorVersion}"))
   compileOnly("org.apache.spark:spark-sql_${sparkScala.scalaMajorVersion}") { forSpark(sparkScala.sparkVersion) }
   compileOnly("org.apache.spark:spark-core_${sparkScala.scalaMajorVersion}") { forSpark(sparkScala.sparkVersion) }