Merge pull request #248 from adpi2/perf/git-described-version

Perf: Eliminate redundant call to `git.describedVersion`
sbt · Sep 18, 2024 · 76cb0e4 · 76cb0e4
2 parents f079e4b + ebd8d34
commit 76cb0e4
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/main/scala/com/github/sbt/git/GitPlugin.scala b/src/main/scala/com/github/sbt/git/GitPlugin.scala
@@ -155,7 +155,15 @@ object SbtGit {
     commands += GitCommand.command,
     gitTagToVersionNumber := git.defaultTagByVersionStrategy,
     gitDescribePatterns := Seq.empty[String],
-    gitDescribedVersion := gitReader.value.withGit(_.describedVersion((ThisProject / gitDescribePatterns).value)).map(v => git.gitTagToVersionNumber.value(v).getOrElse(v)),
+    gitDescribedVersion := {
+      val projectPatterns = gitDescribePatterns.value
+      val buildPatterns = (ThisBuild / gitDescribePatterns).value
+      val projectTagToVersionNumber = gitTagToVersionNumber.value
+      val buildTagToVersionNumber = (ThisBuild / gitTagToVersionNumber).value
+      if (projectPatterns == buildPatterns && projectTagToVersionNumber == buildTagToVersionNumber)
+        (ThisBuild / gitDescribedVersion).value
+      else gitReader.value.withGit(_.describedVersion(projectPatterns)).map(v => projectTagToVersionNumber(v).getOrElse(v))
+    },
   )
 
   /** A Predefined setting to use JGit runner for git. */