Skip to content

[OSS-Fuzz] Stop Static Linking after migrating to ubuntu 24.04 runner#42567

Merged
mergify[bot] merged 1 commit intoproject-chip:masterfrom
Alami-Amine:AA/StopStaticLinking
Jan 8, 2026
Merged

[OSS-Fuzz] Stop Static Linking after migrating to ubuntu 24.04 runner#42567
mergify[bot] merged 1 commit intoproject-chip:masterfrom
Alami-Amine:AA/StopStaticLinking

Conversation

@Alami-Amine
Copy link
Copy Markdown
Contributor

Summary

  • Stop static linking of libraries after migrating to Ubuntu 24.04 runners, as static versions of many system libraries are no longer provided by the distribution, increasing maintenance cost and build complexity.

  • For example, the libmount-dev package no longer includes a static .a library.

Testing

…s since the static version of many of the libraries is not available easily
Copilot AI review requested due to automatic review settings January 7, 2026 18:51
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the static linking logic for OSS-Fuzz builds. The change is a direct response to the migration to Ubuntu 24.04 runners, which lack static versions of some system libraries. By deleting the special handling for oss_fuzz builds in build/config/linux/pkg_config.gni, the build system will now default to dynamic linking. This is a clean and correct solution to adapt to the new build environment. The change is well-contained and I have no further comments.

@andy31415 andy31415 added the sdk-maintainer-approved PR marked by `matter-sdk-maintainers` as suitable for MERGE - meets guideline & sufficient reviews. label Jan 7, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes static linking logic for OSS-Fuzz builds in response to the migration to Ubuntu 24.04 runners, where many system static libraries (like libmount-dev) are no longer provided.

Key Changes:

  • Removed the conditional static linking block for OSS-Fuzz builds from the pkg_config template
  • OSS-Fuzz builds will now use dynamic linking instead of static linking

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 7, 2026

PR #42567: Size comparison from 371b821 to 4825fb9

Full report (35 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, efr32, esp32, nrfconnect, nxp, psoc6, qpg, realtek, stm32, telink)
platform target config section 371b821 4825fb9 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1105660 1105660 0 0.0
RAM 178970 178970 0 0.0
bl702 lighting-app bl702+eth FLASH 660708 660708 0 0.0
RAM 135041 135041 0 0.0
bl702+wifi FLASH 836292 836292 0 0.0
RAM 124509 124509 0 0.0
bl706+mfd+rpc+littlefs FLASH 1070096 1070096 0 0.0
RAM 117373 117373 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 902634 902634 0 0.0
RAM 105964 105964 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 982998 982998 0 0.0
RAM 109876 109876 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 770716 770716 0 0.0
RAM 103408 103408 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 783736 783736 0 0.0
RAM 108728 108728 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 728736 728736 0 0.0
RAM 97484 97484 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 713312 713312 0 0.0
RAM 97684 97684 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 554596 554596 0 0.0
RAM 205488 205488 0 0.0
lock CC3235SF_LAUNCHXL FLASH 588472 588472 0 0.0
RAM 205736 205736 0 0.0
efr32 lock-app BRD4187C FLASH 964960 964960 0 0.0
RAM 123808 123808 0 0.0
BRD4338a FLASH 759504 759504 0 0.0
RAM 254396 254396 0 0.0
window-app BRD4187C FLASH 1060388 1060380 -8 -0.0
RAM 120036 120036 0 0.0
esp32 all-clusters-app c3devkit DRAM 102844 102844 0 0.0
FLASH 1831630 1831630 0 0.0
IRAM 93540 93540 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 936484 936484 0 0.0
RAM 161740 161740 0 0.0
nxp contact mcxw71+release FLASH 743960 743960 0 0.0
RAM 67152 67152 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1681372 1681372 0 0.0
RAM 214204 214204 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1595940 1595940 0 0.0
RAM 211372 211372 0 0.0
light cy8ckit_062s2_43012 FLASH 1460180 1460180 0 0.0
RAM 197816 197816 0 0.0
lock cy8ckit_062s2_43012 FLASH 1494268 1494268 0 0.0
RAM 225688 225688 0 0.0
qpg lighting-app qpg6200+debug FLASH 838144 838144 0 0.0
RAM 128000 128000 0 0.0
lock-app qpg6200+debug FLASH 776040 776040 0 0.0
RAM 118944 118944 0 0.0
realtek light-switch-app rtl8777g FLASH 700768 700768 0 0.0
RAM 113448 113448 0 0.0
lighting-app rtl8777g FLASH 742696 742696 0 0.0
RAM 114652 114652 0 0.0
stm32 light STM32WB5MM-DK FLASH 470092 470092 0 0.0
RAM 141408 141408 0 0.0
telink bridge-app tl7218x FLASH 710792 710792 0 0.0
RAM 93064 93064 0 0.0
light-app-ota-compress-lzma-shell-factory-data tl3218x FLASH 796794 796794 0 0.0
RAM 41256 41256 0 0.0
light-app-ota-shell-factory-data tl7218x FLASH 788136 788136 0 0.0
RAM 96156 96156 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 716742 716742 0 0.0
RAM 54248 54248 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 752552 752552 0 0.0
RAM 73272 73272 0 0.0
light-switch-app-ota-factory-data tl3218x_retention FLASH 721110 721110 0 0.0
RAM 34976 34976 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 608882 608882 0 0.0
RAM 117944 117944 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 821640 821644 4 0.0
RAM 94544 94544 0 0.0

@mergify mergify bot merged commit ec7aeec into project-chip:master Jan 8, 2026
87 of 88 checks passed
@Alami-Amine Alami-Amine deleted the AA/StopStaticLinking branch February 7, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gn review - pending sdk-maintainer-approved PR marked by `matter-sdk-maintainers` as suitable for MERGE - meets guideline & sufficient reviews.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants