[103118] trunk/dports/devel

[Andrea D'Amore]

On Fri, Feb 15, 2013 at 11:19 PM, Ryan Schmidt <ryandesign at macports.org> wrote:
> On Feb 15, 2013, at 11:05, and.damore at macports.org wrote:
>> +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)).

Also the bin:: dependency already looks into $prefix/bin, I just got it wrong.

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

It's just due to the habit of using %%PREFIX%%, it won't harm but feel
free to change it if you're so inclined.

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

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

This was a typo, I catched a couple other of these but not this one
while diff'ing before the commit.

Fixed 103158.


--
Andrea