-
-
Notifications
You must be signed in to change notification settings - Fork 119
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
failed to build under android #195
Comments
Hm the relevant part appears to be:
Are you sure this currently works with v1.9.1? It sounds like this could be caused by termux/termux-packages#7788. |
The build system changed in between 1.9.1 and 2.0.0 from setuptools to scikit-build + cmake. So I assume, that cmake is invoking clang with different command line arguments. |
yes i did
|
Hm, looking at this again it appears to build cmake from source and is triggered by https://github.com/scikit-build/cmake-python-distributions/blob/0895db3c6de81da3091fc03dcc425ae8d53357a3/CMakeLists.txt#L193-L196. So you should run into the same issue when running It appears like this is an issue with the clang packaged in termux, so it would make sense to report it to them as well. I could not find any open issues on this over in the termux issue tracker for clang 13.0.1, even though similar issues existed for clang 12.0.1 (termux/termux-packages#7788). Maybe one of the workarounds for clang 12.0.1 works here as well. @henryiii linking you here, since this might be of interest to you given your one of the maintainers of the cmake distribution for Python and scikit-build, so others could run into similar issues when transitioning. |
That's weird, why is it downloading ninja but building cmake? Also, why is Guessing android is not manylinux? What version of pip is that? |
The latter has been downgraded accordingly to the latter. rapidfuzz/RapidFuzz#195 (comment)
building in termux works with v2.0.13, as long as you have cmake and ninja natively installed. |
after v1.9.1 it started to fail in building using pip under android (termux)
logs: log.txt
The text was updated successfully, but these errors were encountered: