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

Change encoding for json.loads #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emrahcetiner
Copy link

@emrahcetiner emrahcetiner commented Dec 22, 2019

Remove encoding argument from json.loads for support to pyhton 3.9 #19

@obilodeau
Copy link

Hey @nbering, can this patch be merged? Python 3.9 is out since early October this year and it is only a matter of time before it starts rolling out into distros.

@dverbeek84
Copy link

@nbering Are you able to get this merged? I am using @obilodeau solution to work on python 3.9

@obilodeau
Copy link

@dverbeek84, for the record, this is not my patch. It is @emrahcetiner's.

It can be quickly applied like this:

cd YOUR_TERRAFORM.PY_LOCATION
wget https://patch-diff.githubusercontent.com/raw/nbering/terraform-inventory/pull/20.diff
patch -p1 < 20.diff
rm 20.diff

@nbering: Arch Linux is now on Python 3.9. terraform.py will now issue a fatal TypeError over there. Can you please merge this patch? Thanks!

@arthurk
Copy link

arthurk commented Jan 29, 2021

Any update on this? Homebrew Python is on 3.9 and this script fails

@The-Loeki
Copy link

Fedora 34 is Py3.9 now too so it's broken there as well without this patch

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.

5 participants