-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
sentry-native: add option to switch crashpad dependency #5717
sentry-native: add option to switch crashpad dependency #5717
Conversation
All green in build 2 (
|
@prince-chrismc can you explain for me the relationship between these sentry-projects? (e.g. sentry, sentry-native, sentry-crashpad, sentry-breakpad, crashpad, breakpad, etc.) |
Oh I am no expert! I'll do my best... There's a bunch of of OSS libraries for "stack trace error handling" (eg crashpad/breakpad) The Sentry community (I like to romanticize it's because Conan did not exist) forked the projects to their build tree and added them as dependencies... over time they made minor changes so there's a slight difference between the original and the As a OSS purist you might be using the original crashpad for you application and do not want to mix the possibly different ABIs... so you replace the sentry version with the one you like |
@prince-chrismc |
* sentry-native: add option to switch crashpad dependency * sentry-native: fix compatibility with sentry-crashpad * sentry-native: add missing patch
Specify library name and version: sentry-native/all
Add option to swtich crashpad dependency between official upstream google crashpad and sentry's crashpad fork.
conan-center hook activated.