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.

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

[R.I.P.] Remove python2legacy and hospython2legacy #1831

Merged
merged 1 commit into from
Jun 2, 2019

Conversation

opacam
Copy link
Member

@opacam opacam commented May 31, 2019

Because:

  • pygame has been removed (one of the purposes of this recipe was to make it work that bootstrap, alongside to be able to target api levels below 21)
  • the end of life of Python 2 it's near (2020)
  • we still have the python2 recipe (version 2.7.15)

Caveats:

  • We loose the ability to target api levels below 21 (Android 5.0 - Lollipop)

See also:

If the code you care about is still on Python 2...the clock is ticking...https://pythonclock.org/

As a side note: I don't think that we can get a green tick mark with this pr in travis...this pr has all the numbers to fail with our CI test rebuild_updated_recipes...similar than the recently merged pr about the pygame removal

Because:
  - pygame has been removed (one of the purposes of this recipe was to make it work that bootstrap, alongside to be able to target api levels below 21)
  - the end of life of Python 2 it's near (2020)
  - we still have the python2 recipe (version 2.7.15)

Caveats:
  - We loose the ability to target api levels below 21  (Android 5.0 - Lollipop)
@inclement
Copy link
Member

I have no objections to this. I'll merge as soon as I have time to review it, or if others are looking at it then go ahead as far as I'm concerned :)

Thanks @opacam

Copy link
Member

@inclement inclement left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks solid, thanks again @opacam :)

@inclement inclement merged commit ff42ca5 into kivy:master Jun 2, 2019
@AndreMiras
Copy link
Member

Nice to see it goes 👋
Funnily my tox tests/test_graph.py were failing until I deleted the empty directories left over after rebasing to master.
The error was:

AttributeError: module 'pythonforandroid.bootstraps.pygame' has no attribute 'bootstrap'

I had to delete the following empty directories:

  • pythonforandroid/recipes/pygame*
  • pythonforandroid/recipes/*python2legacy

@opacam opacam deleted the rip-python2-legacy branch June 15, 2019 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants