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

preparations for lwip2 #707

Merged
merged 1 commit into from
Jan 13, 2018
Merged

preparations for lwip2 #707

merged 1 commit into from
Jan 13, 2018

Conversation

s0170071
Copy link
Contributor

does compile with lwip2, may need tcpCleanup() fixed

@psy0rz
Copy link
Member

psy0rz commented Jan 12, 2018

whats lwip2? some kind of new version of the tcpip stack? whats changed? cant find anything on google quickly. :)

@psy0rz psy0rz closed this Jan 13, 2018
@letscontrolit letscontrolit deleted a comment Jan 13, 2018
@psy0rz psy0rz reopened this Jan 13, 2018
@psy0rz
Copy link
Member

psy0rz commented Jan 13, 2018

sorry for the weird things that happened, we had a spambot on github.

@beicnet
Copy link
Contributor

beicnet commented Jan 13, 2018

Yeah, saw it and reported it also, he is gone now! 😉

@psy0rz
Copy link
Member

psy0rz commented Jan 13, 2018

thanks, github was very quick, i like that :) like within 5 or 10 minutes he was gone. :)

@s0170071
Copy link
Contributor Author

see discussion here: esp8266/Arduino#4080
There it says:

The newer lwip solves more issues than it creates...

LWIP for ESP8266 repo is here: https://github.com/d-a-v/esp82xx-nonos-linklayer/tree/arduino-2.4.0
The issue they meant is esp8266/Arduino#3362

I think we should adapt it. You may have noticed that I do like to keep the ESPMega branch compiling with the latest core. I am aware of the fact that this is experimental and things may not work. Therefore the #ifdefs.
If you think I am messing things up around here please let me know :-)

@psy0rz
Copy link
Member

psy0rz commented Jan 13, 2018

Ahh thanks. Well this is the espeasy development branch and i expect everything to migrate to core 2.4 anyway. So thanks:)

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.

3 participants