From mboxrd@z Thu Jan 1 00:00:00 1970 From: Duncan Hare Date: Mon, 14 May 2018 15:26:04 +0000 (UTC) Subject: [U-Boot] net: [PATCH v10 3/3] Adding wget In-Reply-To: References: <20180414234336.26636-1-DH@synoia.com> <20180414234336.26636-4-DH@synoia.com> <217820715.1487025.1524002336830@mail.yahoo.com> <2109276801.3390689.1524453754403@mail.yahoo.com> <2098767198.759415.1524666799073@mail.yahoo.com> <566131510.980666.1524700329499@mail.yahoo.com> <2087144993.768107.1526245517987@mail.yahoo.com> Message-ID: <485806508.1037928.1526311564721@mail.yahoo.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de From: Simon Glass To: Duncan Hare =20 Cc: "joe.hershberger at ni.com" ; U-Boot Mailing Li= st Sent: Sunday, May 13, 2018 3:00 PM Subject: Re: net: [U-Boot] [PATCH v10 3/3] Adding wget =20 >>>> >>>>Please setup a test that can run in an environment without the >>>>Internet. That is critical for unit tests. >>>> >>>>Hand tests for Internet usage and the environmental effects are great, >>>>but that can't be what we include in the auto tests for repeat-ability >>>>reasons. Simon is asking for a separate type of test. >>> >>> Is the test environment another patch in the series or an addition to t= he >>> wget patch? >I suggest a separate patch. >>Regards,>>Simon Separate patch, patch 4 of the series or a completely new patch? Conceptual approach: apt-get install nginx (in debian) I'll provide a config file pointing to test kernel a a specified file locat= ion. Test kernel will be numbered lines of printable characters. Printable, not = binary, easy to expand in a spreadsheet. Can the u-boot print command print sections of memory? That is print $loadaddr $length?=20 That will work for small test kernels, or=20 I tested by using tftp to download a kernel, then used a modified wget to d= ownload a seconf time and verify thedownloads were identical. compare $loadaddr1 $loadaddr2 $length Is that a preferred approach? =C2=A0Duncan Hare 714 931 7952 =20