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

I updated your script to use Python 3, please accept pull request #1

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

Conversation

pklimai
Copy link

@pklimai pklimai commented Aug 27, 2016

I fixed the script to work for both Python3 and Python2.

Also added readme to stress that single-quotes must be used when launching script. Othewise BASH tries to substitute $vars and script crashes.

pklimai and others added 11 commits August 27, 2016 23:11
Include msuksong's changes for encrypting, as well
The Python 3 argparse module does not appear to support the "version" named argument in the same way that Python 2 does.   Removing "version" argument.
Added back a Python 3 compatible --version option, added a --result-only option to display just the converted string, without filler text or source string.
Updated README with new flag and examples
Include msuksong's changes for encoding, update for Python 3
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