Re: Xpore Autopkgtest Review ( Bug #1035276 )

> On Fri, 29 Dec 2023 at 1:36 PM, Nilesh Patra  wrote:

> On Fri, Dec 29, 2023 at 05:57:13PM +0530, Nilesh Patra wrote:
> > Hi Israel,
> >
> > On Fri, Dec 29, 2023 at 01:01:47PM +0100, Komolehin Israel wrote:
> > > I worked on the Xpore package by providing autopkgtest and embedding
> test
> > > data. CI was successful.
> > >
> > > This closes #1035276 bug.
> > >
> > > I look forward to its review and upload.
> >
> > I have two comments:
> >
> > * Add in the copyright information for the test data.
> > * Add in a script to automate downloading of this test data in future,
> similar to[1]
>
> > You can also look at [2][3][4] which seem more relevant.
>
> > [1]:
> https://salsa.debian.org/med-team/yanagiba/-/blob/master/debian/tests/get-test-data?ref_type=heads
> > [2]:
> https://salsa.debian.org/med-team/dcm2niix/-/blob/debian/master/debian/tests/get-test-data?ref_type=heads
> > [3]:
> https://salsa.debian.org/med-team/resfinder/-/blob/master/debian/tests/get-test-data?ref_type=heads
> > [4]:
> https://salsa.debian.org/med-team/multiqc/-/blob/master/debian/tests/get-test-data?ref_type=heads
>
> Thanks Nilesh,
I will check them out.

Regards,
Israel