konnected-io / konnected-security

Konnected connects wired sensors and switches to SmartThings, Home Assistant, Hubitat and OpenHAB
https://konnected.io
Apache License 2.0
416 stars 322 forks source link

Build fails with nodemcu 3.0 #109

Closed greghaynes closed 4 years ago

greghaynes commented 4 years ago

When following the build docs I get a build failure of the following:

CC app/user/user_main.c
user_main.c:81:42: error: 'NODEMCU_EAGLEROM_PARTITION' undeclared here (not in a function)
     { NODEMCU_PARTITION_EAGLEROM,         0x00000,     0x0B000},
                                          ^
user_main.c:85:42: error: 'NODEMCU_IROM0TEXT_PARTITION' undeclared here (not in a function)
     { NODEMCU_PARTITION_IROM0TEXT,        0x10000,      0x0000},
                                          ^
user_main.c:86:42: error: 'NODEMCU_LFS0_PARTITION' undeclared here (not in a function)
     { NODEMCU_PARTITION_LFS,              0x0,          LUA_FLASH_STORE},
                                          ^
user_main.c:87:42: error: 'NODEMCU_SPIFFS0_PARTITION' undeclared here (not in a function)
     { NODEMCU_PARTITION_SPIFFS,           0x0,          SPIFFS_MAX_FILESYSTEM_SIZE},
                                          ^
user_main.c: In function 'nodemcu_init':
user_main.c:302:5: error: implicit declaration of function 'dbg_printf' [-Werror=implicit-function-declaration]
     NODE_ERR("\n");
     ^
cc1: all warnings being treated as errors
../../Makefile:422: recipe for target '.output/eagle/debug/obj/user_main.o' failed

This seems to be caused by the recent 3.0 "release" of nodemcu. Checking out the latest 2.x release in the nodemcu dir using git checkout 2.2.1-master_20190405 fixes this.

heythisisnate commented 4 years ago

We haven't validated the application on NodeMCU 3.0 yet, so this isn't expected to work. I think we'll have to reconcile the changes in the firmware build config variables and update our overlays here.

Pull requests are welcome if you want to work on this. We typically give out free product or swag for open-source contributions.