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

admbrowser.py needs a "crunch bang" statement #21

Closed
stroobandt opened this issue Nov 6, 2022 · 1 comment
Closed

admbrowser.py needs a "crunch bang" statement #21

stroobandt opened this issue Nov 6, 2022 · 1 comment

Comments

@stroobandt
Copy link
Contributor

admbrowser.py needs the "crunch bang" statement #!/usr/bin/env python to render it executable on *nix systems as per documentation.

The documentation also needs editing as it mentions browser.py, whereas it should read admbrowser.py.

#!/usr/bin/env python is preferred over #!/usr/bin/python as this is less dogmatic about the organisation of the file system. For example, the first statement will work on BSD systems, whereas the second will not.

I have prepared two pull requests that resolve these issues:

@alandmoore
Copy link
Owner

alandmoore commented Nov 7, 2022

Fixed by 5d9826d

Thanks!

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

2 participants