dopy icon indicating copy to clipboard operation
dopy copied to clipboard
verified publisher icon Wiredcraft

Replace basestring with str type to support Python3

Open srkiNZ84 opened this issue 9 years ago • 6 comments

Python3 does not have a "basestring" type (as per: https://stackoverflow.com/questions/34803467/unexpected-exception-name-basestring-is-not-defined-when-invoking-ansible2) and the recommended action is to replace it with the "str" type.

This was the only instance of it that I managed to find in the repository.

Issue(s): None

srkiNZ84 avatar Jan 23 '17 08:01 srkiNZ84

+1

ghost avatar Aug 07 '17 14:08 ghost

Is there any chance this could get merged any time soon?

DakotaNelson avatar Aug 22 '18 06:08 DakotaNelson

Hey there, Ansible is currently using dopy for their Digital Ocean modules. Is there any way we can get this fix published? Currently even trying to create a droplet from Python3 throws an exception.

jbaublitz avatar Oct 13 '18 17:10 jbaublitz

ping :)

ioggstream avatar Sep 04 '20 14:09 ioggstream

I haven't got the permissions to merge this PR and it looks like the repository hasn't been committed to since 2016. Think the Ansible module might need to change the dependency or find a different DigitalOcean library to use.

srkiNZ84 avatar Sep 06 '20 23:09 srkiNZ84

@srkiNZ84 we can get in touch with wirecraft or ask to pypi to provide another source for the package distribution...

ioggstream avatar Sep 07 '20 07:09 ioggstream