Add shebang to bash scripts
|Target version:||Candidate for next minor release|
gmt_uninstall.sh scripts don't have a shebang, but are expected to be executed as bash scripts.
This caused executable-not-elf-or-script lintian issues for the Debian package build.
The attached patch fixes the issue.
#3 Updated by Florian over 1 year ago
Should a similar fix be applied to share/tools/gmt_completion.bash?
I added #!/bin/csh to the two cshell scripts. OK?
No, you must not add the shebang to any script that has to be sourced and changes the environment of the current shell. With the shebang a subshell is started and you would break the feature.