Commit 0437216
committed
Support build by clang.
Based on https://github.com/gengjiawen/android-jsc/tree/feature/abi_support and some enhancements:
- Build by clang
- NDK r17c
- Keep app_platform (minSdkVersion) at API 16 for 32bits, follows
original JSC & RN's spec. 64bits will be API 21.
- Repack AAR with POM by gradle and publish into npm. Works referenced from
https://github.com/react-community/jsc-android-buildscripts.
This supports to use local maven repository directly from node_modules.
- Docker build based on pure openjdk:8-jdk image. The docker builder
referenced from https://github.com/gengjiawen/android-ndk.
- All build steps are reproducible by Travis CI.
Some tech notes:
- patches/log2.patch fallback for log2() only defined in NDK after API 19.
- Travis build process getting a little tricky.
Travis limits one job cost at maximum 50 minutes and we cannot finish
all build before 50 minutes.
The workaround is to split whole process into differnt build stage,
i.e. multiple build jobs.
And I store the last build status into docker hub at
https://hub.docker.com/r/kudojscbuilder/android-jsc.
1. Prepare build environment docker image.
2. Based on previous image, build only armv7/x86 binary and upload
new image to docker hub.
Note that I also store buck-out/* object files into docker image.
So that we don't need to rebuild 32bits assets for next stage.
3. Based on previous image, do full build and deploy into npm.
Migration Guide:
1. yarn add @kudo-ci/android-jsc
2. Use @kudo-ci/android-jsc local maven repository before jcenter()
diff --git a/ReactAndroid/build.gradle b/ReactAndroid/build.gradle
index a419728d77..cec9b33a7b 100644
--- a/ReactAndroid/build.gradle
+++ b/ReactAndroid/build.gradle
@@ -324,7 +314,7 @@ dependencies {
api "com.squareup.okhttp3:okhttp:${OKHTTP_VERSION}"
api "com.squareup.okhttp3:okhttp-urlconnection:${OKHTTP_VERSION}"
api 'com.squareup.okio:okio:1.14.0'
- compile project(':android-jsc')
+ compile 'org.webkit:android-jsc:r174650'
diff --git a/build.gradle b/build.gradle
index 08845e85fc..5339ff8981 100644
--- a/build.gradle
+++ b/build.gradle
@@ -20,10 +20,13 @@ buildscript {
allprojects {
repositories {
+ maven {
+ url "$rootDir/node_modules/@kudo-ci/android-jsc/dist"
+ }
+
google()
jcenter()
mavenLocal()
-
def androidSdk = System.getenv("ANDROID_SDK")
maven {
url "$androidSdk/extras/m2repository/"1 parent 9560d9a commit 0437216
21 files changed
Lines changed: 540 additions & 180 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | | - | |
3 | 1 | | |
4 | 2 | | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | | - | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
37 | 9 | | |
38 | 10 | | |
39 | 11 | | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
40 | 19 | | |
41 | 20 | | |
42 | 21 | | |
| |||
45 | 24 | | |
46 | 25 | | |
47 | 26 | | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | | - | |
73 | | - | |
74 | | - | |
75 | | - | |
76 | | - | |
77 | | - | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | | - | |
86 | | - | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
117 | 30 | | |
118 | 31 | | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
| 2 | + | |
2 | 3 | | |
3 | 4 | | |
4 | 5 | | |
| 6 | + | |
5 | 7 | | |
6 | 8 | | |
7 | 9 | | |
| |||
10 | 12 | | |
11 | 13 | | |
12 | 14 | | |
| 15 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
7 | 8 | | |
8 | | - | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
2 | 47 | | |
3 | | - | |
4 | 48 | | |
5 | 49 | | |
6 | 50 | | |
7 | | - | |
8 | | - | |
| 51 | + | |
| 52 | + | |
9 | 53 | | |
10 | 54 | | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
17 | 61 | | |
18 | | - | |
19 | | - | |
20 | | - | |
| 62 | + | |
21 | 63 | | |
22 | 64 | | |
23 | | - | |
24 | | - | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | | - | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
0 commit comments