-
Notifications
You must be signed in to change notification settings - Fork 448
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
Python3 migration #620
Python3 migration #620
Conversation
Following the discussions in #542 regarding not using macOS system Python, I've done what I suggested and added a I've merged your changes for Python 3 support too so looks like we finally have fix?! |
yeah i quite like the i was thinking alternatively to just patching the shebang to |
I considered this too. I guess the problem with this approach is people using the repository clone. |
If I understand correctly, |
@happyaron if only it were that simple: see #542 (comment); |
so if we use it'll still work on linux without hardcoding the shebang (git checkout) too. |
Yes, I think that is actually the simplist/best solution. |
… the rpm specfile (debian tbc)
ok done - reset to using will have to think about the debian patch but that's outside of this repo anyway. |
Fixes #542