Skip to content

Commit

Permalink
Use build/setuptools to build crc32c
Browse files Browse the repository at this point in the history
The distutils module was removed in 3.12, so let's make sure setuptools
is installed every time instead.

Signed-off-by: Rodrigo Tobar <[email protected]>
  • Loading branch information
rtobar committed Dec 15, 2024
1 parent 8ed9364 commit 783f9a9
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 3 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ jobs:
with:
python-version: '3.13'

- name: Install setuptools
run: pip install setuptools

- name: Install crc32c
run: python setup.py develop

Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/build_and_release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,11 @@ jobs:
with:
python-version: '3.13'

- name: Install build
run: pip install build

- name: Build sdist
run: python setup.py sdist
run: python -m build -s

- uses: actions/upload-artifact@v4
with:
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@ jobs:
architecture: x64
- name: Checkout
uses: actions/checkout@v3
- name: Install build
run: pip install build
- name: Install dev requirements
run: pip install -r requirements-dev.txt
- name: Install package
run: python setup.py bdist_wheel && pip install dist/crc32c*.whl
- name: Build and install package
run: python -m build -w && pip install dist/crc32c*.whl
- name: Run mypy
run: mypy --strict src test
- name: Run black
Expand Down

0 comments on commit 783f9a9

Please sign in to comment.