Skip to content

Commit

Permalink
Don't build for Windows and some Linux versions.
Browse files Browse the repository at this point in the history
Don't build for Windows and some Linux versions:
  - Building HTSlib on Windows is not possible.
  - Building HTSlib does not work on Linux:
      - x86:
          - len() as u64 is not valid on 32-bit
      - aarch64 and ppc64le:
          - expected `*mut u8`, found `&mut i8`
          - Fixed by: rust-bio/rust-htslib#415
  - Building zlib-ng does not work for all Linux targets:
      - armv7:
          - Might get fixed in the future if zig gets fixed:
            ziglang/zig#10411
      - s390:
          - Error: Unrecognized opcode: `stfle'
  • Loading branch information
ghuls committed Feb 5, 2024
1 parent b7b20e2 commit a54d31f
Showing 1 changed file with 3 additions and 26 deletions.
29 changes: 3 additions & 26 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
target: [x86_64, x86, aarch64, armv7, s390x, ppc64le]
target: [x86_64]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
Expand All @@ -33,7 +33,7 @@ jobs:
uses: PyO3/maturin-action@v1
with:
target: ${{ matrix.target }}
args: --release --out dist --find-interpreter
args: --release --out dist --zig --find-interpreter
sccache: 'true'
manylinux: auto
- name: Upload wheels
Expand All @@ -42,29 +42,6 @@ jobs:
name: wheels
path: dist

windows:
runs-on: windows-latest
strategy:
matrix:
target: [x64, x86]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: '3.10'
architecture: ${{ matrix.target }}
- name: Build wheels
uses: PyO3/maturin-action@v1
with:
target: ${{ matrix.target }}
args: --release --out dist --find-interpreter
sccache: 'true'
- name: Upload wheels
uses: actions/upload-artifact@v3
with:
name: wheels
path: dist

macos:
runs-on: macos-latest
strategy:
Expand Down Expand Up @@ -106,7 +83,7 @@ jobs:
name: Release
runs-on: ubuntu-latest
if: "startsWith(github.ref, 'refs/tags/')"
needs: [linux, windows, macos, sdist]
needs: [linux, macos, sdist]
steps:
- uses: actions/download-artifact@v3
with:
Expand Down

0 comments on commit a54d31f

Please sign in to comment.