lwip-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lwip-devel] (unit) testing of lwip


From: Mate Varga
Subject: Re: [lwip-devel] (unit) testing of lwip
Date: Mon, 01 Jun 2020 11:51:51 +0200
User-agent: Cyrus-JMAP/3.3.0-dev0-519-g0f677ba-fm-20200601.001-g0f677ba6

Ah okay, thanks. I have probably read some old documentation about who to run 
the test suite. I'll submit the other changeset then soon.

On Mon, Jun 1, 2020, at 11:45, goldsimon wrote:
> 
> 
> Am 1. Juni 2020 11:37:28 MESZ schrieb Mate Varga <m@matevarga.net>:
> >Hello,
> >
> >I have made a small bug fix for lwip (#58473), and as a follow-up I'd
> >like to make sure that this simple, compact functionality (converting
> >hostname into a QNAME field) is unit tested. I have written a test and
> >I'm trying to run the test suite, but it looks to me that the suite is
> >broken in various ways (building it under *nix + gcc doesn't work, I
> >can get into details later about how exactly).
> 
> It works for me...
> 
> >
> >I can try and fix the tests, but first I'd like to understand whether
> >it is the intention of the project owners to keep the suite and the
> >examples maintained -- I don't want to submit a patch and find out that
> >it's not welcome :). Also it looks a bit odd that lwip's unit tests are
> >run from and depend on lwip_contrib.
> 
> There's no "lwip_contrib" any more. The files are integrated into the 
> mail repo now. I guess you're not using latest git master?
> 
> Regards,
> Simon
> 
> >
> >Should I invest effort into making some refactoring here? First of all
> >I'd probably try to run unit tests without requiring lwip_contrib.
> >
> >Thanks,
> >Mate
> 
> _______________________________________________
> lwip-devel mailing list
> lwip-devel@nongnu.org
> https://lists.nongnu.org/mailman/listinfo/lwip-devel
>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]