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

Python 3 migration required for Android builds #462

Merged
merged 1 commit into from
Jun 5, 2021

Conversation

cbracken
Copy link
Member

@cbracken cbracken commented Jun 5, 2021

This migrates all Python scripts required to build Android targets to be
Python 3.x compatible.

Main changes:

  • Communication with processes now returns bytestreams by default. By
    specifying universal_newlines=True, these streams are decoded as UTF-8.
  • md5.update() takes a bytes type, but strings are now utf-8 by default.
    We call string.encode('utf-8') convert to bytes.
  • print now uses function syntax

Issue: flutter/flutter#83043

This migrates all Python scripts required to build Android targets to be
Python 3.x compatible.

Main changes:
* Communication with processes now returns bytestreams by default. By
  specifying universal_newlines=True, these streams are decoded as UTF-8.
* md5.update() takes a bytes type, but strings are now utf-8 by default.
  We call string.encode('utf-8') convert to bytes.
* print now uses function syntax

Issue: flutter/flutter#83043
@cbracken cbracken merged commit 525709f into flutter:master Jun 5, 2021
@cbracken cbracken deleted the python3-android branch June 5, 2021 21:18
richkadel pushed a commit to richkadel/buildroot that referenced this pull request Jun 25, 2021
This migrates all Python scripts required to build Android targets to be
Python 3.x compatible.

Main changes:
* Communication with processes now returns bytestreams by default. By
  specifying universal_newlines=True, these streams are decoded as UTF-8.
* md5.update() takes a bytes type, but strings are now utf-8 by default.
  We call string.encode('utf-8') convert to bytes.
* print now uses function syntax

Issue: flutter/flutter#83043
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants