-
Notifications
You must be signed in to change notification settings - Fork 139
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
Tumbleweed-aarch64 .venv build fails on gevent wheel #2693
Comments
It is assumed that pre-built wheels are available for all our other OS & arch targets; and so we do not see this same failure as not wheel build is then required. |
Pinning gevent in pyproject.toml to 23.9.0 and updating dependencies results in:
So there is no 23.9.0 released. |
Repeating last comment but for 23.9.0.post1 https://pypi.org/project/gevent/23.9.0.post1/ resulted in what looks like the same failure we had originally. |
Thanks to @FroggyFlox for assisting with this issue re:
But we also have:
And we have an embeded / vendored c library our our apparent failure so we may also just have availability of pre-build wheels variation per arch/Python version and are unable to build ourselves when one is not found due to TW compiler version or related. The most recent attempt to
However we cant' change our pyproject.toml just for one arch and in just TW. So we may have to forgo the TW-aarch target for the time being: at least until we can do our next Python update which will likely be after we do our next Django update to 3.2 given Py3.9 is latest compatible with Django 2.2. |
N.B. the only difference in the resulting poetry.lock from our pyproject at (Py3.9), as per linked draft PR, and post (py3.10) experiment, was the Python version and checksum. So it does not look like any more wheels were created and made available in the interim. |
Linking to the Wheel info fro our successful Py3.10 experiment:
And by way of comparison with our amd64 arch where on Py3.9 we did not have the issue detailed here:
|
Do you have to worry about this note on the arm64 wheels/binaries for gevent in general? https://pypi.org/project/gevent/ (highlight is mine)
|
@Hooverdan96 Thanks, nice find.
We are to remove our dependency on gevent all-together in the mid term so I think we will be OK. But for the time being I'm just trying to nudge as many dependencies along as I can, wise to trying to keep our Py and Django mostly happy. Nearly there however. Just a few stragglers to tend to and we can approach some simplification hopefully. And we also need to be moving towards the next stable fairly soon. Bit of a juggle however. |
@Hooverdan96 & @FroggyFlox
Maybe we are suffering for our now aeging Poetry here. Just noting for future reference as I'm not entirely sure of this detail as it goes; re the venv creation etc. But at least once we have our newer Python in-play we can look to a Poetry update: but that will have all of it's own challenges I suspect. And will likely have to be within our rpm scriptlets. But that is for another issue and repository. |
ok, sounds good. Then (while I asked in the other thread) for completeness sake, reading this: https://www.gevent.org/install.html
Since I don't think we've been using this "extra" anywhere in previous versions of gevent during the install., not sure whether it is indeed needed ... but probably no need to to question this further at this time, since, like you said we will likely remove a reliance on this entirely ... anyway that statement above in the documents does not make it clear to me whether that is included as a default requirement now or not. |
From our incoming poetry.lock file for the linked Py3.9 move we have the following available pre-compiled wheels:
Hence having to build our own on this Py/linux/arch target - and failing for some compatibility reason. Likewise the same incoming poetry.lock has what we end up getting in the above proof of concept Py3.10 experiment:
So all-in this should be resolved in time as we progress with our dependency updates (technical dept). |
As per the last linked PR (a Django 3.2 LTS update) we are additionally now using Django 4.2 LTS (#2752), so the remaining issue to re-gain TW aarch64 target compatibility is a Python update to at least Py3.10 give a tentative experiment in the comments here indicated a wheel available for that Python version and up. |
Closing as: |
When building for all OS targets & architectures (Leap 15.4/15.5 & TW) for PR:"Update Python dependency to 3.9 #2691" #2692 we have a single rpmbuild failure in our % check only on aarch64.
From a manual source build of the same PR specific code branch we have the following partial .venv:
We then have the following which is assumed to be from our Poetry running under our default system Python of Py3.10:
The text was updated successfully, but these errors were encountered: