python 3.13 rebuild: part 6 #28182
Draft
python 3.13 rebuild: part 6 #28182
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thunder-coding
force-pushed
the
python313-dep6
branch
from
6a02bfd to
81dd160
Compare
5 tasks
thunder-coding
force-pushed
the
python313-dep6
branch
2 times, most recently
from
97daf04 to
47a8d3c
Compare
patch for configure is no longer needed patch for multiprocessing.c no longer required due to python/cpython@fa1d675 Remove 0006-do-not-use-xattr.patch since we can just add a configure argument and it works just as well We no longer need to manually add libraries for libpython as it is now added by default when building for android. See upstream commit: python/cpython@7f5e3f0 No longer required patch for Lib/aifc.py after python/cpython@036da3b mailcap module has been removed in Python 3.13. https://docs.python.org/3/whatsnew/3.13.html#whatsnew313-pep594 add post install message when site-packages/ is detected for python3.12 Also mark as unsafe for on-device building. The plan is to move to building python with host-python, this isn't possible on device. I will be looking to get some fixes made in future to get on-device builds as well, but it is not the priority at the moment
Causes the built libpython3.so to not include symbols which we need in the final binary
maybe the issue with the build system is fixed now?
We need build python for cross compilation as Ubuntu's version of python is different. Also crossenv recommends building own python revisions instead of distribution compiled ones added lld to setup-ubuntu as it's needed by clang driver when using -fuse-ld=lld. Not sure how we went this far without having to deal with this during host builds Also we need libpython3.so for cross-compilation of some of the pip packages. So build with --enable-shared
documentation BREAKING CHANGE: This potentially breaks a lot of builds that might need python with ubuntu's packaged pip packages/additional features
Do not try to use build python for running meson
This is a slightly modified version of robotpy/crossenv@3ef761b We only need the simple shebang, so only use that always.
Also update setuptools to 80.9.0 while we are at full rebuilds with python 3.13
fixes crossenv broken
Relevant upstream commit python/cpython@1f8b24e Also apply this same fix in host build of python as cross-python uses it's own cutils instead of what we have built for target in our cross builds.
…roid api version `python -m build` builds android wheels as android_$TERMUX_ARCH.whl but pip during installation expects it to be android_$TERMUX_API_LEVEL_$TERMUX_ARCH.whl so we need make sure that it is installable without renaming the wheel
[[TODO: add more descriptive commit message]]
thunder-coding
force-pushed
the
python313-dep6
branch
from
47a8d3c to
5cdfe98
Compare
Also use ninja for building pyarrow for faster builds than unix makefiles. Also remove logic for unsetting PYTHONPATH, as it is needed for Cython to be used.
thunder-coding
force-pushed
the
python313-dep6
branch
from
5cdfe98 to
7ef9d2e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.