[123423] trunk/dports/perl/p5-email-mime-contenttype/Portfile
Ryan Schmidt
ryandesign at macports.org
Sun Aug 10 05:20:33 PDT 2014
On Aug 9, 2014, at 8:31 PM, Joshua Root <jmr at macports.org> wrote:
>
> On 2014-8-10 04:07 , Lawrence Velázquez wrote:
>> On Aug 9, 2014, at 11:47 AM, Frank Schima <mf2k at macports.org> wrote:
>>
>>> Just curious about why you are removing the indenting spacing in these perl ports? I would say it is more common to indent in blocks. I’m open to a discussion about what is preferred. I know when I edit p5 and other ports, I add 4 spaces as indent for all blocks. It makes things much more clear in my mind. I don’t want to get into the situation where people are accidentally reversing each other’s edits! :) I think a consensus would be a good idea here.
>>
>> Many Perl ports don't indent the 'if {${perl5.major} != ""}' block. I've always assumed that it's because that block often contains a significant portion of the ports' logic, and most of the content would end up being indented. I find it rather ugly.
>
> I think I may have scripted the bulk addition of those if statements at
> one point, and not bothered trying to indent them properly.
And I assumed Mojca was trying to go back to that old style in preparation for a future removal of those if statements and going back to a single version of perl, as has been recently discussed but IIRC not yet decided upon.
For now I am just adding all perl versions to ports as I get to them, so that we can address any perl5.18- or perl5.20-specific issues as they come up, and get closer to a time when all ports are shown to work with perl5.20.
More information about the macports-dev
mailing list