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

Replace bashisms with POSIX syntax #1006

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeroen-mostert
Copy link

This replaces some bashisms (notably equality check with == and substring replacement) with POSIX equivalents, for distros where sh operates in strict POSIX mode. I will not claim that all bashisms are thereby purged from the configure files (they may lurk in conditionals not taken), but at least it allows a basic build to proceed.

@swx-jenkins3
Copy link

Can one of the admins verify this patch?

@manjugv
Copy link
Contributor

manjugv commented Aug 7, 2024

@jeroen-mostert What is your affiliation? Thanks.

@jeroen-mostert
Copy link
Author

jeroen-mostert commented Aug 7, 2024

This patch is a product of my spare time, neither commissioned by, nor approved of, nor sponsored by, nor produced during working hours, nor aligned with the purposes of the company I also happen to be employed for. I will not sign any document, waiver, contract or agreement to release any rights associated with it, but release this code into the public domain as per the CC0 universal deed, including but not limited to giving others permission to use the same code for merging purposes without attribution. I hope that answers your question.

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

Successfully merging this pull request may close these issues.

3 participants