Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Mobile/Android: Rebuild Entangled libs #1464

Merged
merged 1 commit into from
May 6, 2019
Merged

Conversation

rajivshah3
Copy link
Member

@rajivshah3 rajivshah3 commented May 3, 2019

Description

Rebuilds Entangled libraries with -ldl flag to prevent crashes on Android 5.0/5.1 (API level 21/22)

Related to bazelbuild/bazel#6959
Fixes #1463

Type of change

  • Bug fix (a non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested on Google Nexus 6, Android 5.0.1 and 5.1.1 (Firebase Test Lab)

Checklist:

  • My code follows the style guidelines for this project
  • I have performed a self-review of my own code
  • For changes to mobile that include native code (including React Native modules): I have verified that both iOS and Android successfully build in both Debug and Release modes

@rajivshah3 rajivshah3 added T - Bug Type - Bug: Something isn't working P - Android Platform - Specifically relates to Android C - Mobile L - WIP Lifecycle - Work in Progress P2 - High Priority - Prefer to fix or merge this labels May 3, 2019
@rajivshah3 rajivshah3 changed the title Mobile/Android: Rebuild Entangled libs with -ldl flag Mobile/Android: Rebuild Entangled libs May 3, 2019
@rajivshah3 rajivshah3 added L - Need reviews Lifecycle - Each PR must have at least two reviewers and removed L - WIP Lifecycle - Work in Progress labels May 4, 2019
@rajivshah3 rajivshah3 marked this pull request as ready for review May 4, 2019 01:44
Copy link
Contributor

@laumair laumair left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

@rajivshah3 rajivshah3 merged commit cec20b2 into develop May 6, 2019
@rajivshah3 rajivshah3 deleted the fix/android-21-22 branch May 6, 2019 18:48
rihardsgravis pushed a commit that referenced this pull request May 7, 2019
* develop:
  Desktop: Settings and Onboarding fixes (#1457)
  Mobile/Android: Rebuild Entangled libs (#1464)
  Bump dart-sass from 1.19.0 to 1.20.1 in /src/desktop (#1469)
  Bump webpack-cli from 3.3.1 to 3.3.2 in /src/desktop (#1472)
  Bump electron from 3.1.8 to 3.1.9 in /src/desktop (#1475)
  Shared: [Snyk] Fix for 1 vulnerable dependencies (#1474)
  Mobile: [Snyk] Fix for 1 vulnerable dependencies (#1473)
  Bump eslint-plugin-react-native from 3.2.1 to 3.7.0 (#1389)
  Bump fastlane from 2.121.1 to 2.122.0 in /src/mobile/android (#1468)
  Bump fastlane from 2.121.1 to 2.122.0 in /src/mobile/ios (#1467)
  Bump eslint-plugin-react from 7.12.4 to 7.13.0 (#1460)
  Bump lint-staged from 8.1.5 to 8.1.6 (#1461)
rihardsgravis pushed a commit that referenced this pull request May 9, 2019
* develop: (23 commits)
  Bump snyk from 1.162.0 to 1.163.0 (#1491)
  New Crowdin translations (#1446)
  Desktop: Add migration explanation (#1445)
  Add issue template metadata [ci skip] (#1486)
  Update issue templates to newer version (#1484)
  Shared: Bump update versioning (#1478)
  Bump snyk from 1.161.1 to 1.162.0 in /src/shared (#1482)
  Bump snyk from 1.161.1 to 1.162.0 (#1481)
  Bump @babel/plugin-proposal-export-namespace-from in /src/shared (#1450)
  Desktop 0.5.0 Release (#1479)
  Desktop: Settings and Onboarding fixes (#1457)
  Mobile/Android: Rebuild Entangled libs (#1464)
  Bump dart-sass from 1.19.0 to 1.20.1 in /src/desktop (#1469)
  Bump webpack-cli from 3.3.1 to 3.3.2 in /src/desktop (#1472)
  Bump electron from 3.1.8 to 3.1.9 in /src/desktop (#1475)
  Shared: [Snyk] Fix for 1 vulnerable dependencies (#1474)
  Mobile: [Snyk] Fix for 1 vulnerable dependencies (#1473)
  Bump eslint-plugin-react-native from 3.2.1 to 3.7.0 (#1389)
  Bump fastlane from 2.121.1 to 2.122.0 in /src/mobile/android (#1468)
  Bump fastlane from 2.121.1 to 2.122.0 in /src/mobile/ios (#1467)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C - Mobile L - Need reviews Lifecycle - Each PR must have at least two reviewers P - Android Platform - Specifically relates to Android P2 - High Priority - Prefer to fix or merge this T - Bug Type - Bug: Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash on launch when running Android 5.0/5.1
2 participants