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

Upgrade lwIP to 2.0.3 #939

Merged
merged 2 commits into from
Mar 21, 2018
Merged

Conversation

ccollins476ad
Copy link
Contributor

This upgrades to the latest stable version of lwIP: 2.0.3. Currently, we are using 2.0.2.

@utzig
Copy link
Member

utzig commented Mar 21, 2018

You need to remove include/netif/ppp/pppoe.h and src/netif/ppp/pppoe.c because they are category X license (4-clause BSD).

@ccollins476ad
Copy link
Contributor Author

Thanks for the reminder up, @utzig! I remember the nightmare those files caused in a previous release. I have removed the two files.

@@ -54,7 +54,7 @@ extern "C" {
/** x.X.x: Minor version of the stack */
#define LWIP_VERSION_MINOR 0
/** x.x.X: Revision of the stack */
#define LWIP_VERSION_REVISION 1
Copy link
Member

@utzig utzig Mar 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm, either we had 2.0.1 or they forgot to change this number when upgrading to 2.0.2 :-P

@utzig
Copy link
Member

utzig commented Mar 21, 2018

I will definitely not test this, but PR looks ok!

@ccollins476ad ccollins476ad merged commit f36351b into apache:master Mar 21, 2018
@ccollins476ad ccollins476ad deleted the lwip-2.0.3 branch March 21, 2018 18:27
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.

None yet

2 participants