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

Fix shebangs in utils #750

Closed
wants to merge 1 commit into from
Closed

Conversation

jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Oct 28, 2023

Shebang need to be on the first line or the script will fail to execute on some systems:

make: ./utils/check_tests_enabled.sh: No such file or directory

Shebang need to be on the first line or the script will fail to execute on some systems:

    make: ./utils/check_tests_enabled.sh: No such file or directory
@robe2 robe2 requested a review from strk November 26, 2023 02:28
Copy link
Member

@strk strk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I don't know how I missed this (not approving to avoid the temptation of clicking the green button, as we're using git.osgeo.org/gitea/postgis as the official forge)

@jtojnar
Copy link
Contributor Author

jtojnar commented Nov 26, 2023

Should I reopen this there now that I have OSGeo account?

@robe2
Copy link
Member

robe2 commented Nov 27, 2023

Should I reopen this there now that I have OSGeo account?

Not necessary. I'll commit shortly to 3.3 - 3.5 branches

@robe2 robe2 closed this in 69fdc99 Nov 27, 2023
@robe2
Copy link
Member

robe2 commented Nov 27, 2023

I guess these were new so no need to add to prior branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants