Skip to content

Delete _os_log_set_nscf_formatter related lines to make the framework run on macOS 10.11 #784

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

Merged

Conversation

ikesyo
Copy link
Member

@ikesyo ikesyo commented Jan 11, 2017

The symbol does exist in libSystem.B.dylib on 10.12 but does not on 10.11 (which was introduced in #709).

The issue should have been happening in swiftlang/swift-corelibs-xctest#176 and swiftlang/swift-corelibs-xctest#182.

The fix is based on this: https://developer.apple.com/library/content/documentation/MacOSX/Conceptual/BPFrameworks/Concepts/WeakLinking.html.

… macOS 10.11

The symbol does exist in libSystem.B.dylib on 10.12 but does not on 10.11.
@modocache
Copy link
Contributor

@swift-ci please test

@parkera
Copy link
Contributor

parkera commented Jan 11, 2017

Actually - please just delete these lines.

@parkera
Copy link
Contributor

parkera commented Jan 11, 2017

CFRuntime.c 958-961 and 1190-1192

@ikesyo ikesyo changed the title Weakly link _os_log_set_nscf_formatter to make the framework run on macOS 10.11 Delete _os_log_set_nscf_formatter related lines to make the framework run on macOS 10.11 Jan 11, 2017
@ikesyo
Copy link
Member Author

ikesyo commented Jan 11, 2017

@parkera Okay, done in c1dfbb3.

@parkera
Copy link
Contributor

parkera commented Jan 11, 2017

Cool, thanks.

@parkera
Copy link
Contributor

parkera commented Jan 11, 2017

@swift-ci please test and merge

@swift-ci swift-ci merged commit 892fe1c into swiftlang:master Jan 11, 2017
@ikesyo ikesyo deleted the weakly-link-_os_log_set_nscf_formatter branch January 11, 2017 23:42
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.

4 participants