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

SNTP: backport espressif sntp api for lwip2 #7097

Merged
merged 3 commits into from
Feb 22, 2020
Merged

Conversation

d-a-v
Copy link
Collaborator

@d-a-v d-a-v commented Feb 21, 2020

@arendst Can you try it ?

fixes #7092

@arendst
Copy link

arendst commented Feb 22, 2020

Thx.

Solves my issue indeed.

@d-a-v d-a-v added this to the 2.7.0 milestone Feb 22, 2020
@d-a-v d-a-v merged commit f69e404 into esp8266:master Feb 22, 2020
@Jason2866
Copy link
Contributor

Jason2866 commented Feb 22, 2020

Any idea why the code size for Tasmota increases from core v.2.6.1 to feature stage from
568k to 576k ?

Even more bad news. Tasmota does not work reliable with latest. Device hangs and sensors connected to i2c bus doesnt work anymore

@d-a-v
Copy link
Collaborator Author

d-a-v commented Feb 22, 2020

@Jason2866 More details are needed. Did you try 2.6.3 ? Can you open a new issue to track these regressions ?

@d-a-v d-a-v deleted the sntpbkprt branch February 22, 2020 17:03
@Jason2866
Copy link
Contributor

@d-a-v will do if have more tested and info collected.
So far i have tested Tasmota does work until commit e752e96
Is this info enough? If yes i open a issue.

@Jason2866
Copy link
Contributor

@d-a-v opened issue #7103. Found the PR which generates the problems

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

Successfully merging this pull request may close these issues.

After Stage merge #6993 I lost sntp functionality
3 participants