8000 chore: Fix the Android CI job by robinlinden · Pull Request #2390 · TokTok/c-toxcore · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: Fix the Android CI job #2390

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

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Conversation

robinlinden
Copy link
Member
@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
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   +/-   ##
===================
A999
===================
  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
0