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

just corrected shebang #12

Merged
merged 2 commits into from
Aug 7, 2020
Merged

Conversation

alexandregz
Copy link
Contributor

I use gomp.py (legacy) without alias, so:

alex@vosjod::~/bin $ ./gomp
-bash: ./gomp: usr/bin/env: bad interpreter: No such file or directory

alex@vosjod::~/bin $ vi gomp
alex@vosjod::~/bin $ ./gomp
usage: gomp [-h] [--key] [--recut] [src] dest
gomp: error: too few arguments

@philipkiely-markforged
Copy link

Thanks for the PR. Would it be possible to change in accordance with https://stackoverflow.com/questions/6908143/should-i-put-shebang-in-python-scripts-and-what-form-should-it-take/19305076#19305076

to use #!/usr/whatever python2 and #!/usr/whatever python3 as appropriate?

Also, could you please format the commit message as [PATCH] what this PR accomplishes

@jeremy-mf jeremy-mf added the bug Something isn't working label Aug 6, 2020
@davidbenhaim
Copy link

Also, could you please format the commit message as [PATCH] what this PR accomplishes

@philipkiely-markforged we should add a contributions guidelines if we're going to make asks like that :)

Summary
corrected shebangs, suggested by philipkiely-markforged :-)
@alexandregz
Copy link
Contributor Author

alexandregz commented Aug 7, 2020

Done 😄

thx for your software

@philipkiely-markforged philipkiely-markforged merged commit d8718f1 into MarkForged:main Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants