[103118] trunk/dports/devel

[Ryan Schmidt]

On Feb 15, 2013, at 11:05, and.damore at macports.org wrote:

> Revision: 103118
>          https://trac.macports.org/changeset/103118
> Author:   and.damore at macports.org
> Date:     2013-02-15 09:05:11 -0800 (Fri, 15 Feb 2013)
> Log Message:
> -----------
> new port: ninka, source code license identification tool


> Added: trunk/dports/devel/ninka/Portfile


> +depends_lib         bin:/bin/perl:perl5.12

This should be bin:perl:perl5 (the perl5.12 port does not provide a binary called "perl"; the perl5 port does (at least until #29763 is resolved)).


> +post-patch {
> +    reinplace "s|%%PREFIX%%|${prefix}/share/${name}|" ${worksrcpath}/ninka.pl
> +}

That's weird; based on the placeholder's name, you'd think "%%PREFIX%%" would be replaced with only "${prefix}".


> +destroot {
> +    xinstall -m 755 ${worksrcpath}/ninka.pl ${destroot}/${prefix}/bin/ninka

There should not be a "/" before "${prefix}".