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

Remove fork of KSCrash from dependencies #96

Merged
merged 19 commits into from
Feb 16, 2016

Conversation

kattrali
Copy link
Contributor

Changes

This is a rework of the workspace and dependencies to use standard KSCrash 1.0.2 and above.

This will be a major release.

Source

  • Combine BugsnagNotifier implementations into a single class

Installation / Dependency Management

  • Update installation instructions for manual configuration
  • Convert podspec to JSON
  • Moved KSCrash dependency into Carthage/Checkouts for possible future Carthage support
  • Update minimum deployment targets to iOS 7.0 and OS X 10.8, per changes in KSCrash
  • Add instructions for crash symbolication with bitcode

Organization

  • Use dependency submodule directly in the workspace
  • Split top-level workspace into separate workspaces for iOS and OS X, to handle this Xcode "feature"
  • Reorganized files to reduce nesting and share between workspaces:
    • Source: The library code
    • Tests: Bugsnag tests
    • iOS: iOS-specific build files
    • OSX: OS X-specific build files

New features

  • Swift demangling
  • Bitcode support
  • Improved crash reporting for malloc errors and overwriting link registers

Resolved Issues

Fixes #94
Fixes #80
Fixes #78
Fixes #72
Fixes #67
Fixes #62
Fixes #52
Fixes #44
Fixes #41

kattrali added a commit that referenced this pull request Feb 16, 2016
Remove fork of KSCrash from dependencies
@kattrali kattrali merged commit 8c99da7 into master Feb 16, 2016
@kattrali kattrali deleted the kattrali/remove-kscrash-fork branch February 16, 2016 21:36
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.

1 participant