> On Aug 9, 2014, at 12:07 PM, Lawrence Velázquez <[email protected]> wrote:
>
> On Aug 9, 2014, at 11:47 AM, Frank Schima <[email protected]> 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 it’s an artifact of the cpan2port module which adds the dependencies as
p5-* without any if {${perl5.major} != "”} block. Usually people have just
added that line without indenting the dependencies. We always indent the python
ports which have similar logic. I think the code looks better indented. But I
will stay with the consensus on this (of course).
Cheers!
Frank
_______________________________________________
macports-dev mailing list
[email protected]
https://lists.macosforge.org/mailman/listinfo/macports-dev
