search

calliope-edu / CalliopEO_AstroPi

MIT License
1 stars 2 forks source link

Add shebang #!/usr/bin/env python3 #49

Closed rzbrk closed 3 years ago

rzbrk commented 3 years ago

Add shebang #!/usr/bin/env python3 and make chmod +x CalliopEO.py. Then, the script can be executed with ./CalliopEO.py.

rzbrk commented 3 years ago

See fix 5b84dbd.

Can be closed if #50 is merged.

Amerlander commented 3 years ago

shall we then add chmod +x ./CalliopEO.py in the setup script?

rzbrk commented 3 years ago

Do we really need this? The permissions are usually also stored in the repo. For example, this works in my case for setup.sh and testing.sh. Do you have to set the executable flag for every file?

Amerlander commented 3 years ago

I have to do this for the .sh files. The .py script executes with python3 CalliopEO.py without having chmod +x. So I have to do this for the setup script and then for the testfile (where I think it is okay do do not have this executeable by default)

But I download the Zip file, unpack it localy and then upload everything via ssh. I'll test what happens when I upload the Zip and unpack it on the pi.

Amerlander commented 3 years ago

The test files itself run without being executeable

rzbrk commented 3 years ago

yes, the testcase files do not need the executable flag, only testing.sh in the main directory.

maybe, by copying the individual files via ssh the flag gets lost. but zip also stores them.

Amerlander commented 3 years ago

yes, using the zip works without chmoding anything