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

[ESP32] Fix host setup failure on mac and some build docs #24973

Merged
merged 4 commits into from
Feb 11, 2023
Merged

[ESP32] Fix host setup failure on mac and some build docs #24973

merged 4 commits into from
Feb 11, 2023

Conversation

shubhamdp
Copy link
Contributor

Problem

Fixes #24734 (comment)

This may also fix #24816

Change Overview

#22071 restricted requirements.esp32.txt to only linux platform and hence dependencies required to build the ESP32 examples are not installed on mac.

Restrictions were added because building wheels for gevent (dependency of gdbgui) failed.

Removed the platform restrictions for all the dependencies other than gdbgui. Added some documentation around how to install gdbgui on mac.

And some doc fixes.

Testing

  • Was able successfully bootstrap and build the esp32 application on mac and Ubuntu.

@pullapprove pullapprove bot requested a review from amitnj February 10, 2023 14:06
@github-actions github-actions bot added documentation Improvements or additions to documentation scripts labels Feb 10, 2023
@pullapprove pullapprove bot added review - pending and removed documentation Improvements or additions to documentation scripts labels Feb 10, 2023
@shubhamdp
Copy link
Contributor Author

shubhamdp commented Feb 10, 2023 •
edited
Loading

@andy31415 I took a suggestion from #23477 (comment)

@github-actions
Copy link

github-actions bot commented Feb 10, 2023 •
edited
Loading

PR #24973: Size comparison from 36735ed to 1b882fa

