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 2/3 compatibility and make able to standalone install inside virtualenv #10

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

nazrulworld
Copy link

Background

First I was facing problem install this package inside my project under virtual environment (cryptography and asn1crypto are not installed globally). I know the reason (kind of egg, hen problem): in the setup.py file you made install_requires for those packages, that's fine and also version is coming from smime/init.py that's good but in that file you also import .encrypt.encrypt as a result during installing this product by pip/easy_install, getting import error.
I understand that problem can be solved by globally install or install those inside virtualenv first but I think those options are not that good idea

What I Changes

  1. move .encrypt.encrypt from init.py to newly created module api.py [solved import error issue]
  2. added python2/3 compatibility inside encrypt.py file.
  3. Pipenv file added but those are optional but would very helpful for development

…em while installing this product in virtualenv because of cryptography and asn1crypto dependencies.

* python2/3 compatiblity added in encrypt module
* pipenv introduced for development for easy package management
* tests are made pass due to recent changes
…bject,Bcc,Cc those are included into smime message
* say hello to pytest implementation
* travis file added
* manifest and Changes rst file added
@balena
Copy link
Owner

balena commented Jul 23, 2019

Hi, I've seen your changes now, sorry the delay. I understand all you did, but as Python 2 support is ending its life in January 1st, 2020, I don't think this effort will worth it.

Let me know if you have a different opinion. The case is that many changes have been made, I'm looking for just little and direct changes over the original code, as developing several different unit tests covering everything is particularly time-consuming, and I have no time to do that.

@nazrulworld
Copy link
Author

Hi, @balena thanks for your reply,
Actually that time, I needed a python 2 support in my project,
yes, I agree with about end life of python 2, I appreciate the python3 only approach.

Another thing, do you have any plan to support S/MIME signed and encrypted messages?
pyca/cryptography#1621

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.

2 participants