Hi Andreas,
I've looked at `libtfbs-perl`, it seems that it already has testsuite
similar to those produced by `autopkgtest-pkg-perl`.
If you think it's better to use it instead of your testsuite in
debian/tests/, I can change it (but for me it checks almost the same things
- and if debian/tests/control present, tests from autopkgtest-pkg-perl
won't be run if not to allow them in control file like this:
http://anonscm.debian.org/cgit/pkg-perl/packages/pkg-perl-tools.git/tree/autopkgtest/examples/default-tests-control
).
There is a problem with one of tests: t/04_DB_TRANSFAC.t (you skip this
test in run-unit-test).
It seems that this tests fails because service is not available anymore,
not because of network problems.
When I open url, produced by following line in script, by hand:
my $pfm1 = $db->get_Matrix_by_acc('M00039');
it redirects me to this page:
http://www.cbil.upenn.edu/tess?request=MTX-DBRTRV-Accno&key=M00039
It seems that this test fails not because network errors, but because that
online service is not reachable any more. And probably it should be removed
from testsuite, how do you think?
2016-06-24 17:22 GMT+03:00 Andreas Tille <ti...@debian.org>:
> Hi Bas,
>
> I moved the package to Git and importet your patches.
>
>
> Tanya, since this seems to be the right moment and you have gathered
> experience
> with testst in Perl packages: Would you mind adding a test to
>
> https://anonscm.debian.org/git/debian-med/libtfbs-perl.git
>
> to have a soon upload after moving VCS with sensible changes?
>
> Kind regards
>
> Andreas.
>
> On Fri, Jun 24, 2016 at 02:09:19PM +0200, Sebastiaan Couwenberg wrote:
> > On Fri, 17 Jun 2016 02:45:14 +0200 Bas Couwenberg wrote:
> > > As part of the PDL update to 2.016 [0], all its reverse dependencies
> have
> > > been rebuilt in preparation of the transition (from pdlapi-10 to
> pdlapi-12).
> > >
> > > While libtfbs-perl doesn't require changes for the upcoming pdl
> > > transition, I couldn't resist updating the package to incorporate some
> > > best practices.
> > >
> > > Please condider applying the attached patches.
> > >
> > > [0]
> https://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/2016-June/100093.html
> >
> > A couple additional patches have been applied in todays NMU.
> >
> > Kind Regards,
> >
> > Bas
>
>
>
> > _______________________________________________
> > Debian-med-packaging mailing list
> > debian-med-packag...@lists.alioth.debian.org
> >
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-med-packaging
>
>
> --
> http://fam-tille.de
>
--
Best wishes,
Tanya.
