[Debian-med-packaging] Bug#827499: libtfbs-perl: Patches for misc packaging improvements

[Andreas Tille]

On Sat, Jun 25, 2016 at 11:23:10AM +0300, merlettaia wrote:
> 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/,

Funny - thanks for explaining me that I did something I simply forgot
and failed to check. ;-)


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?

ACK.  Feel free to commit a patch.

Thanks a lot for double checking my work

   Andreas.
 
> 2016-06-24 17:22 GMT+03:00 Andreas Tille <tille at 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-packaging at lists.alioth.debian.org
> > >
> > http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-med-packaging
> >
> >
> > --
> > http://fam-tille.de
> >
> 
> 
> 
> -- 
> Best wishes,
> Tanya.

-- 
http://fam-tille.de