Skip to content

[Bug 574] Fixed: can not build out arm64 version for linux-android pl…#575

Open
db0218 wants to merge 1 commit into
yodaos-project:masterfrom
db0218:master
Open

[Bug 574] Fixed: can not build out arm64 version for linux-android pl…#575
db0218 wants to merge 1 commit into
yodaos-project:masterfrom
db0218:master

Conversation

@db0218

@db0218 db0218 commented Jan 19, 2021

Copy link
Copy Markdown

…atform

Cause: cmake param "-DANDROID_ABI" should be "arm64-v8a", rather than "armeabi-v7a".
Solution: 1. correct the wrong description in doc. 2. Add "-DANDROID_ABI" in the deps cmake files, otherwise these deps static libraries will all be default arm32.
Fixed Version: 2021.1.19
Branch: master

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added

…atform

Cause: cmake param "-DANDROID_ABI" should be "arm64-v8a", rather than "armeabi-v7a".
Solution: 1. correct the wrong description in doc.  2. Add "-DANDROID_ABI" in the deps cmake files, otherwise these deps static libraries will all be default arm32.
Fixed Version: 2021.1.19
Branch: master
@yorkie

yorkie commented Jan 19, 2021

Copy link
Copy Markdown
Member

@db0218 May I ask you to push again to retrigger the build, it seems failed caused by networks.

@yorkie

yorkie commented Jan 19, 2021

Copy link
Copy Markdown
Member

@algebrait @legendecas have a review?

@algebrait algebrait self-requested a review January 19, 2021 05:39

@algebrait algebrait left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@yorkie yorkie added the build label Jan 19, 2021
@db0218

db0218 commented Jan 19, 2021

Copy link
Copy Markdown
Author

@db0218 May I ask you to push again to retrigger the build, it seems failed caused by networks.

Should I close this one and start a new pull request ?

@yorkie

yorkie commented Jan 20, 2021

Copy link
Copy Markdown
Member

You could just do git push -f :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants