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

[Backport][ipa-4-8] configure.ac: don't rely on bashisms #3510

Closed
wants to merge 1 commit into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Aug 7, 2019

This PR was opened automatically because PR #3485 was pushed to master and backport to ipa-4-8 is required.

93fb037 introduced unportable shell
syntax, which led to erros like
./configure: 3179: ./configure: CFLAGS+= -D__STDC_WANT_LIB_EXT1__=1: not found
in case a posix shell is used.
@tiran
Copy link
Member Author

tiran commented Aug 7, 2019

PR was ACKed automatically because this is backport of PR #3485. Wait for CI to finish before pushing. In case of questions or problems contact @hermannch who is author of the original PR.

@tiran tiran added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Aug 7, 2019
@tiran
Copy link
Member Author

tiran commented Aug 7, 2019

ipa-4-8:

  • 6bef867 configure.ac: don't rely on bashisms

@tiran tiran closed this Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants