suggested feedback: pass revision as build parameter, only create ver…

…sion file when present (or file does not exist). Remove input based on .git/index.

build.gradle

@@ -112,8 +112,3 @@ task devBuild(dependsOn: [":assemble"]) {
     }
   }
 }
-gradle.taskGraph.whenReady { graph ->
-  if (graph.hasTask(devBuild)) {
-    project.skipVersionFileRegeneration = true
-  }
-}

geode-core/build.gradle

@@ -220,11 +220,10 @@ task createVersionPropertiesFile {
   onlyIf {
     // Project properties are parsed as strings.
     // Use boolean comparison, not the value itself, to determine truth.
-    !(propertiesFile.exists() && project.skipVersionFileRegeneration)
+    !propertiesFile.exists() || project.sourceRevision != "0"
   }
 
   outputs.file propertiesFile
-  inputs.dir "../.git/index"
 
   doLast {
     def props = [

gradle.properties

@@ -39,6 +39,9 @@ buildId = 0
 productName = Apache Geode
 productOrg = Apache Software Foundation (ASF)
 
+# This value is specified on the commandline at build, or else is zero.
+sourceRevision = 0
+
 org.gradle.daemon = true
 org.gradle.jvmargs = -Xmx3g
 
@@ -66,7 +69,3 @@ org.gradle.caching = true
 org.gradle.parallel = false
 org.gradle.configureondemand = false
 
-# Setting this property to a truthy value, typically via command line -PskipVersionFileRegeneration,
-# allows for a faster build as part of a normal development cycle.
-# This property is also assigned to true in the devBuild target.
-skipVersionFileRegeneration=