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

Fix CHIPTool build on iOS. #18523

Merged
merged 1 commit into from
May 17, 2022
Merged

Fix CHIPTool build on iOS. #18523

merged 1 commit into from
May 17, 2022

Conversation

bzbarsky-apple
Copy link
Contributor

The compiler is not happy with nullability specified for only some
things in a header, so have todo it for all of them.

Also fixes incorrect nullability specifications in CHIPTestStorage.

Fixes #18518

Problem

See above.

Change overview

See above.

Testing

Compiled CHIPTool for iOS, commissioned all-clusters-app, issued some commands.

@bzbarsky-apple
Fix CHIPTool build on iOS.
d3fbff1 
The compiler is not happy with nullability specified for only some
things in a header, so have todo it for all of them.

Also fixes incorrect nullability specifications in CHIPTestStorage.

Fixes project-chip#18518
@boring-cyborg boring-cyborg bot added the darwin label May 17, 2022
@github-actions
Copy link

github-actions bot commented May 17, 2022 •
edited
Loading

PR #18523: Size comparison from d79a0e3 to d3fbff1

Full report (37 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section d79a0e3 d3fbff1 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 692151 692151 0 0.0
(read/write) 159384 159384 0 0.0
.bss 74516 74516 0 0.0
.data 3412 3412 0 0.0
.rodata 103823 103823 0 0.0
.text 587848 587848 0 0.0
lock-ftd LP_CC2652R7 (read only) 676863 676863 0 0.0
(read/write) 165776 165776 0 0.0
.bss 72676 72676 0 0.0
.data 3236 3236 0 0.0
.rodata 94967 94967 0 0.0
.text 581412 581412 0 0.0
lock-mtd LP_CC2652R7 (read only) 625695 625695 0 0.0
(read/write) 145508 145508 0 0.0
.bss 68412 68412 0 0.0
.data 3236 3236 0 0.0
.rodata 94855 94855 0 0.0
.text 530348 530348 0 0.0
pump-app LP_CC2652R7 (read only) 660387 660387 0 0.0
(read/write) 183532 183532 0 0.0
.bss 72932 72932 0 0.0
.data 3268 3268 0 0.0
.rodata 80467 80467 0 0.0
.text 579440 579440 0 0.0
pump-controller-app LP_CC2652R7 (read only) 653555 653555 0 0.0
(read/write) 190172 190172 0 0.0
.bss 72996 72996 0 0.0
.data 3232 3232 0 0.0
.rodata 83571 83571 0 0.0
.text 569504 569504 0 0.0
shell LP_CC2652R7 (read only) 686538 686538 0 0.0
(read/write) 160820 160820 0 0.0
.bss 77140 77140 0 0.0
.data 3416 3416 0 0.0
.rodata 101370 101370 0 0.0
.text 584684 584684 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 624766 624766 0 0.0
.app_xip_area 528396 528396 0 0.0
.bss 79012 79012 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 627826 627826 0 0.0
.app_xip_area 532912 532912 0 0.0
.bss 77588 77588 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 572266 572266 0 0.0
.app_xip_area 467636 467636 0 0.0
.bss 87008 87008 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 915068 915068 0 0.0
(read/write) 133220 133220 0 0.0
.bss 131160 131160 0 0.0
.data 2060 2060 0 0.0
.text 915060 915060 0 0.0
BRD4161A+rpc (read only) 949304 949304 0 0.0
(read/write) 149904 149904 0 0.0
.bss 147640 147640 0 0.0
.data 2264 2264 0 0.0
.text 949296 949296 0 0.0
BRD4161A+rs911x (read only) 784416 784416 0 0.0
(read/write) 128444 128444 0 0.0
.bss 126372 126372 0 0.0
.data 2068 2068 0 0.0
.text 784408 784408 0 0.0
lock-app BRD4161A+wf200 (read only) 945156 945156 0 0.0
(read/write) 122924 122924 0 0.0
.bss 120900 120900 0 0.0
.data 2024 2024 0 0.0
.text 945148 945148 0 0.0
window-app BRD4161A (read only) 895188 895188 0 0.0
(read/write) 133280 133280 0 0.0
.bss 131232 131232 0 0.0
.data 2048 2048 0 0.0
.text 895180 895180 0 0.0
esp32 all-clusters-app c3devkit (read only) 998376 998376 0 0.0
(read/write) 1477026 1477026 0 0.0
.dram0.bss 67128 67128 0 0.0
.dram0.data 14576 14576 0 0.0
.flash.rodata 209888 209888 0 0.0
.flash.text 998376 998376 0 0.0
.iram0.text 62954 62954 0 0.0
m5stack (read only) 1053635 1053635 0 0.0
(read/write) 479024 479024 0 0.0
.dram0.bss 72656 72656 0 0.0
.dram0.data 34184 34184 0 0.0
.flash.rodata 240188 240188 0 0.0
.flash.text 1048251 1048251 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 682516 682516 0 0.0
.bss 80200 80200 0 0.0
.data 2016 2016 0 0.0
.text 598596 598596 0 0.0
lock k32w061+release (read/write) 729012 729012 0 0.0
.bss 80632 80632 0 0.0
.data 1976 1976 0 0.0
.text 644700 644700 0 0.0
linux all-clusters-app debug (read only) 2738905 2738905 0 0.0
(read/write) 174624 174624 0 0.0
.bss 83904 83904 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 82488 82488 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 238877 238877 0 0.0
.text 2323938 2323938 0 0.0
bridge-app debug+rpc (read only) 2021385 2021385 0 0.0
(read/write) 146552 146552 0 0.0
.bss 71744 71744 0 0.0
.data 3936 3936 0 0.0
.data.rel.ro 65288 65288 0 0.0
.dynamic 592 592 0 0.0
.got 4272 4272 0 0.0
.init 27 27 0 0.0
.init_array 688 688 0 0.0
.rodata 167401 167401 0 0.0
.text 1696002 1696002 0 0.0
chip-tool debug (read only) 9282453 9282453 0 0.0
(read/write) 578864 578864 0 0.0
.bss 22816 22816 0 0.0
.data 1136 1136 0 0.0
.data.rel.ro 548608 548608 0 0.0
.dynamic 624 624 0 0.0
.got 5000 5000 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 479061 479061 0 0.0
.text 7474613 7474613 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 9058756 9058756 0 0.0
(read/write) 644993 644993 0 0.0
.bss 41105 41105 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 583864 583864 0 0.0
.dynamic 560 560 0 0.0
.got 14992 14992 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 445124 445124 0 0.0
.text 7149732 7149732 0 0.0
lighting-app debug+rpc (read only) 2315209 2315209 0 0.0
(read/write) 152000 152000 0 0.0
.bss 73536 73536 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 70664 70664 0 0.0
.dynamic 608 608 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 792 792 0 0.0
.rodata 186473 186473 0 0.0
.text 1961522 1961522 0 0.0
lock-app debug (read only) 2228937 2228937 0 0.0
(read/write) 146680 146680 0 0.0
.bss 72224 72224 0 0.0
.data 1568 1568 0 0.0
.data.rel.ro 67192 67192 0 0.0
.dynamic 592 592 0 0.0
.got 4336 4336 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 197049 197049 0 0.0
.text 1870994 1870994 0 0.0
ota-provider-app debug (read only) 2049873 2049873 0 0.0
(read/write) 139728 139728 0 0.0
.bss 71680 71680 0 0.0
.data 1768 1768 0 0.0
.data.rel.ro 60488 60488 0 0.0
.dynamic 608 608 0 0.0
.got 4504 4504 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 177267 177267 0 0.0
.text 1714210 1714210 0 0.0
ota-requestor-app debug (read only) 2079201 2079201 0 0.0
(read/write) 142568 142568 0 0.0
.bss 72352 72352 0 0.0
.data 1992 1992 0 0.0
.data.rel.ro 62584 62584 0 0.0
.dynamic 592 592 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 173468 173468 0 0.0
.text 1745874 1745874 0 0.0
shell debug (read only) 2555289 2555289 0 0.0
(read/write) 198552 198552 0 0.0
.bss 114632 114632 0 0.0
.data 1376 1376 0 0.0
.data.rel.ro 76776 76776 0 0.0
.dynamic 608 608 0 0.0
.got 4192 4192 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 220050 220050 0 0.0
.text 2172530 2172530 0 0.0
thermostat-no-ble arm64 (read only) 2343780 2343780 0 0.0
(read/write) 175121 175121 0 0.0
.bss 86465 86465 0 0.0
.data 1520 1520 0 0.0
.data.rel.ro 79336 79336 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 376 376 0 0.0
.rodata 147108 147108 0 0.0
.text 1966848 1966848 0 0.0
tv-app debug (read only) 2841185 2841185 0 0.0
(read/write) 277472 277472 0 0.0
.bss 189592 189592 0 0.0
.data 4672 4672 0 0.0
.data.rel.ro 76960 76960 0 0.0
.dynamic 592 592 0 0.0
.got 4720 4720 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 219345 219345 0 0.0
.text 2438562 2438562 0 0.0
tv-casting-app debug (read only) 5403545 5403545 0 0.0
(read/write) 223776 223776 0 0.0
.bss 77240 77240 0 0.0
.data 2368 2368 0 0.0
.data.rel.ro 137952 137952 0 0.0
.dynamic 608 608 0 0.0
.got 4728 4728 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 337921 337921 0 0.0
.text 4705314 4705314 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2418784 2418784 0 0.0
.bss 201588 201588 0 0.0
.data 5872 5872 0 0.0
.text 1381428 1381428 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1180243 1180243 0 0.0
bss 138356 138356 0 0.0
rodata 152956 152956 0 0.0
text 810160 810160 0 0.0
p6 all-clusters-app default (read/write) 2532168 2532168 0 0.0
.bss 135096 135096 0 0.0
.data 2808 2808 0 0.0
.text 1490432 1490432 0 0.0
light-app default (read/write) 2418776 2418776 0 0.0
.bss 128416 128416 0 0.0
.data 2608 2608 0 0.0
.text 1377040 1377040 0 0.0
lock-app default (read/write) 2429424 2429424 0 0.0
.bss 128232 128232 0 0.0
.data 2568 2568 0 0.0
.text 1387688 1387688 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 782116 782116 0 0.0
bss 70592 70592 0 0.0
noinit 40416 40416 0 0.0
text 553028 553028 0 0.0
lighting-app tlsr9518adk80d (read/write) 802208 802208 0 0.0
bss 70848 70848 0 0.0
noinit 40416 40416 0 0.0
text 569794 569794 0 0.0

@andy31415 andy31415 merged commit b4bffa0 into project-chip:master May 17, 2022
@bzbarsky-apple bzbarsky-apple deleted the fix-CHIPTool-iOS-build branch May 17, 2022 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anush-apple anush-apple anush-apple approved these changes

@andy31415 andy31415 andy31415 approved these changes

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@hawk248 hawk248 Awaiting requested review from hawk248

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@sagar-apple sagar-apple Awaiting requested review from sagar-apple

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
darwin review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CHIPTool App compilation error on iOS
3 participants
@bzbarsky-apple @andy31415 @anush-apple