svn commit: r365227 - head/Mk/Uses

[Antoine Brodin]

On Sun, Aug 17, 2014 at 9:38 PM, Kurt Jaeger <pi at freebsd.org> wrote:
> Hi!
>
>> > Log:
>> >   Mk/Uses/perl5.mk: Fix logic for removal of empty .bs files
> [...]
>
>> >  # Starting with perl 5.20, the empty bootstrap files are not installed any
>> >  # more.  As we don't need them anyway, remove it altogether.
>> > -.if ${PERL_LEVEL} < 502000
>> > +.if ${PERL_LEVEL} >= 502000
>> >  fix-perl-bs:
>> >         -@${FIND} ${STAGEDIR} -name '*.bs' -size 0 -delete
>> >  .endif
>>
>>
>> This is totally wrong, please revert.
>
> I had the case with devel/p5-File-Map that it still installs a
> .bs in the STAGEDIR. So I searched on how to fix it and found this
> please in Mk/Uses/perl5.mk and thought:
>
> Isn't the logic the wrong way around ?
>
> Please explain the logic behind the fix-perl-bs for ${PERL_LEVEL} >= 502000 ?

The logic is that with perl 5.20,  you don't need to remove the .bs files.

Cheers,

Antoine