Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

fix: Merge loadLibrary calls for sentry-native and clean up CMake files#373

Merged
marandaneto merged 3 commits intomasterfrom
fix/library-loading
Apr 23, 2020
Merged

fix: Merge loadLibrary calls for sentry-native and clean up CMake files#373
marandaneto merged 3 commits intomasterfrom
fix/library-loading

Conversation

@Swatinem
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

This loads all the non-essential-system libraries explicitly in order, to hopefully avoid dynamic linker errors such as #341

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

@Swatinem Swatinem requested a review from marandaneto April 22, 2020 14:35
@Swatinem Swatinem requested a review from bruno-garcia as a code owner April 22, 2020 14:35
Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@Swatinem
Copy link
Contributor Author

Don’t get your hopes up to soon. I have no idea if this actually fixes the problem :-D

@bruno-garcia
Copy link
Member

lol. Well, worth the shot

@marandaneto
Copy link
Contributor

Don’t get your hopes up to soon. I have no idea if this actually fixes the problem :-D

yeah its a try :) thx for that, left a comment btw.

Copy link
Contributor

@marandaneto marandaneto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marandaneto marandaneto merged commit e6810f9 into master Apr 23, 2020
@marandaneto marandaneto deleted the fix/library-loading branch April 23, 2020 09:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants