Removed Task.supersededBy method, for a simpler approach

Author: gmazzo

plugin/src/main/kotlin/com/github/gmazzo/buildconfig/BuildConfigTask.kt

@@ -21,8 +21,6 @@ public abstract class BuildConfigTask : DefaultTask() {
     @get:OutputDirectory
     public abstract val outputDir: DirectoryProperty
 
-    private var supersededBy: String? = null
-
     init {
         onlyIf("There are build config fields to generate") { task ->
             (task as BuildConfigTask).specs.get()
@@ -33,8 +31,6 @@ public abstract class BuildConfigTask : DefaultTask() {
 
     @TaskAction
     public fun generateBuildConfigFile() {
-        check(supersededBy == null) { "This has been superseeded by $supersededBy" }
-
         val dir = outputDir.get().asFile
         dir.deleteRecursively()
 
@@ -61,8 +57,4 @@ public abstract class BuildConfigTask : DefaultTask() {
         }
     }
 
-    internal fun supersededBy(other: BuildConfigClassSpec) {
-        this.supersededBy = other.name
-    }
-
 }

plugin/src/main/kotlin/com/github/gmazzo/buildconfig/internal/DefaultBuildConfigSourceSet.kt

@@ -64,9 +64,11 @@ internal abstract class DefaultBuildConfigSourceSet(
         check(other != this) { "A source set cannot supersede itself: '$name'" }
 
         if (!isSuperseded) {
+            val message = "'${this@DefaultBuildConfigSourceSet.name}' was superseded by '${other.name}' source set"
+
             isSuperseded = true
-            generateTask.configure { t ->
-                t.supersededBy(other)
+            generateTask.configure {
+                it.doFirst { error(message) }
             }
         }