change id and steps

Author: LeonHibnik

.github/workflows/cpp-golang-rust.yml

@@ -73,7 +73,6 @@ jobs:
 
   test-curve:
     name: Test Curve on ${{ matrix.backend }} with ${{ matrix.runner_os }}
-    # runs-on: [self-hosted, Linux, X64, icicle]
     runs-on: ${{ matrix.runner }}
     needs: [check-changed-files, check-format, extract-cuda-backend-branch, extract-metal-backend-branch]
     strategy:
@@ -126,16 +125,16 @@ jobs:
       working-directory: ./icicle/backend/${{ matrix.backend }}
       id: extract-backend-sha
       run: |
-        ${{ matrix.backend_upper }}BACKEND_SHA=$(git rev-parse HEAD)
-        echo "${{ matrix.backend_upper }} Backend Commit SHA: $${{ matrix.backend_upper }}_BACKEND_SHA"
-        echo "${{ matrix.backend }}-backend-sha=$${{ matrix.backend_upper }}_BACKEND_SHA" >> $GITHUB_OUTPUT
+          BACKEND_SHA=$(git rev-parse HEAD)
+          echo "Backend Commit SHA: $BACKEND_SHA"
+          echo "${{ matrix.backend }}-backend-sha=$BACKEND_SHA" >> $GITHUB_ENV
     - name: Set ${{ matrix.backend_upper }} backend flag
       if: needs.check-changed-files.outputs.golang == 'true' || needs.check-changed-files.outputs.cpp == 'true' ||  needs.check-changed-files.outputs.rust == 'true'
       id: backend-flag
       run: |
-        ${{ matrix.backend_upper }}_BACKEND_SHA=${{ steps.extract-${{ matrix.backend }}-sha.outputs.${{ matrix.backend }}-backend-sha }}
+        BACKEND_SHA=${{ env.${{ matrix.backend }}-backend-sha }}
         CURVE=${{ matrix.curve.name }}
-        COMMIT_FILE="gh_commit_sha_${CURVE}_${${{ matrix.backend_upper }}_BACKEND_SHA}"
+        COMMIT_FILE="gh_commit_sha_${CURVE}_${BACKEND_SHA}"
         if [ "${{ needs.extract-${{ matrix.backend }}-backend-branch.outputs.${{ matrix.backend }}-backend-branch }}" == "main" ]; then
           INSTALL_PATH=${{ github.workspace }}/../../main_lib/curve/$CURVE
           echo "INSTALL_PATH=${{ github.workspace }}/../../main_lib/curve/$CURVE" >> $GITHUB_OUTPUT
@@ -174,23 +173,23 @@ jobs:
       if: needs.check-changed-files.outputs.golang == 'true' || needs.check-changed-files.outputs.cpp == 'true' ||  needs.check-changed-files.outputs.rust == 'true'
       run: |
         mkdir -p build && rm -rf build/*
-        cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=ON -DCURVE=${{ matrix.curve.name }} ${{ matrix.curve.build_args }} ${{ steps.${{ matrix.backend }}-flag.outputs.${{ matrix.backend_upper }}_FLAG }} ${{ steps.${{ matrix.backend }}-flag.outputs.CMAKE_INSTALL_PREFIX }} -S . -B build
+        cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=ON -DCURVE=${{ matrix.curve.name }} ${{ matrix.curve.build_args }} ${{ steps.backend-flag.outputs.${{ matrix.backend_upper }}_FLAG }} ${{ steps.backend-flag.outputs.CMAKE_INSTALL_PREFIX }} -S . -B build
         cmake --build build --target install -j
-        rm -rf ${{ steps.${{ matrix.backend }}-flag.outputs.INSTALL_PATH }}/lib/gh_commit_sha_${{ matrix.curve.name }}*
-        touch ${{ steps.${{ matrix.backend }}-flag.outputs.COMMIT_FILE_PATH }}
+        rm -rf ${{ steps.backend-flag.outputs.INSTALL_PATH }}/lib/gh_commit_sha_${{ matrix.curve.name }}*
+        touch ${{ steps.backend-flag.outputs.COMMIT_FILE_PATH }}
 
     - name: Run C++ Curve Tests
       working-directory: ./icicle/build/tests
       if: needs.check-changed-files.outputs.cpp == 'true'
       run: |
-        export ICICLE_BACKEND_INSTALL_DIR=${{ steps.${{ matrix.backend }}-flag.outputs.ICICLE_BACKEND_INSTALL_DIR }}
+        export ICICLE_BACKEND_INSTALL_DIR=${{ steps.backend-flag.outputs.ICICLE_BACKEND_INSTALL_DIR }}
         ctest --output-on-failure
     - name: Run C++ examples
       working-directory: ./examples/c++
       if: needs.check-changed-files.outputs.cpp == 'true' || needs.check-changed-files.outputs.examples == 'true'
       run: |
         CURVE=${{ matrix.curve.name }}
-        export ICICLE_BACKEND_INSTALL_DIR=${{ steps.${{ matrix.backend }}-flag.outputs.ICICLE_BACKEND_INSTALL_DIR }}
+        export ICICLE_BACKEND_INSTALL_DIR=${{ steps.backend-flag.outputs.ICICLE_BACKEND_INSTALL_DIR }}
         for dir in $(grep -l -r "\\-DCURVE=$CURVE" . | xargs -L1 dirname | sort -u); do
             if [ -d "$dir" ]; then
                 echo "Running command in $dir"
@@ -223,7 +222,7 @@ jobs:
         CURVE=${{ matrix.curve.name }}
         CURVE_DIR=$(echo ${{ matrix.curve.name }} | sed -e 's/_//g')
         export ICICLE_BACKEND_INSTALL_DIR=${{ steps.cuda-flag.outputs.ICICLE_BACKEND_INSTALL_DIR }}
-        export LD_LIBRARY_PATH=${{ steps.${{ matrix.backend }}-flag.outputs.INSTALL_PATH }}/lib
+        export LD_LIBRARY_PATH=${{ steps.backend-flag.outputs.INSTALL_PATH }}/lib
         export CGO_LDFLAGS="-L$LD_LIBRARY_PATH -licicle_field_$CURVE -licicle_curve_$CURVE  -lstdc++ -Wl,-rpath=LD_LIBRARY_PATH"
         go test ./$CURVE_DIR/tests -count=1 -failfast -p 2 -timeout 60m -v