<p>osmith has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/23953">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">OBS: refactor git_version_gen related logic<br><br>Move git_version_gen calls into an own function and add some of the<br>description from I76e3713f0b01a6110091ff90e8e53aa79533c374 where this<br>code was added.<br><br>Don't call it inside get_commit_version anymore, but call it before.<br>Don't try to cat the resulting .tarball-version there if it doesn't<br>exist.<br><br>Related: SYS#5370<br>Change-Id: I9a1b6ae4b4311abb77dc6390733c5e330e3d489e<br>---<br>M scripts/common-obs.sh<br>M scripts/osmocom-latest-packages.sh<br>M scripts/osmocom-nightly-packages.sh<br>3 files changed, 21 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/53/23953/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/scripts/common-obs.sh b/scripts/common-obs.sh</span><br><span>index 156dfba..a585e46 100644</span><br><span>--- a/scripts/common-obs.sh</span><br><span>+++ b/scripts/common-obs.sh</span><br><span>@@ -243,3 +243,12 @@</span><br><span>        git commit -m "auto-commit: apply $patch" debian/</span><br><span>  cd ..</span><br><span> }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+# Run git-version-gen inside Osmocom repositories, so the .tarball-version</span><br><span style="color: hsl(120, 100%, 40%);">+# becomes part of the source repository. Usually this would be done with</span><br><span style="color: hsl(120, 100%, 40%);">+# "make dist", but we use git-buildpackage instead.</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_obs_git_version_gen() {</span><br><span style="color: hsl(120, 100%, 40%);">+       if [ -x ./git-version-gen ]; then</span><br><span style="color: hsl(120, 100%, 40%);">+             ./git-version-gen . > .tarball-version 2>/dev/null</span><br><span style="color: hsl(120, 100%, 40%);">+      fi</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span>diff --git a/scripts/osmocom-latest-packages.sh b/scripts/osmocom-latest-packages.sh</span><br><span>index c530f27..a0bac45 100755</span><br><span>--- a/scripts/osmocom-latest-packages.sh</span><br><span>+++ b/scripts/osmocom-latest-packages.sh</span><br><span>@@ -122,9 +122,7 @@</span><br><span>     debian_branch="$FEED"</span><br><span>   fi</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-  if [ -x ./git-version-gen ]; then</span><br><span style="color: hsl(0, 100%, 40%);">-    ./git-version-gen . > .tarball-version 2>/dev/null</span><br><span style="color: hsl(0, 100%, 40%);">-  fi</span><br><span style="color: hsl(120, 100%, 40%);">+  osmo_obs_git_version_gen</span><br><span> </span><br><span>   osmo_obs_add_depend_deb "./debian/control" "$project" "osmocom-$FEED"</span><br><span> </span><br><span>diff --git a/scripts/osmocom-nightly-packages.sh b/scripts/osmocom-nightly-packages.sh</span><br><span>index ea69b94..07b0c81 100755</span><br><span>--- a/scripts/osmocom-nightly-packages.sh</span><br><span>+++ b/scripts/osmocom-nightly-packages.sh</span><br><span>@@ -53,15 +53,18 @@</span><br><span>   echo "${VER}"</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+# Return a version based on the latest tag and commit (e.g. "1.5.1.93.47cc")</span><br><span style="color: hsl(120, 100%, 40%);">+# or fall back to the last debian version with current date appended</span><br><span style="color: hsl(120, 100%, 40%);">+# (e.g. "2.2.6.202104270126"). Run osmo_obs_git_version_gen before.</span><br><span style="color: hsl(120, 100%, 40%);">+# $PWD must be inside a git repository.</span><br><span> get_commit_version() {</span><br><span style="color: hsl(0, 100%, 40%);">-  # return a version based on the commit</span><br><span style="color: hsl(0, 100%, 40%);">-  local version</span><br><span style="color: hsl(120, 100%, 40%);">+  local version=""</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-  # determine git version *and generate the .tarball-version file*</span><br><span style="color: hsl(0, 100%, 40%);">-  test -x ./git-version-gen && ./git-version-gen . > .tarball-version 2>/dev/null</span><br><span style="color: hsl(0, 100%, 40%);">-  version=$(cat .tarball-version)</span><br><span style="color: hsl(0, 100%, 40%);">-  # debian doesn't allow '-' in version.</span><br><span style="color: hsl(0, 100%, 40%);">-  version=$(echo "$version" | sed  's/-/./g' )</span><br><span style="color: hsl(120, 100%, 40%);">+  if [ -e ".tarball-version" ]; then</span><br><span style="color: hsl(120, 100%, 40%);">+    version=$(cat .tarball-version)</span><br><span style="color: hsl(120, 100%, 40%);">+    # debian doesn't allow '-' in version.</span><br><span style="color: hsl(120, 100%, 40%);">+    version=$(echo "$version" | sed  's/-/./g' )</span><br><span style="color: hsl(120, 100%, 40%);">+  fi</span><br><span> </span><br><span>   # deb version</span><br><span>   deb_version=$(head -1 debian/changelog | cut -d ' ' -f 2 | sed 's,(,,'  | sed 's,),,')</span><br><span>@@ -139,6 +142,7 @@</span><br><span>   cd "$repodir"</span><br><span> </span><br><span>   if [ "$changelog" = "commit" ] ; then</span><br><span style="color: hsl(120, 100%, 40%);">+    osmo_obs_git_version_gen</span><br><span>     # Add date to increase version even if commit did not change (OS#5135)</span><br><span>     VER="$(get_commit_version).$DT"</span><br><span>     osmo_obs_add_depend_deb "./debian/control" "$name" "osmocom-$FEED" "$dependver"</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ci/+/23953">change 23953</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ci/+/23953"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ci </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I9a1b6ae4b4311abb77dc6390733c5e330e3d489e </div>
<div style="display:none"> Gerrit-Change-Number: 23953 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>