apache / incubator-weex

Apache Weex (Incubating)
https://weex.apache.org
Apache License 2.0
13.75k stars 1.82k forks source link

Feature(Android): Support x86_64 #3167

Closed YorkShen closed 4 years ago

YorkShen commented 4 years ago

Please note: x86_64 is supported only if the JS interpretator supports x86_64, which means:

As JavaScriptCore.so is no longer provided after Weex 0.28, the above condition is not a big problem for us.

weex-bot commented 4 years ago
Warnings
:warning: No Changelog changes! - Can you add a Changelog? To do so,append your changes to the changelog.md
:warning: Current pr not bind the milestone
:warning: If your PR is about fixing a bug excluding crash the code,you should add the demo link in the PR description. Demo link: http://dotwe.org/vue

Generated by :no_entry_sign: dangerJS against 56a407c7a4538c77eaa5201866132d3c08304fb6

cnryb commented 4 years ago

The Travis CI build is so slow, and it is easy to fail because of network problems.

YorkShen commented 4 years ago

@cnryb I could fix it by removing maven mirrors of Alibaba Cloud, which would however cause a problem to our users in China.

YorkShen commented 4 years ago

The failure of previous build is because that JavaScriptCore.so dont' support x86_64.

@cnryb Please review it again.

FYI: We still need to figure out how to sovle the network problem on CI server, however.