From: Ward Vandewege <ward@curii.com>
Date: Thu, 20 May 2021 21:24:18 +0000 (-0400)
Subject: Make sure our four digit tags are excluded from the version number
X-Git-Tag: 2.2.0~4
X-Git-Url: https://git.arvados.org/arvados-workbench2.git/commitdiff_plain/ec46a9b5c55b01008eda1f1475779c8f1cee9e1f

Make sure our four digit tags are excluded from the version number
calculation.

No issue #

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward@curii.com>
---

diff --git a/version-at-commit.sh b/version-at-commit.sh
index 89684cf2..84d7d0b4 100755
--- a/version-at-commit.sh
+++ b/version-at-commit.sh
@@ -1,4 +1,7 @@
 #!/bin/bash
+# Copyright (C) The Arvados Authors. All rights reserved.
+#
+# SPDX-License-Identifier: AGPL-3.0
 
 set -e -o pipefail
 commit="$1"
@@ -35,9 +38,10 @@ else
 
     if git merge-base --is-ancestor "$nearest_tag" "$merge_base" ; then
         # x.(y+1).0.devTIMESTAMP, where x.y.z is the newest version that does not contain $commit
-	# grep reads the list of tags (-f) that contain $commit and filters them out (-v)
-	# this prevents a newer tag from retroactively changing the versions of everything before it
-        v=$(git tag | grep -vFf <(git tag --contains "$commit") | sort -Vr | head -n1 | perl -pe 's/\.(\d+)\.\d+/".".($1+1).".0"/e')
+        # grep reads the list of tags (-f) that contain $commit and filters them out (-v)
+        # this prevents a newer tag from retroactively changing the versions of everything before it
+        # We also filter out four-digit faux semantic versioning tags, they gum up the works.
+        v=$(git tag | grep -vFf <(git tag --contains "$commit") |grep -v -P '\d.\d.\d.\d' | sort -Vr | head -n1 | perl -pe 's/\.(\d+)\.\d+/".".($1+1).".0"/e')
     else
         # x.y.(z+1).devTIMESTAMP, where x.y.z is the latest released ancestor of $commit
         v=$(echo $nearest_tag | perl -pe 's/(\d+)$/$1+1/e')