libperl stage-qa check and libraries that do not link against libperl.so

[Mathieu Arnold]

+--On 17 janvier 2016 23:12:36 +0100 Raphael Kubo da Costa
<rakuco at FreeBSD.org> wrote:
| Mathieu Arnold <mat at FreeBSD.org> writes:
| 
|> +--On 17 janvier 2016 15:42:46 +0100 Raphael Kubo da Costa
|> <rakuco at FreeBSD.org> wrote:
|> | Warning:
|> | /wrkdir/stage/usr/local/lib/perl5/site_perl/mach/5.20/auto/Net/Libprox
|> | y/L ibproxy.so is not linked with libperl.so.5.20, not respecting
|> | lddlflags? Error: None of the .so in
|> | /wrkdir/stage/usr/local/lib/perl5/site_perl/mach/5.20 are linked with
|> | libperl.so.5.20, see above for the full list.
|> 
|> Ok, there are two kinds of .so that gets into PERL_ARCH:
|> 
|> 1) the Perl modules, that are linked with libperl.so, and for which must
|> have the rpath set to be able to get libperl.so from where it's
|> installed. 2) The rest of them, they're general .so that do not contain
|> Perl code, and are not linked with libperl.so. For these, if the
|> non-linking is intentional, and not needed, ignore the ***WARNING***.
| 
| Right. If linking against libperl.so and not linking against it are both
| fine, does it still make sense to have that stage-qa check?

There is a stage-qa check because if you link against libperl.so, you MUST
have rpath set to the path of libperl.so, it happens if the port is doing
what Perl should be doing and getting it wrong.  Also, sometime, you don't
link with libperl.so, and it's a mistake, and it should be.

There are a few cases where it's not needed, but when I added the qa check
when I changed libperl.so to be versionned, out of all the ports that
install .so in PERL_ARCH, there were, say, 3 or 4 that had a .so that did
not need to be linked with libperl.so.

-- 
Mathieu Arnold
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 949 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-perl/attachments/20160118/134787e1/attachment.sig>