[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
>