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 compatability #71

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

Python 3 compatability #71

wants to merge 1 commit into from

Conversation

benjsec
Copy link

@benjsec benjsec commented Nov 6, 2017

This is an updated to PR #69, with commits squashed to present a cleaner history. It still addresses issue #36.

This updates to include compatibility with py3, whilst retaining
all functionality in py2.

The evernote library is not yet py3 compatible, so is not used in
the py3 version. It is however still retained when using py2.

Also fabric isn't compatible with py3 yet either. I haven't worked with it before, so am not sure how important it is to the dev workflow here. For now I've changed tests to run directly instead of via the fabric generated runtests.py.

This updates to include compatability with py3, whilst retaining
all functionality in py2.

The evernote library is not yet py3 compatible, so is not used in
the py3 version. It is however still retained when using py2.
@jamesbroadhead
Copy link

(I am not a core contributor)

This looks like a good PR, but has a lot of unrelated changes along with the functional ones. It would be easier to review if these were separate.

both evergreen & fabric forked for python3 rather than supporting both versions - I believe that evernote3 and fabric3 could be dropped in & directly used here.

@benjsec
Copy link
Author

benjsec commented Aug 5, 2018

This PR is from quite a while ago now, the maintainer seems to have abandoned this project sadly. I did continue work on this on my own fork but don't have much time for it now either.

@twitu
Copy link

twitu commented Mar 3, 2019

The evernote dependency is not allowing installation for python3 is there anyway to ignore the dependency because I don't need its functionality

@mdbiz
Copy link

mdbiz commented Mar 30, 2019

@twitu
Even if you remove evernote dependency it still doesn't work on Py3 for other reasons.

you could use @benjsec fork which adds Py3 compatibility

or try to use my fork. I completely removed evernote. You have install instructions in the fork.

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.

4 participants