Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use latest Buildozer.spec #5

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Use latest Buildozer.spec #5

wants to merge 4 commits into from

Conversation

Julian-O
Copy link
Contributor

@Julian-O Julian-O commented Nov 24, 2023 •
edited
Loading

Use latest Buildozer.spec, reapplying same changes, plus:

  • osx.kivy_version now 2.2.1
  • accepts android.accept_sdk_license = True
  • Cython version restricted (will be installed by Buildozer in its next release)

Note: android.archs is now new default

This fixes #4 (complaints about android.arch and aidl being missing), and avoids new Cython version issues.

@Julian-O
Use latest Buildozer.spec
42295bb 
Reapply same changes, plus:

osx.kivy_version now 2.2.1
accepts android.accept_sdk_license = True
Cython version restricted (will be installed by Buildozer in its next release)

Note: android.archs now new default
@misl6
Copy link
Member

misl6 commented Nov 24, 2023

@Julian-O , if you're wondering why the CI is failing, is due to the installed Cython version (which should be <3)

@Julian-O
Copy link
Contributor Author

@Julian-O , if you're wondering why the CI is failing, is due to the installed Cython version (which should be <3)

One day, GitHub will not make me cry. Today is not that day.

@misl6
Copy link
Member

misl6 commented Nov 24, 2023

One day, GitHub will not make me cry. Today is not that day.

😅 The Cython version that should be pinned is not the one in requirements, but the one python-for-android is using to Cythonize the sources (aka the one that is installed into the same python environment which runs buildozer or p4a)

@Julian-O
Revert to hardcoded p4a
cf8952c 
CI Build depends on 3rd Party GitHub build action that depends on obsolete Docker image on DockerHub that still includes
old version of Cython that doesn't work with stable or master versions of p4a.
@Julian-O
Copy link
Contributor Author

Best I can figure: The CI Build depends on ArtemSBulgakov/buildozer-action which depends on the latest DockerHub image which is obsolete and includes old version of Cython which causes build errors when used with recent versions of p4a.

Solutions include:

In the meantime: locked down the p4a version to an old version to get this building.

@Julian-O Julian-O requested a review from misl6 November 28, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@misl6 misl6 Awaiting requested review from misl6

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Building failing due to obsolete build.spec
2 participants
@Julian-O @misl6