Stef-aap / RFLink-ESP

Home Assistant / Domoticz tested with a RFLink, modified for ESP8266 and ESP32
MIT License
108 stars 49 forks source link

Code doesn't compile #9

Closed plampix closed 4 years ago

plampix commented 4 years ago

Hi Stef,

When I try to compile the latest version, I get the following error message:

In file included from /tmp/arduino_build_668737/sketch/Sensor_Wifi.h:33:0, from /tmp/arduino_build_668737/sketch/Sensor_Receiver_2.h:717, from /home/peterp/src/Arduino/Stef-aap_RFLink-ESP/RFLink_ESP/RFLink_ESP.ino:74: /tmp/arduino_build_668737/sketch/Wifi_Settings.h: In function 'void AddAccessPoints(ESP8266WiFiMulti*)': Wifi_Settings.h:31:36: error: 'Local_Wifi_Name' was not declared in this scope _AddAccessPoint ( wifiMulti, Local_Wifi_Name , Local_Wifi_PWD );

I can't find any definition of Local_Wifi_Name in the source, shouldn't this be __SECRET_Wifi_Name instead?

Stef-aap commented 4 years ago

I think you're right (I renamed those constants.

At the moment I'm unable to double-check that.

On 2020-05-04 13:05, plampix wrote:

Hi Stef,

When I try to compile the latest version, I get the following error message:

In file included from /tmp/arduino_build_668737/sketch/Sensor_Wifi.h:33:0, from /tmp/arduino_build_668737/sketch/Sensor_Receiver_2.h:717, from /home/peterp/src/Arduino/Stef-aap_RFLink-ESP/RFLink_ESP/RFLink_ESP.ino:74: /tmp/arduino_build_668737/sketch/Wifi_Settings.h: In function 'void AddAccessPoints(ESP8266WiFiMulti*)': Wifi_Settings.h:31:36: error: 'Local_Wifi_Name' was not declared in this scope _AddAccessPoint ( wifiMulti, Local_Wifi_Name , Local_Wifi_PWD );

I can't find any definition of Local_Wifi_Name in the source, shouldn't this be __SECRET_Wifi_Name instead?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Stef-aap/RFLink-ESP/issues/9, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZRPLP4DUKSQTEKSNVHKVTRP2OQFANCNFSM4MYVB6HQ.

Stef-aap commented 4 years ago

Thanks for finding this bug.

I handcrafted this file, because the original file contains all my Passowrds and other secrets

The file Wifi_Settings.h is corrected on Github.

On 2020-05-04 13:05, plampix wrote:

Hi Stef,

When I try to compile the latest version, I get the following error message:

In file included from /tmp/arduino_build_668737/sketch/Sensor_Wifi.h:33:0, from /tmp/arduino_build_668737/sketch/Sensor_Receiver_2.h:717, from /home/peterp/src/Arduino/Stef-aap_RFLink-ESP/RFLink_ESP/RFLink_ESP.ino:74: /tmp/arduino_build_668737/sketch/Wifi_Settings.h: In function 'void AddAccessPoints(ESP8266WiFiMulti*)': Wifi_Settings.h:31:36: error: 'Local_Wifi_Name' was not declared in this scope _AddAccessPoint ( wifiMulti, Local_Wifi_Name , Local_Wifi_PWD );

I can't find any definition of Local_Wifi_Name in the source, shouldn't this be __SECRET_Wifi_Name instead?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Stef-aap/RFLink-ESP/issues/9, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZRPLP4DUKSQTEKSNVHKVTRP2OQFANCNFSM4MYVB6HQ.

plampix commented 4 years ago

Thanks!