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

Pull in fixes from v5.23.5 #2

Merged

Conversation

regnerjr
Copy link

@regnerjr regnerjr commented Aug 7, 2020

Goal

Bring in all changes along the v5 branch of Official Upstream Bugsnag. (including fixes required for building with Xcode 12.

Fixes: Not being able to build with Xcode 12.

Changeset

I created this branch by fetching from upstream then resetting hard to the commit which tag v5.23.5 points to.
i.e. bugsnag@c64b617

Tests

I have not tested this at all. Full faith in the Bugsnag devs and community.

Review

Outstanding Questions

Once this is merged into https://github.com/walmartlabs-wmusiphone/bugsnag-cocoa.git we can re-tag as v5.23.5-walmart and point our cartfile at this to continue our Xcode 12 migration.

kattrali and others added 30 commits January 15, 2020 14:40
…snag#442)

write(2) already intelligently handles buffering/writing, so this buffer
is unneeded in addition to being unsafe for the user-initiated reporting
case.

The buffer was added in kstenerud/KSCrash@be9bedb
and thus the original intent is lost to the mists.
On a Mac: git ls-files -ci --exclude-standard -z | xargs -0 git rm --cached (from https://stackoverflow.com/a/7532131/2431627)
… if returning to the foreground after more than 60 seconds
…vents-foreground-background

(fix) Automatic session reporting now correctly records a new session…
Clarify and expand the CONTRIBUTING.md notes
fractalwrench and others added 22 commits May 21, 2020 15:05
Reduces complexity of PR template as discussed in retro
Fix DYLD lock mechanism preventing compilation on iOS <10
@Sohil Sohil merged commit 43aa5fb into walmartlabs-wmusiphone:master Aug 10, 2020
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.

6 participants