<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">Le sam. 24 nov. 2018 à 11:06, Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com">thomas.petazzoni@bootlin.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello François,<br>
<br>
Thanks a lot for this work on adding tests for Perl packages. It is<br>
really great to see the testing infrastructure being used more widely.<br>
A few questions below.<br>
<br>
On Sat, 24 Nov 2018 10:07:19 +0100, Francois Perrad wrote:<br>
<br>
> +class TestPerllibwwwperl(TestPerlBase):<br>
> +    """<br>
> +    package:<br>
> +        libwww-perl<br>
> +    direct dependencies:<br>
> +        Encode-Locale<br>
> +        File-Listing<br>
> +        HTML-Parser<br>
> +        HTTP-Cookies<br>
> +        HTTP-Daemon<br>
> +        HTTP-Date<br>
> +        HTTP-Message<br>
> +        HTTP-Negotiate<br>
> +        LWP-MediaTypes<br>
> +        Net-HTTP<br>
> +        Try-Tiny<br>
> +        URI<br>
> +        WWW-RobotRules<br>
> +    indirect dependencies:<br>
> +        HTML-Tagset<br>
> +        IO-HTML<br>
> +    """<br>
<br>
Sorry if I haven't followed the previous discussion, but why do we have<br>
this docstring ? Is it just for information ?<br>
<br></blockquote><div><br></div><div>Yes, this is for information.</div><div>By this way, we know that we don't need to write another test for HTTP-Date or IO-HTML.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +    config = TestPerlBase.config + \<br>
> +        """<br>
> +        BR2_PACKAGE_PERL=y<br>
> +        BR2_PACKAGE_PERL_LIBWWW_PERL=y<br>
> +        """<br>
> +<br>
> +    def test_run(self):<br>
> +        self.login()<br>
> +        self.module_test("LWP")<br>
> +        self.module_test("LWP::UserAgent")<br>
> +        self.module_test("LWP::Authen::Basic")<br>
> +        self.module_test("LWP::Authen::Digest")<br>
> +        self.module_test("HTTP::Message")<br>
> +        self.module_test("HTTP::Daemon")<br>
> +        self.module_test("WWW::RobotRules")<br>
<br>
Why are you testing specifically those modules ? What is the rule to<br>
decide what modules are tested here ?<br>
<br></blockquote><div><br></div><div>libwww-perl is a special package/distribution from the beginning of the WWW history</div><div>(started in 1995, see <a href="https://metacpan.org/changes/distribution/libwww-perl">https://metacpan.org/changes/distribution/libwww-perl</a>).</div>In fact, libwww-perl collection is a set of Perl modules.<div>LWP (Lib World-Wide Web Perl) is just a name space for some of them, and doesn't match with the distribution name libwww.<br></div><div>So, we need load most of them.<br></div><div><br></div><div>François<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks!<br>
<br>
Thomas<br>
-- <br>
Thomas Petazzoni, CTO, Bootlin<br>
Embedded Linux and Kernel engineering<br>
<a href="https://bootlin.com" rel="noreferrer" target="_blank">https://bootlin.com</a><br>
_______________________________________________<br>
buildroot mailing list<br>
<a href="mailto:buildroot@busybox.net" target="_blank">buildroot@busybox.net</a><br>
<a href="http://lists.busybox.net/mailman/listinfo/buildroot" rel="noreferrer" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
</blockquote></div></div></div>