Skip to content

chore: Fix the Android CI job#2390

Merged
robinlinden merged 1 commit into
TokTok:masterfrom
robinlinden:ci-fix-android
Aug 16, 2023
Merged

chore: Fix the Android CI job#2390
robinlinden merged 1 commit into
TokTok:masterfrom
robinlinden:ci-fix-android

Conversation

@robinlinden
Copy link
Copy Markdown

@robinlinden robinlinden commented Aug 15, 2023

  • Bump the targeted API on armeabi and x86 to 19

    • Starting with the r24 NDK, Jelly Bean (APIs 16, 17, and 18) is no longer supported.
  • Build libsodium w/ --disable-pie

    • Workaround for ld: error: relocation R_386_PC32 cannot be used against symbol 'crypto_auth_hmacsha512_init'; recompile with -fPIC

This change is Reviewable

* Bump the targeted API on armeabi and x86 to 19
  - Starting with the r24 NDK, Jelly Bean (APIs 16, 17, and 18) is no
    longer supported.

* Build libsodium w/ --disable-pie
  - Workaround for `ld: error: relocation R_386_PC32 cannot be used
    against symbol 'crypto_auth_hmacsha512_init'; recompile with -fPIC`
@robinlinden robinlinden requested a review from a team as a code owner August 15, 2023 23:00
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 15, 2023

Codecov Report

Merging #2390 (b0f6331) into master (2b1a6b0) will increase coverage by 0.00%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #2390   +/-   ##
======================================
  Coverage    9.98%   9.99%           
======================================
  Files         141     141           
  Lines       32530   32530           
======================================
+ Hits         3249    3250    +1     
+ Misses      29281   29280    -1     

see 5 files with indirect coverage changes

@Green-Sky Green-Sky added this to the v0.2.19 milestone Aug 15, 2023
@robinlinden robinlinden merged commit b0f6331 into TokTok:master Aug 16, 2023
@robinlinden robinlinden deleted the ci-fix-android branch August 16, 2023 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants