danielkucera / esp-arduino-ebus

67 stars 11 forks source link

Platform io upgrade #4

Closed HiQual closed 1 year ago

HiQual commented 1 year ago

Hello. I see there is a SW change to convert adapter function to Read Only. I used the Platform io option to do the update. Fortunately, it failed at the upload because I had forgotten to disable my Windows firewall. I noted some compile error messages during the compile per the attached img file.

image

Compiling .pio/build/esp12e-ota/lib4d9/ESP8266WiFi/ESP8266WiFiAP.cpp.o src/main.cpp: In function 'void setup()': src/main.cpp:68:53: warning: 'void reset_config()' is deprecated: Use IRAM_ATTR in place of ICACHE_RAM_ATTR to move functions into IRAM [-Wdeprecated-declarations] 68 | attachInterrupt(digitalPinToInterrupt(RESET_PIN), reset_config, CHANGE); | ^~~~ src/main.cpp:46:22: note: declared here 46 | ICACHE_RAM_ATTR void reset_config() { | ^~~~ src/main.cpp:68:73: warning: 'void reset_config()' is deprecated: Use IRAM_ATTR in place of ICACHE_RAM_ATTR to move functions into IRAM [-Wdeprecated-declarations] 68 | attachInterrupt(digitalPinToInterrupt(RESET_PIN), reset_config, CHANGE); | ^ src/main.cpp:46:22: note: declared here 46 | ICACHE_RAM_ATTR void reset_config() { | ^~~~ Compiling .pio/build/esp12e-ota/lib4d9/ESP8266WiFi/ESP8266WiFiGeneric.cpp.o

Before I try the firmware upload again, could you let me know if the compile error messages can be ignored? Many thanks ... Ian

danielkucera commented 1 year ago

Hi Ian, sorry for late reply, this is only warning, it is safe to continue but I have updated the code so it doesn't complain anymore. Feel free to download the latest version.

danielkucera commented 1 year ago

Try to find the IP address of the adapter and replace esp-ebus.local in platformio.ini with that IP

HiQual commented 1 year ago

I had forgotten to start Python for Windows b4 running the update. That fixed it. Sorry to have bothered you. The adapter has been really useful and helped me to fine tune my heating system for maximum efficiency. Many thanks again.