search

dbcli / litecli

CLI for SQLite Databases with auto-completion and syntax highlighting
https://litecli.com
BSD 3-Clause "New" or "Revised" License
2.12k stars 68 forks source link

chmod a-x litecli/main.py #68

Closed arp242 closed 5 years ago

arp242 commented 5 years ago

Because this file is marked as executable I tried to run it:

    ./litecli/main.py

But there is no hashbang (#!/usr/bin/env python) so it got run as a shell script. It printed out some warnings and then crashed my system. Do'h >_<

Remove the executable flag since this running this file with "python2" or "python3" doesn't seem to work, and may prevent someone in the future from making the same mistake.

amjith commented 5 years ago

Sorry you were misled by this. I'm happy to merge the PR.

I'm curious to know why you were trying to run the file. Were you trying to make changes and see if you could run your changes after you've made them?

If you're interested in doing development on this project you can try the contribution guide which has instructions on how to get started with local development.

:clinking_glasses:

arp242 commented 5 years ago

Yeah, I just wanted to run it for doing other modifications; didn't feel like mucking about with all that virtualenv stuff; it often works with many projects. I ended up using that anyway.