search

sydro / atom-ansible-vault

Atom package to create and modify ansible-vault file
MIT License
32 stars 10 forks source link

Python 3 #15

Closed certifiedloud closed 7 years ago

certifiedloud commented 7 years ago

Use future to make this package support python 2.x and 3.x simultaneously. The print statements in the python files cause this to fail on systems that use python 3 by default.

kinnarr commented 7 years ago

It's also possible to use #!/bin/env python2 instead of #!/bin/env python in lib/bin/vault_wrapper.py.

cinemast commented 7 years ago

Could the shebang be changed? The package is currently unusable in arch linux.

sydro commented 7 years ago

does this #!/bin/env python shebang not work in arch linux?

kinnarr commented 7 years ago

In arch linux python 3 is the default, so #!/bin/env python resolves to python3 instead of python2

cinemast commented 7 years ago

That is exactly the problem. If you require python2 please also specify python2 in the shebang. @sydro could you fix this please and prepare a new release?

sydro commented 7 years ago

@cinemast i fix shebang to #!/usr/bin/env python2 to restore wrapper with python2 in 5710987 commit . i'm working to made it compatible with python3. Thanks

sydro commented 7 years ago

I removed the python libs dependencies. Now en/decrypter is only javascript, and it call ansible-vault binary.