Could Debian Perl team take over PDL?
Niko Tyni
ntyni at debian.org
Fri Jun 17 17:14:23 UTC 2016
On Fri, Jun 17, 2016 at 02:51:11AM +0200, Sebastiaan Couwenberg wrote:
> Only a few hardening lintian issues remain (mostly
> no-fortify-functions), that I've not been able to resolve yet. More
> importantly we should get the two binary-or-shlib-defines-rpath errors
> resolved. The tests failures are also not fixed yet.
Thanks for your work, this is awesome!
I skimmed through the pdl packaging in git, and a few things I noticed:
- the scantree.pl call in pdl.postinst can't work without
doc_dirlist.patch I think? It's passing in multiple directories but
the unpatched scantree.pl doesn't handle those
- the existing slatec_default_integer_8.patch fixes all the test failures
expect t/minuit.t for me on amd64
- the existing minuit_default_integer_8.patch fixes the remaining one
- there's some trivial conflict between the new spelling-errors.patch
and minuit_default_integer_8.patch, resolved with the attached patch
- I was a bit surprised that doc_vendor_install.patch is now needed, but
apparently it's because upstream v2.007_03 added a "missing"
doc_vendor_install target to Makefile.PL. Conceptually, running
scantree.pl and mkhtmldoc.pl during the package build/install
phase seems useless to me, as they will be re-run from the postinst
when the package is actually installed. I'm not sure if it would
make more sense to just patch that away from Makefile.PL? And
if we don't, I guess the results shouldn't go in places like
/usr/lib/x86_64-linux-gnu/perl5/5.22/PDL/HtmlDocs but rather under
/var ?
Hope this makes sense; I'm still far from a domain expert on the PDL stuff...
--
Niko
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Amend-minuit-patch-after-spelling-errors.patch.patch
Type: text/x-diff
Size: 1889 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20160617/fd9378b9/attachment-0001.patch>
More information about the pkg-perl-maintainers
mailing list