-
Notifications
You must be signed in to change notification settings - Fork 712
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
Added support for SIGTERM #472
Added support for SIGTERM #472
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great to see the support for SIGTERM
signal being added, thanks for this! We're currently focusing on the release-2.0
branch, and since this change isn't critical for version 1, could you please base your changes on release-2.0
instead? Would appreciate your help with this shift!
7cb393a
to
8217d6b
Compare
Used for sharing the defines for SWIFTPM_MODULE_BUNDLE.
As in getsentry/sentry-cocoa#3895 and firebase/firebase-ios-sdk#12881, this adds support for the SIGTERM signal which Apple often uses before sending SIGKILL.