Skip to content

ci: fix esp32s3 -Werror at source (zenoh-pico PR #1083) and get twister green#19

Merged
EnejiOhieku merged 7 commits into
masterfrom
ci/suppress-esp32s3-werror
Jun 30, 2026
Merged

ci: fix esp32s3 -Werror at source (zenoh-pico PR #1083) and get twister green#19
EnejiOhieku merged 7 commits into
masterfrom
ci/suppress-esp32s3-werror

Conversation

@bmagyar

@bmagyar bmagyar commented Jun 22, 2026

Copy link
Copy Markdown
Member
  • Fix the esp32s3 -Werror: pin zenoh-pico to upstream PR #1083 (refs/pull/1083/head), which routes the Zephyr build through the templated config.h instead of clashing -D flags, and enable a real zenoh feature set in the demos. Temporary pin — move back to a tag once #1083 is released.
  • Build inverted_pendulum on esp32s3: rewrite zenbedded_rcl/CMakeLists.txt as a proper zephyr_library() (it was a standalone-app file being pulled in as a module).
  • Drop native_sim from both demos (host libc is incompatible with Zephyr's POSIX + net stack) and add a west update step so the twister job builds against the PR's manifest.

Comment thread demos/rcl_cpp_test/boards/esp32s3_devkitc_esp32s3_procpu.conf Outdated
@bmagyar bmagyar changed the title ci: suppress esp32s3 -Werror and realign twister after restructure ci: fix esp32s3 -Werror at source (zenoh-pico PR #1083) and get twister green Jun 29, 2026
@EnejiOhieku

Copy link
Copy Markdown
Contributor

this is nice. thanks sir

@EnejiOhieku EnejiOhieku merged commit 9fd549d into master Jun 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants