Skip to content

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