sighingnow / libclang

(Unofficial) Release libclang (clang.cindex) on pypi.
https://pypi.org/project/libclang
Other
84 stars 21 forks source link

Use auditwheel to compute the platform tag #32

Closed mayeut closed 2 years ago

mayeut commented 2 years ago

libclang currently publishes a manylinux1 wheel however, it is not compatible with manylinux1. Use auditwheel to compute the correct platform tag.

@sighingnow, even with #31, the published tag is still not the right one.

sighingnow commented 2 years ago

Hi @mayeut,

Thanks for pointing me the issue. I suggest change the ABI tag to manylinux2010_x86_64 when python3 setup.py bdist_wheel should be enough.

We don't need do the auditwheel and patchelf job, IMO.

sighingnow commented 2 years ago

As we are working inside a manylinux2010 image.

sighingnow commented 2 years ago

Addressed by https://github.com/sighingnow/libclang/commit/fb2803965a6efb487d4179cb3afdb7dd283b2ac4.