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.

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

Stop installing openssl in Darwin CI. #24950

Merged

Conversation

bzbarsky-apple
Copy link
Contributor

Now that #24929 is merged, openssl is not needed for Darwin.

@github-actions
Copy link

github-actions bot commented Feb 9, 2023

PR #24950: Size comparison from 9c631df to 59f7266

Decreases (2 builds for cc32xx, qpg)
platform target config section 9c631df 59f7266 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20194396 20194395 -1 -0.0
qpg lock-app qpg6105+debug (read/write) 1116736 1116728 -8 -0.0
.text 563832 563824 -8 -0.0
Full report (3 builds for cc32xx, qpg)
platform target config section 9c631df 59f7266 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 20194396 20194395 -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 532524 532524 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 1116728 -8 -0.0
.bss 96396 96396 0 0.0
.data 860 860 0 0.0
.text 563832 563824 -8 -0.0

Now that project-chip#24929 is
merged, openssl is not needed for Darwin.
@bzbarsky-apple bzbarsky-apple enabled auto-merge (squash) February 10, 2023 02:54
@github-actions
Copy link

PR #24950: Size comparison from 3a8096d to 5bfdd55

Decreases (1 build for cc32xx)
platform target config section 3a8096d 5bfdd55 change % change
cc32xx lock CC3235SF_LAUNCHXL (read only) 640233 640225 -8 -0.0
.debug_info 20194396 20194394 -2 -0.0
.text 532524 532516 -8 -0.0
Full report (1 build for cc32xx)
platform target config section 3a8096d 5bfdd55 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 640233 640225 -8 -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 20194396 20194394 -2 -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 532516 -8 -0.0

@github-actions
Copy link

PR #24950: Size comparison from 3a8096d to 3449421

Decreases (1 build for cc32xx)
platform target config section 3a8096d 3449421 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20194396 20194395 -1 -0.0
Full report (4 builds for cc32xx, nrfconnect)
platform target config section 3a8096d 3449421 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 20194396 20194395 -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 532524 532524 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 1108392 0 0.0
bss 142687 142687 0 0.0
rodata 111364 111364 0 0.0
text 774564 774564 0 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

@bzbarsky-apple bzbarsky-apple merged commit ed22d1e into project-chip:master Feb 10, 2023
@bzbarsky-apple bzbarsky-apple deleted the ci-darwin-no-openssl branch February 10, 2023 16:27
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
Now that project-chip#24929 is
merged, openssl is not needed for Darwin.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants