This repository has been archived by the owner on Jun 4, 2021. It is now read-only.

Fix encoding bug when calling Popen.communicate #138

Closed

peterebden wants to merge 1 commit into google:master from peterebden:master

peterebden commented Mar 1, 2019

communicate expects a bytes-like thing for input; was erroring out with

memoryview: a bytes-like object is required, not 'str'

Depends on the destination repo whether that codepath is hit or not though so doesn't happen every time.


          Fix encoding

d06b422

Contributor

jonjohnsonjr commented Mar 5, 2020

Landed in #160

jonjohnsonjr closed this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet