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

TypeError: Couldn't build proto file into descriptor pool: duplicate symbol 'pw.rpc.EchoMessage' #35870

Open
VaishaliAvhale opened this issue Oct 2, 2024 · 3 comments
Labels
bug Something isn't working needs triage

Comments

@VaishaliAvhale
Copy link

VaishaliAvhale commented Oct 2, 2024

Reproduction steps

  1. connectedhomeip$ source script/bootstrap.sh
  2. connectedhomeip$ source script/activate.sh
  3. Enable RPCs in the build using menuconfig and build and flash the lighting-app/esp32
  4. build pigweed/rpc_console and execute chip-console --device /dev/ttyUSB0 command

Bug prevalence

Everytime

GitHub hash of the SDK that was being used

83159c2

Platform

esp32

Platform Version(s)

V1.4

Anything else?

ESP-IDF version: v5.3
commit e0991facf5ecb362af6aac1fae972139eb38d2e4 (HEAD, tag: v5.3)
image

@VaishaliAvhale VaishaliAvhale added bug Something isn't working needs triage labels Oct 2, 2024
@dhrishi
Copy link
Contributor

dhrishi commented Oct 25, 2024

@andy31415 This is about trying PW RPC on the ESP32 platform. Can the label be changed to Google or pigweed?

@andy31415 andy31415 removed the esp32 label Oct 25, 2024
@andy31415
Copy link
Contributor

Removed the label - it looks like the chip-console has issues here. Unsure about who maintains pw-rpc things though at this point.

@andy31415
Copy link
Contributor

I was updating some logging and asserts packages, when checking RPC I hit this in #36297

It seems that the issue is that the chip console was not maintained (we seem to have no tests?) and it changged over time:

  • pw_system.console already imports the echo service
  • several arguments removed (no ipython, no output, several flags changed)

I updated it to work in that PR and tested locally on linux so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage
Projects
None yet
Development

No branches or pull requests

3 participants