Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting "target" and "manylinux" to build musl only builds python3.8 wheels #56

Closed
tgross35 opened this issue Jul 11, 2022 · 6 comments
Closed

Comments

@tgross35
Copy link

I haven't been able to figure out what exactly causes this, but in this example configuration:

    - uses: messense/maturin-action@v1
      with:
        manylinux: auto
        command: build
        args: --release -o dist --manifest-path crates/stringmetrics_py/Cargo.toml
    # NOTE: Seems to only build musl wheels for cpython 3.8
    - uses: messense/maturin-action@v1
      with:
        target: x86_64-unknown-linux-musl
        manylinux: musllinux_1_1
        command: build
        args: --release -o dist --manifest-path crates/stringmetrics_py/Cargo.toml

The first step generates wheels for 3.7-3.10 as expected, but the second only does 3.8.

@messense
Copy link
Member

messense commented Jul 12, 2022

Please upgrade to maturin 0.13.0 and try the new —-find-interpreter option.

@tgross35
Copy link
Author

Can maturin version be specified in the action? --find-interpreter works on my local but is not recognized there.

@messense
Copy link
Member

@tgross35
Copy link
Author

@messense I still have not been able to have it build successfully. My configuration is here which is almost identical to what is used in auditwheel-symbols (I don't need the full matrix for all platforms). I also tried turning "container" off like auditwheel does, but that wouldn't run successfully.

Here is the raw output from a run. Do you have any suggestions?

@messense
Copy link
Member

Can you try replacing --find-interpreter for musl with -i 3.7 3.8 3.9 3.10?

@tgross35
Copy link
Author

Good news, that worked. Thanks for the help.

Working file for anyone who needs it: https://github.com/pluots/stringmetrics/blob/a763935dec7e1c91f17c7c46f9e57f7c61b16a9d/.github/workflows/wheels.yml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants