[Bug 1001873] Patches to upgrade lwip to 1.4.1

[bugzilla-daemon@bugs.ecos.sourceware.org]

Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001873

Will Wagner <willw@carallon.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |MODIFIED

--- Comment #27 from Will Wagner <willw@carallon.com> ---
Hi Ilija,

We actually did some work on this at the end of last year, but then it got lost
in amongst other things.

(In reply to comment #25)
> 
> 1. Compilation of tests breaks on compiling the PPP test. The following
> lwip_net.cdl hack makes it compile:
> 
>          calculated      { "tests/httpd_sequential tests/httpd_simple " .
> -                          "tests/nc_test_slave tests/ppp " .
> +                          "tests/nc_test_slave " .
>                            "tests/socket tests/sys_timeout tests/tcpecho " .
>                            "tests/udpecho" }
> 
> This is, however, a workaround rather than fix. The PPP test program should
> be either fixed (synchronised with PPP upgrade) or, if obsolete, removed
> completely.

We have a patch I'll attach shortly that removes the test. Neither of us who
have worked on this are in any way ppp experts, however we felt it was just not
possible to port the test to the new version of lwip so we have removed it.

> 
> 2. There are still some compiler warnings ( GCC 4.6.3 - eCos GNU Tools
> 4.6.3-20120623 )

We see no warnings (CodeConfidence tools GCC 4.3.2). If you give details we can
try to take a look at them.

> 
> 3. There's no ChangeLog record.

Will attach patch shortly

> 
> 4. Copyright banners of eCos specific files that you have touched should be
> updated.

Will attach patch shortly

-- 
You are receiving this mail because:
You are on the CC list for the bug.