Skip to content

Commit

Permalink
fix(build): resolve Python version and source directory issues
Browse files Browse the repository at this point in the history
This commit addresses two build configuration issues:

- Remove incorrect python-source setting from pyproject.toml
- Add explicit Python interpreter selection in maturin build
- Update pip installation commands for better compatibility
- Add architecture specification for Python setup

These changes ensure that:
1. Maturin correctly identifies the Python package structure
2. Build process uses the intended Python version
3. Dependencies are properly installed in the build environment
  • Loading branch information
dirvine committed Nov 28, 2024
1 parent c26abfd commit 9e04b06
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/python-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,16 +81,18 @@ jobs:
- uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
architecture: x64
- name: Install dependencies
run: |
python -m pip install --user cffi
python -m pip install --user patchelf
python -m pip install --upgrade pip
python -m pip install cffi
python -m pip install patchelf
- name: Build wheels
uses: PyO3/maturin-action@v1
with:
target: ${{ matrix.target }}
manylinux: "2014"
args: --release --out dist --compatibility manylinux2014 --skip-auditwheel
args: --release --out dist --compatibility manylinux2014 --skip-auditwheel --interpreter python${{ matrix.python-version }}
container: quay.io/pypa/manylinux2014_x86_64
sccache: 'true'
- name: Upload wheels
Expand Down
1 change: 0 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ features = ["python"]
module-name = "self_encryption._self_encryption"
bindings = "pyo3"
develop = true
python-source = "python"
compatibility = "manylinux2014"

[tool.pytest.ini_options]
Expand Down

0 comments on commit 9e04b06

Please sign in to comment.