search

Dilbert66 / esphome-vistaECP

This is an implementation of an ESPHOME custom component and ESP Library to interface directly to a Safewatch/Honeywell/Ademco Vista 15/20 alarm system using the ECP interface and very inexpensive ESP8266/ESP32 modules .
GNU Lesser General Public License v2.1
124 stars 21 forks source link

[dev] Compile Problems on Case Sensitive FS #93

Closed rcmurphy closed 1 year ago

rcmurphy commented 1 year ago

vistaAlarm.h is referenced as vistaalarm.h; I was going to submit a PR to just change the include, but I think a better solution would be to switch to lower case filenames for .cpp / .h.

Dilbert66 commented 1 year ago

Yes, I was switching the includes to lower case for a few includes (except for the serial library for now). Not sure where you got the reference error though. I had not pushed the update yet. I'll do that now.

Dilbert66 commented 1 year ago

Interesting. i just did a push and it did not change the case on the pushed version but my copy is fine. I guess, git doesnt change the case. I have to read up on this and correct it.

Dilbert66 commented 1 year ago

Was a git setting to tell it not to ignore case. I repushed again with corrected file names.

rcmurphy commented 1 year ago

Thank you!