-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Roll up Pigweed to 444f6d0 #12648
Roll up Pigweed to 444f6d0 #12648
Conversation
What is failing to link? Why is CI passing without this change? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add some CI/tests to verify pigweed? Sounds like it was broken without us knowing via CI.
It is breaking esp32 all-cluster-app if RPC is turned on. CL didn't catch it since RPC is off according to the script and sdkconfig defaults in sdkconfig_c3devkit.defaults. Do will want to have the cl run RPC turned on? Or both on and off. |
I believe #12700 will add sufficient CI to validate RPC builds for ESP32 (and ipv6 as well). Adding all possible builds is hard because there are a lot of variations, but we should try to have reasonable coverage. |
/rebase |
PR #12648: Size comparison from 6f89b9b to a870b5f Full report (10 builds for k32w, p6, qpg, telink)
|
/rebase |
Fast tracking third party updates |
PR #12648: Size comparison from d9e7e54 to 2e15278 Increases (2 builds for mbed)
Decreases (5 builds for efr32, mbed, nrfconnect)
Full report (30 builds for efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
/rebase |
Fix a linking error with building with pw_trace.
This is due to a recent pw_rpc change.
PR #12648: Size comparison from 586aca7 to 8a63286 Increases (2 builds for mbed)
Decreases (4 builds for mbed, nrfconnect)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fix a linking error with building with pw_trace.