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

Fixed deprecated getargspec() function for Python 3.x #297

Closed
wants to merge 1 commit into from

Conversation

ctingwai
Copy link

@ctingwai ctingwai commented Jun 2, 2014

A fix for Python 3.x when trying to run aws-cli tool "ValueError: Function has keyword-only arguments or annotations, use getfullargspec() API which can support them" error

@quiver
Copy link
Contributor

quiver commented Jun 3, 2014

@ctingwai This is a Pull-Request against aws/aws-cli#800, right?

@ctingwai
Copy link
Author

ctingwai commented Jun 3, 2014

@quiver Yeah. That's the same stack trace I got.

@jamesls
Copy link
Member

jamesls commented Jun 3, 2014

Interesting, it looks like py3.4 breaks backwwards compat with getargspec. Taking a look.

@jamesls
Copy link
Member

jamesls commented Jun 4, 2014

Ok this looks like this happens only in py3.4.1. We abstract py2/3 differences in our compat.py module, so this should probably be in the compat module. I also am going to update our .travis.yml config to run tests against py3.4 as well. Should be fixed soon. Thanks for the investigation.

jamesls added a commit to jamesls/botocore that referenced this pull request Jun 4, 2014
There was a fix needed for python3.4.1,
as shown in boto#297

This also updates travis/tox to run tests against py3.4

Also verified the CLI tests pass.
@jamesls jamesls mentioned this pull request Jun 4, 2014
@ctingwai
Copy link
Author

ctingwai commented Jun 4, 2014

Sure, it's always my pleasure to be of help.

@jamesls
Copy link
Member

jamesls commented Jun 6, 2014

Fixed via #300

@jamesls jamesls closed this Jun 6, 2014
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