search

espressif / esp-sr

Speech recognition
Other
549 stars 103 forks source link

Building error when used in platformIO/ESPHome context (AIS-1527) #92

Closed X-Ryl669 closed 6 months ago

X-Ryl669 commented 6 months ago

When used in ESPHome context (for Home Assistant mainly), any version later than this commit fails CMake to build on this line idf_component_get_property(esp_dsp_lib espressif__esp-dsp COMPONENT_LIB)

This is because espressif__esp-dsp isn't defined so CMake choke on bad number of argument to the idf_component_get_property, it see 2 arguments instead of 3.

I do have a __idf_esp-dsp variable, but not a espressif__esp-dsp variable. I'm able to successfully pass this step if I just replace that line with set(esp_dsp_lib "esp-dsp")

X-Ryl669 commented 6 months ago

@igrr I don't know how the espressif__esp-dsp is created (maybe there's an additional step to perform here that I didn't). Do you have an idea?

igrr commented 6 months ago

Could you please describe how you add esp-sr to your project?

Perhaps you could also attach the build log of a clean (failing) build, as well as the (successful) build log after the modification you made?

X-Ryl669 commented 6 months ago

Nevermind, the issue was that I added the esp-dsp component myself instead of letting esp-sr add it automatically via its dependency.

But I still don't understand how the espressif__esp-dsp variable comes from, can you explain it?