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

Use KSCrash dependency in CocoaPods instead of self-hosting #41

Closed
orta opened this issue Mar 17, 2015 · 6 comments
Closed

Use KSCrash dependency in CocoaPods instead of self-hosting #41

orta opened this issue Mar 17, 2015 · 6 comments

Comments

@orta
Copy link

orta commented Mar 17, 2015

screen shot 2015-03-17 at 3 33 23 pm

KSCrash is it's own independent CocoaPod, people are less likely to get class collisions in their apps. If they are including both ( I am in the tests for ARAnalytics )

@ConradIrwin
Copy link
Contributor

Hi @orta,

For now we have a few unmerged changes to KSCrash. We'll kick off a project to get them merged upstream (it requires a bit of cleanup), so this should be fixed eventually.

(That said, Bugsnag will work without those changes, it will just block the UI thread more than we'd like).

Conrad

@orta
Copy link
Author

orta commented Mar 18, 2015

👍 good luck :)

@gsamat
Copy link

gsamat commented Jun 5, 2015

We are getting class collision with https://appmetrika.yandex.ru/ , which is quite popular in Russia.

Could you please estimate when would you be able to ship fix for that dependency?

That is important for us as we will consider forking your library in case it will take another few months for you to fix.

@jessicard
Copy link
Contributor

@gsamat Unfortunately we are still a bit away before we can start this project. I'd recommend forking the notifier for now.

@crossle
Copy link

crossle commented Jul 24, 2015

The following build commands failed:
CompileC Bugsnag/KSCrash/Source/KSCrash/Recording/KSCrashReport.c normal armv7 c com.apple.compilers.llvm.clang.1_0.compiler (1 failure)

KSCrash compile failed, use command compile

@joshdholtz
Copy link

I agree with @orta on this 👍 Are we any closer to getting that added in?

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 a pull request may close this issue.

6 participants