nkitanov / iaq_board

IAQ Board is a DIY (Do-It-Yourself) device for measuring internal air quality
370 stars 33 forks source link

Firmware doesnt compile on 2022.12.X ESPHome - linking problem #27

Closed Dinth closed 8 months ago

Dinth commented 1 year ago

Hi. After the update to 2022.12 the firmware stopped compiling.

Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/esp32-hal-uart.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/firmware_msc_fat.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/libb64/cdecode.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/libb64/cencode.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/main.cpp.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/stdlib_noniso.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/wiring_pulse.c.o
Compiling /data/iaq-kitchen/.pioenvs/iaq-kitchen/FrameworkArduino/wiring_shift.c.o
Archiving /data/iaq-kitchen/.pioenvs/iaq-kitchen/libFrameworkArduino.a
Linking /data/iaq-kitchen/.pioenvs/iaq-kitchen/firmware.elf
/data/cache/platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: /data/cache/platformio/packages/framework-arduinoespressif32/tools/sdk/esp32/lib/libvfs.a(vfs.c.obj): in function `esp_vfs_utime':
/Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/vfs/vfs.c:585: multiple definition of `utime'; /data/iaq-kitchen/.pioenvs/iaq-kitchen/src/main.cpp.o:/config/esphome/.esphome/build/iaq-kitchen/src/main.cpp:68: first defined here
collect2: error: ld returned 1 exit status
*** [/data/iaq-kitchen/.pioenvs/iaq-kitchen/firmware.elf] Error 1
========================= [FAILED] Took 59.34 seconds ==================
fraserp commented 1 year ago

Experiencing the same, have cleaned the build files to no avail.

paveldn commented 1 year ago

Change line 427 in iaq_board.yaml:

    - platform: uptime
      name: Uptime
      id: utime

to

    - platform: uptime
      name: Uptime
      id: utime_sensor

worked for me

fraserp commented 1 year ago

Confirmed working

Dinth commented 1 year ago

Works for me too. Would you mind submitting a PR?

nkitanov commented 1 year ago

@Dinth I appreciate loggin PR if that solves the issues. I'd suggest adding a comment which is the required version of esphome. I do not actively maintain this project anymore and all my devices still use the original binary build. Thanks for your contribution!

Badn1nja commented 8 months ago

I just realised this is still an issue and have created this PR

nkitanov commented 8 months ago

I just realised this is still an issue and have created this PR

Thanks for your contribution. I can't test your code now, but if you can confirm it is building with the merged yaml I will close this issue.

Badn1nja commented 8 months ago

It's paveldn's change I have just made the PR - but yes It's working on the latest ESPHome, I have used this change since early 2023

Badn1nja commented 8 months ago

Sorry I should have made another branch for testing.... I have made a PR to fix this version..