Increases (3 builds for bl702, psoc6)
platform target config section 36735ed 1b882fa change % change
bl702 lighting-app bl702+rpc .debug_info 44938490 44938491 1 0.0
.text 1029630 1029632 2 0.0
psoc6 all-clusters-minimal cy8ckit_062s2_43012 .debug_info 27460071 27460072 1 0.0
light cy8ckit_062s2_43012 .debug_info 22795134 22795135 1 0.0
Decreases (6 builds for bl602, bl702, cc32xx, psoc6)
platform target config section 36735ed 1b882fa change % change
bl602 lighting-app bl602 .text 1023324 1023322 -2 -0.0
bl602+rpc .text 1054256 1054254 -2 -0.0
bl702 lighting-app bl702 .debug_info 40536301 40536299 -2 -0.0
.text 952438 952434 -4 -0.0
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20194395 20194394 -1 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 27913082 27913081 -1 -0.0
lock cy8ckit_062s2_43012 .debug_info 23025031 23025030 -1 -0.0
Full report (19 builds for bl602, bl702, cc32xx, cyw30739, k32w, linux, psoc6, qpg)
platform target config section 36735ed 1b882fa change % change
bl602 lighting-app bl602 (read/write) 1346582 1346582 0 0.0
.bss 94794 94794 0 0.0
.data 9736 9736 0 0.0
.text 1023324 1023322 -2 -0.0
bl602+rpc (read/write) 1392014 1392014 0 0.0
.bss 102842 102842 0 0.0
.data 10128 10128 0 0.0
.text 1054256 1054254 -2 -0.0
bl702 lighting-app bl702 (read only) 3358 3358 0 0.0
(read/write) 1184651 1184651 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 69905 69905 0 0.0
.bss_psram 30048 30048 0 0.0
.comment 48 48 0 0.0
.data 4056 4056 0 0.0
.debug_abbrev 1550382 1550382 0 0.0
.debug_aranges 134192 134192 0 0.0
.debug_frame 491492 491492 0 0.0
.debug_info 40536301 40536299 -2 -0.0
.debug_line 5272965 5272965 0 0.0
.debug_loc 3409446 3409446 0 0.0
.debug_ranges 371752 371752 0 0.0
.debug_str 3558337 3558337 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 107120 107120 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 572518 572518 0 0.0
.symtab 173424 173424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 0 0 0 0.0
952438 952434 -4 -0.0
bl702+rpc (read only) 3358 3358 0 0.0
(read/write) 1277535 1277535 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 77953 77953 0 0.0
.bss_psram 30304 30304 0 0.0
.comment 48 48 0 0.0
.data 4608 4608 0 0.0
.debug_abbrev 1698796 1698796 0 0.0
.debug_aranges 142424 142424 0 0.0
.debug_frame 519236 519236 0 0.0
.debug_info 44938490 44938491 1 0.0
.debug_line 5671672 5671672 0 0.0
.debug_loc 3606205 3606205 0 0.0
.debug_ranges 395520 395520 0 0.0
.debug_str 3961701 3961701 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 121696 121696 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 633311 633311 0 0.0
.symtab 191840 191840 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 0 0 0 0.0
1029630 1029632 2 0.0
cc32xx lock CC3235SF_LAUNCHXL (read only) 640233 640233 0 0.0
(read/write) 204084 204084 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197488 197488 0 0.0
.comment 194 194 0 0.0
.data 1476 1476 0 0.0
.debug_abbrev 928439 928439 0 0.0
.debug_aranges 87352 87352 0 0.0
.debug_frame 299840 299840 0 0.0
.debug_info 20194395 20194394 -1 -0.0
.debug_line 2649797 2649797 0 0.0
.debug_loc 2785922 2785922 0 0.0
.debug_ranges 280720 280720 0 0.0
.debug_str 3001474 3001474 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105585 105585 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 375840 375840 0 0.0
.symtab 255856 255856 0 0.0
.text 0 0 0 0.0
532524 532524 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 584566 584566 0 0.0
.app_xip_area 461376 461376 0 0.0
.bss 65632 65632 0 0.0
.data 736 736 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 588538 588538 0 0.0
.app_xip_area 460052 460052 0 0.0
.bss 70920 70920 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 549990 549990 0 0.0
.app_xip_area 432288 432288 0 0.0
.bss 60192 60192 0 0.0
.data 692 692 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w contact k32w0+release (read/write) 668632 668632 0 0.0
.bss 77636 77636 0 0.0
.data 2200 2200 0 0.0
.text 569684 569684 0 0.0
light k32w0+release (read/write) 667484 667484 0 0.0
.bss 77332 77332 0 0.0
.data 2188 2188 0 0.0
.text 568852 568852 0 0.0
lock k32w0+release (read/write) 624316 624316 0 0.0
.bss 75468 75468 0 0.0
.data 2132 2132 0 0.0
.text 543988 543988 0 0.0
linux chip-tool-ipv6only arm64 (read only) 11997396 11997396 0 0.0
(read/write) 727720 727720 0 0.0
.bss 34248 34248 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 670008 670008 0 0.0
.dynamic 560 560 0 0.0
.got 15256 15256 0 0.0
.init 24 24 0 0.0
.init_array 208 208 0 0.0
.rodata 583220 583220 0 0.0
.text 9687412 9687412 0 0.0
thermostat-no-ble arm64 (read only) 2510292 2510292 0 0.0
(read/write) 145224 145224 0 0.0
.bss 56456 56456 0 0.0
.data 1824 1824 0 0.0
.data.rel.ro 77496 77496 0 0.0
.dynamic 560 560 0 0.0
.got 5408 5408 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 151040 151040 0 0.0
.text 2097280 2097280 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 0 0 0 0.0
(read only) 840872 840872 0 0.0
(read/write) 1757516 1757516 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189800 189800 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2672 2672 0 0.0
.debug_abbrev 1251968 1251968 0 0.0
.debug_aranges 111408 111408 0 0.0
.debug_frame 373996 373996 0 0.0
.debug_info 27913082 27913081 -1 -0.0
.debug_line 3784052 3784052 0 0.0
.debug_loc 3680841 3680841 0 0.0
.debug_ranges 362440 362440 0 0.0
.debug_str 3509578 3509578 0 0.0
.heap 840872 840872 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 578322 578322 0 0.0
.symtab 425056 425056 0 0.0
.text 1556656 1556656 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 0 0 0 0.0
(read only) 841688 841688 0 0.0
(read/write) 1699132 1699132 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188992 188992 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2664 2664 0 0.0
.debug_abbrev 1237639 1237639 0 0.0
.debug_aranges 110672 110672 0 0.0
.debug_frame 376440 376440 0 0.0
.debug_info 27460071 27460072 1 0.0
.debug_line 3792396 3792396 0 0.0
.debug_loc 3664571 3664571 0 0.0
.debug_ranges 360408 360408 0 0.0
.debug_str 3495331 3495331 0 0.0
.heap 841688 841688 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 539530 539530 0 0.0
.symtab 410336 410336 0 0.0
.text 1499088 1499088 0 0.0
.zero.table 8 8 0 0.0
light cy8ckit_062s2_43012 0 0 0 0.0
(read only) 850008 850008 0 0.0
(read/write) 1612172 1612172 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180872 180872 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2464 2464 0 0.0
.debug_abbrev 1071973 1071973 0 0.0
.debug_aranges 102880 102880 0 0.0
.debug_frame 346876 346876 0 0.0
.debug_info 22795134 22795135 1 0.0
.debug_line 3344713 3344713 0 0.0
.debug_loc 3342567 3342567 0 0.0
.debug_ranges 319584 319584 0 0.0
.debug_str 3298341 3298341 0 0.0
.heap 850008 850008 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 474698 474698 0 0.0
.symtab 378080 378080 0 0.0
.text 1420448 1420448 0 0.0
.zero.table 8 8 0 0.0
lock cy8ckit_062s2_43012 0 0 0 0.0
(read only) 845024 845024 0 0.0
(read/write) 1646444 1646444 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 185848 185848 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2472 2472 0 0.0
.debug_abbrev 1073677 1073677 0 0.0
.debug_aranges 103320 103320 0 0.0
.debug_frame 348816 348816 0 0.0
.debug_info 23025031 23025030 -1 -0.0
.debug_line 3346607 3346607 0 0.0
.debug_loc 3365446 3365446 0 0.0
.debug_ranges 321496 321496 0 0.0
.debug_str 3315611 3315611 0 0.0
.heap 845024 845024 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 478111 478111 0 0.0
.symtab 380272 380272 0 0.0
.text 1449736 1449736 0 0.0
.zero.table 8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1148688 1148688 0 0.0
.bss 99876 99876 0 0.0
.data 848 848 0 0.0
.text 595788 595788 0 0.0
lock-app qpg6105+debug (read/write) 1116736 1116736 0 0.0
.bss 96396 96396 0 0.0
.data 860 860 0 0.0
.text 563832 563832 0 0.0

