On Sun, 15 Jun 2014 18:47:44 +0300, Niko Tyni wrote:
> There's a pending Perl policy change discussed in #750017 that requires
> the handful of packages in the archive that install nonbinary Perl modules
> into /usr/lib/perl5 (or, more generally, $Config{vendorarch} to add a
> perlapi-* dependency. The attached mostly untested patch implements this
> in dh_perl.
>
> I'll try to find some time to test it myself, but as that could be done by
> others too, I wanted to file this sooner rather than later. Not adding
> the 'patch' tag quite yet, though.
I've tested the patch now, and it didn't work for the first package I
tried (libalien-wxwidgets-perl):
The regexp does not match, $File::Find::dir contains subdirectories
(debian/libalien-wxwidgets-perl/usr/lib/x86_64-linux-gnu/perl5/5.20/Alien
etc.)
If I change
if $File::Find::dir =~ /\Q$vendorarch\E$/;
to
if $File::Find::dir =~ /\Q$vendorarch\E/;
it looks better, and $deps = 10, and we get perlapi-5.20.0-exp1 in Depends.
I've now tested this updated patch with
- 3 of the "special" packages mentioned in #750017
(after removing the manual perlapi-* stuff from d/rules)
- 3 random arch:any packages, and
- 3 random arch:all packages
and haven't seen anything unexpected.
Cheers,
gregor
--
.''`. Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
: :' : Debian GNU/Linux user, admin, and developer - http://www.debian.org/
`. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
`- NP: Sinéad O'Connor: Dancing Lessons
signature.asc
Description: Digital Signature
