Skip to content

Commit

Permalink
Merge pull request #9 from jussmi/manylinux-linux-wheels-jjs
Browse files Browse the repository at this point in the history
Adding manylinux builds
  • Loading branch information
faustomorales committed Apr 16, 2024
2 parents a1a0f50 + 0f3f6c9 commit bcfcb8e
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 6 deletions.
28 changes: 22 additions & 6 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,17 @@ name: pdqhash-python CI/CD

on: workflow_dispatch

# We don't currently build for Linux, because
# we need to sort out manylinux. Maybe another day!
jobs:
build:
name: Build Package
name: Build Windows and MacOS Package
strategy:
matrix:
python: ['3.7', '3.8', '3.9', '3.10']
python: ['3.8', '3.9', '3.10', '3.11']
os: [windows-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
- name: Checkout Code
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
submodules: true
fetch-depth: 0
Expand All @@ -37,4 +35,22 @@ jobs:
- uses: actions/upload-artifact@v3
with:
name: wheels
path: dist/*
path: dist/*
build-linux:
name: Build Linux Package
strategy:
matrix:
python: ['3.8', '3.9', '3.10', '3.11']
runs-on: ubuntu-latest
steps:
- name: Checkout Code
uses: actions/checkout@v4
with:
submodules: true
fetch-depth: 0
- name: Build Manylinux Wheels
run: docker run --rm -v ${{ github.workspace }}:/io --workdir /io quay.io/pypa/manylinux_2_28_x86_64 make PYTHON_VERSION=${{ matrix.python }} manylinux-wheel
- uses: actions/upload-artifact@v3
with:
name: linux-wheels
path: dist/
5 changes: 5 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,8 @@ test:
package:
rm -rf dist
$(PYTHON) setup.py sdist bdist_wheel
manylinux-wheel: PYBIN = "/opt/python/cp$(subst .,,$(PYTHON_VERSION))-cp$(subst .,,$(PYTHON_VERSION))/bin"
manylinux-wheel:
rm -rf dist
$(PYBIN)/pip install -r requirements.txt
$(PYBIN)/python setup.py sdist bdist_wheel

0 comments on commit bcfcb8e

Please sign in to comment.