Bug#870334: pkg-perl-autopkgtest: revisiting smoke prove --recurse

[gregor herrmann]

On Thu, 03 Aug 2017 20:30:11 +0300, Niko Tyni wrote:

> > But yeah, if a realiable implementation of "find out build tests and
> > run them during autopkgtest" is possible then it probably makes
> > sense. And ...
>   
> > > The 'make -n -p | grep TEST_FILES' thing seems most promising to me so far
> > > (even if it's for EU::MM only.)
> 
> I played with this a bit and while it seems to work, I think it
> feels a bit too intrusive to me.  The second package I tried it with,
> libbest-perl, generates tests when running Makefile.PL and then lists
> them one by one in TEST_FILES. So we'd need to do this at least before
> copying the tests to the tempdir, and the skipped tests might then become
> a problem...
> 
> Also I'm a bit worried about other side effects that running Makefile.PL
> might have.

Thanks for investigating further.
I also have a slighly uneasy gut feeling about intrusiveness and side
effects ...
 
> For now, I've pushed what I have to the ntyni/autopkgtest-smoke-list
> branch as a40524732a6967fc64030e51b00f4998b137208a .

Great. The actual code change looks nice and clean.
 
> Any urgency this issue had is gone now that you did all the work of
> fixing the --recurse regressions. Many thanks for that :)

That's what we are here for :)
 
> I think I'm leaving this for now. We can see if it becomes a problem
> later, and if not, just close this.

I agree that it might make sense to just monitor the future
development with the test results and come back to this question
later, if necessary, or leave it at that if possible.


Cheers,
gregor

-- 
 .''`.  https://info.comodo.priv.at/ - Debian Developer https://www.debian.org
 : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 AA06
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 963 bytes
Desc: Digital Signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20170803/d4ec2afc/attachment-0001.sig>