From 2a445c55d82bb4c138d0c602f539aabe287b220c Mon Sep 17 00:00:00 2001
From: Andrei Sokol <andrei@poolside.ai>
Date: Sat, 30 Nov 2024 13:56:17 -0500
Subject: [PATCH] set dir for wheels

---
 .github/workflows/wheel.yml | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/wheel.yml b/.github/workflows/wheel.yml
index 3fbfb482..10d84a17 100644
--- a/.github/workflows/wheel.yml
+++ b/.github/workflows/wheel.yml
@@ -7,6 +7,7 @@ on:
 env:
   DOMAIN: poolside
   REPOSITORY: poolside-dagster
+  WHEEL_DST: /tmp/sp_wheelhouse
 
 jobs:
   build_wheels:
@@ -41,13 +42,13 @@ jobs:
           role-to-assume: arn:aws:iam::939990436136:role/gh-action-publish-artifacts-role
           aws-region: us-east-1
 
-      - name: Build for Mac
-        if: runner.os == 'macOS'
+      - name: Make sure destination dir exists
         run: |
-          cmake -B ${{github.workspace}}/build -DSPM_ENABLE_SHARED=OFF -DCMAKE_INSTALL_PREFIX=${{github.workspace}}/build/root
-          cmake --build ${{github.workspace}}/build --config Release --target install --parallel 8
-        env:
-          CMAKE_OSX_ARCHITECTURES: arm64;x86_64
+          mkdir -p ${{env.WHEEL_DST}}
+          if [ ! -d ${{env.WHEEL_DST}}]; then
+            echo "wheel dest dir does not exist!"
+            exit 1
+          fi
 
       - name: Install cibuildwheel
         working-directory: ${{github.workspace}}/python
@@ -55,6 +56,14 @@ jobs:
           python3 -m pip install --require-hashes --no-dependencies -r ../.github/workflows/requirements/base.txt
           python3 -m pip install --require-hashes --no-dependencies -r ../.github/workflows/requirements/cibuildwheel.txt
 
+      - name: Build for Mac
+        if: runner.os == 'macOS'
+        run: |
+          cmake -B ${{github.workspace}}/build -DSPM_ENABLE_SHARED=OFF -DCMAKE_INSTALL_PREFIX=${{github.workspace}}/build/root
+          cmake --build ${{github.workspace}}/build --config Release --target install --parallel 8
+        env:
+          CMAKE_OSX_ARCHITECTURES: arm64;x86_64
+
       - name: Build wheels
         working-directory: ${{github.workspace}}/python
         run: python3 -m cibuildwheel --output-dir wheelhouse
@@ -77,11 +86,11 @@ jobs:
           files: ./python/dist/*.tar.gz
 
       - name: Build wheel from sdist
-        run: python3 -m pip wheel "${{ steps.sdist.outputs.paths }}" --verbose
+        run: python3 -m pip wheel "${{ steps.sdist.outputs.paths }}" --wheel-dir=${{env.WHEEL_DST}} --verbose
 
       - name: Publish
         run: |
           export TWINE_USERNAME=aws
           export TWINE_PASSWORD=`aws codeartifact get-authorization-token --domain ${{env.DOMAIN}} --domain-owner ${{ secrets.AWS_ACCOUNT_ID }} --query authorizationToken --output text`
           export TWINE_REPOSITORY_URL=`aws codeartifact get-repository-endpoint --domain ${{env.DOMAIN}} --domain-owner ${{ secrets.AWS_ACCOUNT_ID }} --repository ${{ env.REPOSITORY }}  --region us-east-1 --format pypi --query repositoryEndpoint --output text`
-          twine upload --verbose ${{ steps.sdist.outputs.paths }}/*.whl
+          twine upload --verbose ${{env.WHEEL_DST}}/*.whl