Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
mars000 opened this issue Jun 20, 2021 · 4 comments
Closed

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

mars000 opened this issue Jun 20, 2021 · 4 comments

Comments

@mars000
Copy link

mars000 commented Jun 20, 2021

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
Copy link

Use #include "TimeLib.h" instead.

@PaulStoffregen
Copy link
Owner

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

@PaulStoffregen
Copy link
Owner

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
Copy link
Owner

Ok... done, 1.6.1 release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants