Skip to content

Commit

Permalink
Retag wheels to be cpXY (#644)
Browse files Browse the repository at this point in the history
Fixes #641

After building the wheels, this...

* Creates the tag string from `cp` (CPython) and the `python`'s version
* Uses the tag string to replace the `python_tag` and `abi_tag`
* Removes the previous wheels while performing this conversion
* Hands them to `auditwheel` for final steps

Authors:
  - https://github.com/jakirkham

Approvers:
  - Gregory Lee (https://github.com/grlee77)
  - Vyas Ramasubramani (https://github.com/vyasr)
  - Bradley Dice (https://github.com/bdice)
  - Ray Douglass (https://github.com/raydouglass)

URL: #644
  • Loading branch information
jakirkham authored Nov 29, 2023
1 parent 8f2a96e commit 12ec30b
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions ci/build_wheel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,14 @@ cd "${package_dir}"

python -m pip wheel . -w dist -vvv --no-deps --disable-pip-version-check

# Because we built the library with the run script and manually copied the shared pybind11
# library, the subsequent py_project.toml wheel build was as a pure Python package and results in
# tags that incorrectly indicate it is a universal wheel. To fix this, we need to modify the wheel
# to have CPython ABI and Python tags matching the version of Python that the Python bindings were
# built with.
WHEEL_PYTHON_TAG=cp$(echo ${RAPIDS_PY_VERSION} | sed 's/\.//g')
python -m wheel tags --remove --python-tag="${WHEEL_PYTHON_TAG}" --abi-tag="${WHEEL_PYTHON_TAG}" dist/*

mkdir -p final_dist
python -m auditwheel repair -w final_dist dist/*

Expand Down

0 comments on commit 12ec30b

Please sign in to comment.