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

Disable TLS support for iOS. #186

Merged
merged 2 commits into from Jul 25, 2016
Merged

Disable TLS support for iOS. #186

merged 2 commits into from Jul 25, 2016

Conversation

wilx
Copy link
Contributor

@wilx wilx commented Jul 25, 2016

No description provided.

dmigous and others added 2 commits July 24, 2016 17:29
configure.ac: added conditional around AX_TLS_SUPPORT to not call it if
iOS target system

configure.ac: added comment about TLS check guards for iOS
Fix braces nesting. Regenerate the `configure` script.
@wilx wilx self-assigned this Jul 25, 2016
AX_TLS_SUPPORT
dnl Check if TLS is supported, but only if it isn't iOS target, because it
dnl is forbidden by Apple Store to use __tlv_* functions.
AS_IF([test "$target_os" != "darwin" -o "$target_cpu" != "arm",
Copy link
Contributor Author

@wilx wilx Jul 25, 2016

Choose a reason for hiding this comment

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

@dmigous Just FYI, I have caught a bug in this change: The [] are wrongly balanced. This is fixed by 7b5f53d. Next time, please run ./scripts/doautoreconf.sh and inspect the resulting diff as well. :)

@wilx wilx merged commit d8bfde3 into log4cplus:master Jul 25, 2016
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.

None yet

2 participants