-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Are you able to build cffi==1.15.1? #2847
Comments
Which branch are you using? Generally you shouldn't state a version for packages that are obtained through a recipe. With the noted cffi, what you are likely seeing is the patch for version 1.13.2 from the master branch, opposed to the develop branch which has the updated patch for 1.15.1 |
Thanks for the hint. I was indeed using the master branch. But now it crashes town the line.
Why not pin it?! This makes a build reproducible. This is especially important for Bitcoin related software and products. |
@xavierfiechter Please post full log. |
Sure, but the output is too long to paste it here. Here's the build log https://github.com/Bitcoin-Brainbow/Brainbow/actions/runs/5444846600/jobs/9903255128 |
@xavierfiechter in logs I see build is failing because of jnius build. You can fix that by changing your workflow to:
|
Unfortunately, the buildozer image available on docker hub is definitely old and not compatible with newer As suggested by @T-Dynamos , you can change your workflow in order to avoid the usage of the docker image. Spoiler: We have planned to start re-deploying images for
Some packages, like |
Thank you @misl6 and @T-Dynamos for your reply. I will check that out and report back. I'm aware of the patched version in the recipe, and I have contributed a patch for "aiohttp" in the past. I usually go along with the latest patched version within the recipes to keep up to date. There are so many moving parts and the build process breaks occasionally. |
The build is failing. Here's the build file: https://github.com/Bitcoin-Brainbow/Brainbow/blob/master/.github/workflows/build.yml#L35 Here's the log output:
|
@xavierfiechter This line is wrong path: ${{ steps.buildozer.outputs.filename }} (in upload artifact) |
Thanks @T-Dynamos , but computer says no. https://github.com/Bitcoin-Brainbow/Brainbow/actions/runs/5453111703/jobs/9921539537
|
@xavierfiechter these lines are missing from your spec.
add them |
All these lines are already in my spec. https://github.com/Bitcoin-Brainbow/Brainbow/blob/master/buildozer.spec#L471-L477 |
The original issue is a newer version of Cython is required
https://github.com/Android-for-Python/Android-for-Python-Users#error-expression-is-not-assignable As specified in the install docs |
#Cython==0.29.32 -> Cython==0.29.33 # kivy/python-for-android#2847
@RobertFlatt this build is failing with Cython==0.29.33 as well. Here's the log: |
I suggest this is unrelated to Cython, you might want to check if buildozer.spec is truncated. Which I see is also a suggestion above #2847 (comment) |
Thanks, Robert, for your response. Those lines are in buildozer.spec, here: https://github.com/Bitcoin-Brainbow/Brainbow/blob/master/buildozer.spec#L474-L480 Any other ideas? Hints? |
Should be python3 in requirements and the Cython version as mentioned is in relation to the build system ie addressed in build.yml and I would use version 0.29.36 |
Not sure if I understand it correctly. I took the Python version from https://github.com/Bitcoin-Brainbow/Brainbow/blob/master/.github/workflows/build.yml#L62 and added it into the buildozer.spec under requirements. Also added the suggested version of Cpython.
Getting this error from this build https://github.com/Bitcoin-Brainbow/Brainbow/actions/runs/5685921100/job/15411794780. I assume the error is triggered by something else, because both lines |
You have syntax errors in your buildozer.spec The syntax is .ini syntax See https://github.com/Android-for-Python/Android-for-Python-Users#changing-buildozerspec |
I used the format of the file for 100s of builds. Because I can't find any anomaly in the file, I just used this script to read from my buildozer.spec locally.
I had no issue, so it must be something else. But what? https://github.com/Bitcoin-Brainbow/Brainbow/actions/runs/5689024819/job/15419807716 |
I can only explicity paste this from the link previously provided
Given the last item, a deterministic test is probably not meaningful. I'm obviously not going to look for all the issues in your code, but for example |
Build works when using p4a release-2022.12.20. |
👋 We use the issue tracker exclusively for bug reports and feature requests. However, this issue appears to be a support request. Please use our support channels to get help with the project. If you're having trouble installing or using python-for-android, maybe you could be interested in our quickstart guide. Let us know if this comment was made in error, and we'll be happy to reopen the issue. |
Somehow, build fails due to this patch: 5ff00ab
The text was updated successfully, but these errors were encountered: