Skip to content

Run add_arm_to_libomp_dylib.sh only on arm64 #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 3, 2023
Merged

Run add_arm_to_libomp_dylib.sh only on arm64 #91

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 12 additions & 7 deletions .github/workflows/wheels.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,11 @@ jobs:
# manually so that both x86 and arm builds can be built.
run: |
brew install coreutils
brew install libomp
sh add_arm_to_libomp_dylib.sh
if [[ ${{ matrix.cibw_archs }} == "arm64" ]] ; then
echo "Building universal libomp manually"
brew install libomp
sh add_arm_to_libomp_dylib.sh
fi

- name: Build Wheels
env:
Expand All @@ -127,7 +130,7 @@ jobs:
CIBW_ENVIRONMENT_MACOS: BREW_LIBOMP="1" SUITESPARSE_MACOS_ARCH=${{ matrix.cibw_archs }}

# Uncomment to only build CPython wheels
# CIBW_BUILD: "cp*"
# CIBW_BUILD: "cp*"

# Architectures to build specified in matrix
CIBW_ARCHS: ${{ matrix.cibw_archs }}
Expand All @@ -151,7 +154,9 @@ jobs:
CIBW_TEST_COMMAND: "pytest --pyargs suitesparse_graphblas -s -k test_print_jit_config && pytest -v --pyargs suitesparse_graphblas"

# GitHub Actions macOS Intel runner cannot run ARM tests. Uncomment to silence warning.
# CIBW_TEST_SKIP: "*-macosx_arm64"
# CIBW_TEST_SKIP: "*-macosx_arm64"
# XXX: tests are failing for macos_x86_64; let's see if we can figure out what's wrong by releasing a broken package... (sorry!)
# CIBW_TEST_SKIP: "*-macosx_*"

run: |
python -m pip install --upgrade pip
Expand Down Expand Up @@ -181,7 +186,7 @@ jobs:
needs: [build_wheels, build_sdist]
runs-on: ubuntu-latest
if: github.repository == 'GraphBLAS/python-suitesparse-graphblas'
# if: github.event_name == 'release' && github.event.action == 'published'
# if: github.event_name == 'release' && github.event.action == 'published'

steps:
- uses: actions/setup-python@v4
Expand All @@ -202,5 +207,5 @@ jobs:
password: ${{ secrets.PYPI_TOKEN }}

# Test PyPI:
# password: ${{ secrets.TEST_PYPI_API_TOKEN }}
# repository_url: https://test.pypi.org/legacy/
# password: ${{ secrets.TEST_PYPI_API_TOKEN }}
# repository_url: https://test.pypi.org/legacy/