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

Use /usr/bin/env python instead of /usr/bin/python in the shebang #9

Open
ghost opened this issue Nov 7, 2015 · 1 comment
Open

Comments

@ghost
Copy link

ghost commented Nov 7, 2015

Hi,

If you use /usr/bin/env python instead of /usr/bin/python in the shebang line for each of your source files, this script will be a bit more portable. Specifically, it will work properly if someone tries to wrap it in a virtualenv, like I did :)

Here's the super simple .diff that worked for me

diff --git a/mnetsuite/graph.py b/mnetsuite/graph.py
index fd00566..e7a35ee 100755
--- a/mnetsuite/graph.py
+++ b/mnetsuite/graph.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python

 '''
        MNet Suite
diff --git a/mnetsuite/node.py b/mnetsuite/node.py
index a8a7566..f8d46e6 100755
--- a/mnetsuite/node.py
+++ b/mnetsuite/node.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python

 '''
        MNet Suite
diff --git a/mnetsuite/snmp.py b/mnetsuite/snmp.py
index 31a4d49..efd2071 100755
--- a/mnetsuite/snmp.py
+++ b/mnetsuite/snmp.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python

 '''
        MNet Suite

Thanks!

@CalebFenton
Copy link

This is (partially) fixed here #25

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

No branches or pull requests

1 participant