PaulStoffregen / Time

Time library for Arduino
http://playground.arduino.cc/code/time
1.25k stars 666 forks source link

Time.h and time.h issue on windows #159

Closed mars000 closed 3 years ago

mars000 commented 3 years ago

because esp32 already has a built in time.h it gets confused with Time.h as it does not recognize cases. can the developer pls fix this issues once and for all so it works correctly on windows os ?

blackketter commented 3 years ago

Use #include "TimeLib.h" instead.

PaulStoffregen commented 3 years ago

Sadly, this is confirmation we still have a lot of code in the wild which includes Time.h. :-(

PaulStoffregen commented 3 years ago

I am planning to delete Time.h. That's why Time.h gives a warning to use TimeLib.h.

But this issue is also confirmation that obviously a lot of Arduino users have the default setting where Arduino doesn't show the compiler's warning messages, only errors.

Maybe I'll change the warning to an error and publish a new version. Hopefully that will finally get the remaining Time.h includes changed?

PaulStoffregen commented 3 years ago

Ok... done, 1.6.1 release!