Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Eliminate ASL-based stdout forwarding on iOS#3797

Merged
cbracken merged 1 commit into
flutter-team-archive:masterfrom
cbracken:syslog-revisited
Jun 20, 2017
Merged

Eliminate ASL-based stdout forwarding on iOS#3797
cbracken merged 1 commit into
flutter-team-archive:masterfrom
cbracken:syslog-revisited

Conversation

@cbracken

Copy link
Copy Markdown
Contributor

ASL was deprecated in iOS 10 and started causing SIGPIPE issues in iOS
10.3. Under the iOS 8 SDK, syslog() stopped working as of iOS 10.3
devices, with the result that ASL stdout/stderr forwarding was the only
means of logging. The engine now builds against the iOS 10 SDK, with
deployment target of iOS 8. Under this SDK, syslog() works correctly
across all supported OS versions.

NOTE: This is a temporary fix to get developers unblocked. While this
does fix the SIGPIPE issue and put iOS logging on par with the Android
solution, the intent is to move to a dedicated communication channel
with flutter_tools that isn't log-based.

@cbracken

cbracken commented Jun 20, 2017

Copy link
Copy Markdown
Contributor Author

Short-term fix to flutter/flutter#10358.

@tvolkert tvolkert left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

ASL was deprecated in iOS 10 and started causing SIGPIPE issues in iOS
10.3. Under the iOS 8 SDK, syslog() stopped working as of iOS 10.3
devices, with the result that ASL stdout/stderr forwarding was the only
means of logging. The engine now builds against the iOS 10 SDK, with
deployment target of iOS 8. Under this SDK, syslog() works correctly
across all supported OS versions.

NOTE: This is a temporary fix to get developers unblocked. While this
does fix the SIGPIPE issue and put iOS logging on par with the Android
solution, the intent is to move to a dedicated communication channel
with flutter_tools that isn't log-based.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants