Move update version steps into separate composite action; label unopt…

…imized builds
aerospike · Feb 20, 2024 · 5feadd0 · 5feadd0
1 parent 877debe
commit 5feadd0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/.github/workflows/build-wheels.yml b/.github/workflows/build-wheels.yml
@@ -160,6 +160,16 @@ jobs:
       if: ${{ !inputs.run_tests }}
       run: echo "TEST_COMMAND=python -c 'import aerospike'" >> $GITHUB_ENV
 
+    - name: Get current version if building a debug wheel
+      if: ${{ inputs.build-for-debugging }}
+      run: echo DEBUG_VERSION="$(cat VERSION)+unoptimized" >> $GITHUB_ENV
+
+    - name: Label version for debugging
+      if: ${{ inputs.build-for-debugging }}
+      uses: ./.github/actions/update-version
+      with:
+        new_version: ${{ env.DEBUG_VERSION }}
+
     - name: Build without optimizations
       if: ${{ inputs.build-for-debugging }}
       run: echo "CIBW_ENVIRONMENT_PASS_LINUX_VARS='UNOPTIMIZED=1'" >> $GITHUB_ENV

diff --git a/.github/workflows/update-version.yml b/.github/workflows/update-version.yml
@@ -45,11 +45,10 @@ jobs:
         token: ${{ secrets.CLIENT_BOT_PAT }}
         ref: ${{ inputs.ref }}
 
-    - name: Update __version__ in aerospike module
-      run: sed -i "s/const char version\[] = \".*\";/const char version\[] = \"${{ inputs.new_version }}\";/" src/main/aerospike.c
-
-    - name: Update VERSION metadata
-      run: echo ${{ inputs.new_version }} > VERSION
+    - name: Update version in repo
+      uses: .github/action/update-version
+      with:
+        new_version: ${{ inputs.new_version }}
 
     - name: Commit new version
       uses: stefanzweifel/git-auto-commit-action@v4