Uh oh!
There was an error while loading. Please reload this page.
fix(rp2040): resolve bundled Arduino-Pico libraries - #1249
Conversation
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
📝 WalkthroughWalkthroughRP2040 builds now discover bundled framework libraries, resolve selected dependencies, compile their sources, and include their directories. WiFi discovery is tested. Workspace and Python package versions were updated to 2.5.8. ChangesRP2040 framework library integration
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant RP2040Orchestrator
participant Rp2040Cores
participant LDFResolver
participant Compiler
RP2040Orchestrator->>Rp2040Cores: Get bundled libraries and include directories
RP2040Orchestrator->>LDFResolver: Resolve declared framework dependencies
LDFResolver-->>RP2040Orchestrator: Return selected framework sources
RP2040Orchestrator->>Compiler: Compile core and selected framework sources
RP2040Orchestrator->>Compiler: Add framework include directories
Possibly related issues
Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Fixes#1248.\n\nArduino-Pico framework libraries are now discovered by the RP2040/RP2350 package and the RP orchestrator follows the shared framework-library dependency flow already used by STM32 and Teensy. This exposes headers such as WiFi.h and compiles only selected framework-library sources.\n\nIncludes a regression test for the bundled WiFi library and bumps fbuild to 2.5.8 so the FastLED RP2350W bring-up can consume the fix.
Summary by CodeRabbit