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

Build fails with latest docker image (esphome/esphome:dev) #2285

Closed
vijaygill opened this issue Jul 31, 2021 · 2 comments · Fixed by esphome/esphome#2104
Closed

Build fails with latest docker image (esphome/esphome:dev) #2285

vijaygill opened this issue Jul 31, 2021 · 2 comments · Fixed by esphome/esphome#2104

Comments

@vijaygill
Copy link

The problem

Compiling code with latest docker image fails with following error:

`

Compiling .pioenvs/distancemeter-esp32/src/esphome/components/mqtt/mqtt_light.cpp.o
src/esphome/components/mqtt/mqtt_light.cpp:2:56: fatal error: esphome/components/light/light_json_schema.h: No such file or directory
compilation terminated.
*** [.pioenvs/distancemeter-esp32/src/esphome/components/mqtt/mqtt_light.cpp.o] Error 1

`

What is version of ESPHome has the issue?

latest dev

What type of installation are you using?

Docker

What platform are you using?

ESP32

Board

D1 Mini

Component causing the issue

src/esphome/components/mqtt/mqtt_light.cpp

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@vijaygill vijaygill changed the title Build failks with latest docker image (esphome/esphome:dev) Build fails with latest docker image (esphome/esphome:dev) Jul 31, 2021
@paulmonigatti
Copy link

It appears that the MQTT Light component is being (incorrectly?) included when no lights have been defined in the config file.

As a temporary workaround, adding an empty light: into the config will force the missing files to be included.

@vijaygill
Copy link
Author

It appears that the MQTT Light component is being (incorrectly?) included when no lights have been defined in the config file.

As a temporary workaround, adding an empty light: into the config will force the missing files to be included.

thanks @paulmonigatti that worked!

OttoWinter added a commit to esphome/esphome that referenced this issue Aug 1, 2021
OttoWinter added a commit to esphome/esphome that referenced this issue Aug 1, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Nov 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants