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

Build bcrypt wheels for FreeBSD 13.1 #9

Merged

Conversation

webknjaz
Copy link
Member

bcrypt now relies on setuptools-rust causing pypa/pip#6264, plus it needs a Rust compiler that is unavailable in our FreeBSD VMs.
This should fix failures like https://dev.azure.com/ansible/ansible/_build/results?buildId=52061&view=logs&j=96d98051-de5a-5846-d6c0-98e63dcdbfd9&t=c2476f69-8c1f-561a-4f25-a218e506c05f&l=2635.

I'm only adding FreeBSD 13.1 to the matrix since this problem doesn't manifest itself in other VMs.

Comment on lines +6 to +11
- platform_tag: freebsd_12_2_release_amd64
platform_instance: freebsd/12.2
platform_arch: x86_64
python:
- tag: cp38
abi: abi3
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FreeBSD 12.2 is only used in stable-2.12 and stable-2.11 -- not devel. Normally we wouldn't add a non-current version here, but this is easier than manually running a build. We can remove it from the matrix once it's built.

wheel_matrix.yml Outdated Show resolved Hide resolved
wheel_matrix.yml Outdated Show resolved Hide resolved
@webknjaz webknjaz requested a review from mattclay August 25, 2022 18:46
@mattclay mattclay merged commit c3bb36b into ansible:main Aug 25, 2022
webknjaz added a commit to webknjaz/ansible--spare-tire that referenced this pull request Aug 25, 2022
webknjaz added a commit to webknjaz/ansible--spare-tire that referenced this pull request Aug 25, 2022
mattclay pushed a commit that referenced this pull request Aug 25, 2022
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

Successfully merging this pull request may close these issues.

2 participants