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

chmod a-x litecli/main.py #68

Merged
merged 1 commit into from Aug 4, 2019

Conversation

@arp242
Copy link
Contributor

commented Aug 3, 2019

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.

chmod a-x litecli/main.py
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

This comment has been minimized.

Copy link
Member

commented Aug 4, 2019

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.

🥂

@amjith amjith merged commit 288ae91 into dbcli:master Aug 4, 2019

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@arp242 arp242 deleted the arp242:chmod branch Aug 4, 2019

@arp242

This comment has been minimized.

Copy link
Contributor Author

commented Aug 4, 2019

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.