bzbarsky-apple
bzbarsky-apple approved these changes Feb 10, 2023
View reviewed changes
docs/guides/esp32/build_app_and_commission.md Outdated Show resolved Hide resolved
docs/guides/esp32/setup_idf_chip.md Outdated Show resolved Hide resolved
docs/guides/esp32/setup_idf_chip.md Show resolved Hide resolved
@github-actions github-actions bot added documentation Improvements or additions to documentation scripts labels Feb 10, 2023
@github-actions
Copy link

PR #24973: Size comparison from 36735ed to 91af0f5

Decreases (1 build for nrfconnect)
platform target config section 36735ed 91af0f5 change % change
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1108392 1108376 -16 -0.0
text 774564 774560 -4 -0.0
Full report (5 builds for cc32xx, mbed, nrfconnect)
platform target config section 36735ed 91af0f5 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 640233 640233 0 0.0
(read/write) 204084 204084 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197488 197488 0 0.0
.comment 194 194 0 0.0
.data 1476 1476 0 0.0
.debug_abbrev 928439 928439 0 0.0
.debug_aranges 87352 87352 0 0.0
.debug_frame 299840 299840 0 0.0
.debug_info 20194395 20194395 0 0.0
.debug_line 2649797 2649797 0 0.0
.debug_loc 2785922 2785922 0 0.0
.debug_ranges 280720 280720 0 0.0
.debug_str 3001474 3001474 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105585 105585 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 375840 375840 0 0.0
.symtab 255856 255856 0 0.0
.text 532524 532524 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2464576 2464576 0 0.0
.bss 215892 215892 0 0.0
.data 5880 5880 0 0.0
.text 1427220 1427220 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1162364 1162364 0 0.0
bss 143531 143531 0 0.0
rodata 134376 134376 0 0.0
text 804572 804572 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1108392 1108376 -16 -0.0
bss 142687 142687 0 0.0
rodata 111364 111364 0 0.0
text 774564 774560 -4 -0.0
all-clusters-app nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1367600 1367600 0 0.0
bss 105990 105990 0 0.0
rodata 211808 211808 0 0.0
text 764276 764276 0 0.0

@andy31415 andy31415 merged commit e03a286 into project-chip:master Feb 11, 2023
@shubhamdp shubhamdp deleted the esp32_build_failure branch February 11, 2023 16:08
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
@shubhamdp
[ESP32] Fix host setup failure on mac and some build docs (project-ch…
64f6a07 
…ip#24973)

* [ESP32] Restrict only gdbgui to x64 linux

On mac, we can download the gdbgui wheels as binary

* [ESP32] Documentation changes

* address review comments

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

@bzbarsky-apple bzbarsky-apple bzbarsky-apple approved these changes

@dhrishi dhrishi dhrishi approved these changes

@andy31415 andy31415 andy31415 approved these changes

@amitnj amitnj Awaiting requested review from amitnj

@anush-apple anush-apple Awaiting requested review from anush-apple

@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

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

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

@gjc13 gjc13 Awaiting requested review from gjc13

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

@hawk248 hawk248 Awaiting requested review from hawk248

@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

@jmeg-sfy jmeg-sfy Awaiting requested review from jmeg-sfy

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

@ksperling-apple ksperling-apple Awaiting requested review from ksperling-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@lpbeliveau-silabs lpbeliveau-silabs Awaiting requested review from lpbeliveau-silabs

@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

@mspang mspang Awaiting requested review from mspang

@pjzander-signify pjzander-signify Awaiting requested review from pjzander-signify

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@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

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

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

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

@xylophone21 xylophone21 Awaiting requested review from xylophone21

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

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation review - approved scripts
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ModuleNotFound Error: No module named 'lark' Unable to compile all-clusters-app on M1 Mac
4 participants
@shubhamdp @andy31415 @dhrishi @bzbarsky-apple