From 601ae69db9095e70dc4ab68f26153445f89213cc Mon Sep 17 00:00:00 2001 From: Gert Hulselmans Date: Mon, 5 Feb 2024 21:24:49 +0100 Subject: [PATCH] Don't build for Windows and some Linux versions. 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: https://github.com/rust-bio/rust-htslib/pull/415 - Building zlib-ng does not work for all Linux targets: - armv7: - Might get fixed in the future if zig gets fixed: https://github.com/ziglang/zig/issues/10411 --- .github/workflows/CI.yml | 29 +++-------------------------- 1 file changed, 3 insertions(+), 26 deletions(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 1bae4be..5ed96ae 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -23,7 +23,7 @@ jobs: runs-on: ubuntu-latest strategy: matrix: - target: [x86_64, x86, aarch64, armv7, s390x, ppc64le] + target: [x86_64, s390x] steps: - uses: actions/checkout@v3 - uses: actions/setup-python@v4 @@ -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 @@ -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: @@ -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: