alandmoore / admbrowser

A Web browser for kiosks and digital signage, based on Python, PyQt, and Blink
35 stars 22 forks source link

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

Closed stroobandt closed 1 year ago

stroobandt commented 1 year ago

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 commented 1 year ago

Fixed by 5d9826d

Thanks!