svn commit: r350132 - in head/security/p5-Crypt-Rabbit: . files

[Steven Kreuzer]

On Fri, Apr 4, 2014 at 11:19 AM, Alexey Dokuchaev <danfe at freebsd.org> wrote:
> On Fri, Apr 04, 2014 at 03:11:43PM +0000, Steven Kreuzer wrote:
>> New Revision: 350132
>> URL: http://svnweb.freebsd.org/changeset/ports/350132
>> QAT: https://qat.redports.org/buildarchive/r350132/
>>
>> Log:
>>   Fix build on versions of FreeBSD that don't have gcc as the default
>>   compiler, remove the flag schedule-insns2 passed to cc
>>
>> +@@ -1,6 +1,6 @@
>> + use ExtUtils::MakeMaker;
>> +
>> +-my $opt = '-O3 -funroll-loops -fomit-frame-pointer -fschedule-insns2 -Os -fno-strength-reduce -Wall -pipe';
>> ++my $opt = '-O3 -funroll-loops -fomit-frame-pointer -Os -fno-strength-reduce -Wall -pipe';
>
> Does it make sense to have both -O3 and -Os?  Wouldn't it be better if it
> simply uphold our standard $CFLAGS?

Yeah, very good point. this has been fixed in revision 350979. Thanks