Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fixes android builds (cherrypick from master) #22513

Merged
merged 2 commits into from
Sep 9, 2022
Merged

Fixes android builds (cherrypick from master) #22513

merged 2 commits into from
Sep 9, 2022

Conversation

andy31415
Copy link
Contributor

Cherrypicks #22464 and #22486

andy31415 and others added 2 commits September 9, 2022 10:07
@andy31415 @andreilitvin
Fix Android Tests in CI (project-chip#22464)
c1fa1b7 
* Enable android CI on push as well

* Revert "Enable android CI on push as well"

This reverts commit 3bcca58.

* Add submodule checkout to full android: needed to set secure.directory

* Distinct name for full android builds

* Place the submodule checkout to the right location

* Undo the wandalen retry removal

Co-authored-by: Andrei Litvin <[email protected]>
@andy31415
Fix android full build in CI (project-chip#22486)
7493b7b 
* Split out android builds to not use as much space during building

* Ensure we can test android builds with act

* Since we use checkout_submodules, submodules are not needed on the other checkout

* Add some documentation on using act to test

* Fix ENOBUFS being undefined when compiling tv-casting-app on arm
@andy31415 andy31415 added the hotfix urgent fix needed, can bypass review label Sep 9, 2022
@github-actions
Copy link

github-actions bot commented Sep 9, 2022 •
edited
Loading

PR #22513: Size comparison from 5e6279c to 7493b7b

Increases (2 builds for nrfconnect, telink)
platform target config section 5e6279c 7493b7b change % change
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 814168 814172 4 0.0
telink lighting-app tlsr9518adk80d text 589472 589474 2 0.0
Full report (18 builds for bl602, cyw30739, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 5e6279c 7493b7b change % change
bl602 lighting-app bl602 (read/write) 1383898 1383898 0 0.0
.bss 120290 120290 0 0.0
.data 4480 4480 0 0.0
.text 1051048 1051048 0 0.0
bl602+rpc (read/write) 1429410 1429410 0 0.0
.bss 127730 127730 0 0.0
.data 4600 4600 0 0.0
.text 1082808 1082808 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 586118 586118 0 0.0
.app_xip_area 462752 462752 0 0.0
.bss 65800 65800 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 591894 591894 0 0.0
.app_xip_area 463744 463744 0 0.0
.bss 70584 70584 0 0.0
.data 748 748 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 599026 599026 0 0.0
.app_xip_area 476380 476380 0 0.0
.bss 65112 65112 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w light k32w0+release (read/write) 645524 645524 0 0.0
.bss 70232 70232 0 0.0
.data 2044 2044 0 0.0
.text 570520 570520 0 0.0
lock k32w0+release (read/write) 703044 703044 0 0.0
.bss 70704 70704 0 0.0
.data 2052 2052 0 0.0
.text 627560 627560 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10299916 10299916 0 0.0
(read/write) 704593 704593 0 0.0
.bss 33297 33297 0 0.0
.data 3272 3272 0 0.0
.data.rel.ro 649248 649248 0 0.0
.dynamic 560 560 0 0.0
.got 13816 13816 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 498076 498076 0 0.0
.text 8150772 8150772 0 0.0
thermostat-no-ble arm64 (read only) 2356916 2356916 0 0.0
(read/write) 141825 141825 0 0.0
.bss 55345 55345 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 75984 75984 0 0.0
.dynamic 560 560 0 0.0
.got 5048 5048 0 0.0
.init 24 24 0 0.0
.init_array 408 408 0 0.0
.rodata 140524 140524 0 0.0
.text 1978672 1978672 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2454328 2454328 0 0.0
.bss 215076 215076 0 0.0
.data 5872 5872 0 0.0
.text 1416972 1416972 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1180147 1180147 0 0.0
bss 143759 143759 0 0.0
rodata 143268 143268 0 0.0
text 814168 814172 4 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1160023 1160023 0 0.0
bss 142996 142996 0 0.0
rodata 134944 134944 0 0.0
text 803160 803160 0 0.0
p6 all-clusters-app default (read only) 842104 842104 0 0.0
(read/write) 1738996 1738996 0 0.0
.bss 188584 188584 0 0.0
.data 2656 2656 0 0.0
.text 1539368 1539368 0 0.0
all-clusters-minimal-app default (read only) 842824 842824 0 0.0
(read/write) 1683004 1683004 0 0.0
.bss 187864 187864 0 0.0
.data 2656 2656 0 0.0
.text 1484096 1484096 0 0.0
light-app default (read only) 851168 851168 0 0.0
(read/write) 1600292 1600292 0 0.0
.bss 179728 179728 0 0.0
.data 2448 2448 0 0.0
.text 1409728 1409728 0 0.0
lock-app default (read only) 846136 846136 0 0.0
(read/write) 1638012 1638012 0 0.0
.bss 184744 184744 0 0.0
.data 2464 2464 0 0.0
.text 1442416 1442416 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 808808 808808 0 0.0
bss 71480 71480 0 0.0
noinit 43488 43488 0 0.0
text 571350 571350 0 0.0
lighting-app tlsr9518adk80d (read/write) 830776 830776 0 0.0
bss 72336 72336 0 0.0
noinit 43488 43488 0 0.0
text 589472 589474 2 0.0

@andy31415 andy31415 merged commit b5563e7 into project-chip:sve-2 Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
github hotfix urgent fix needed, can bypass review workflows
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@andy31415