[33790] trunk/dports/devel/cut/Portfile

[Juan Manuel Palacios]

On Feb 5, 2008, at 1:04 PM, Ryan Schmidt wrote:

>
> On Feb 5, 2008, at 07:40, brett at macports.org wrote:
>
>> Revision: 33790
>>          http://trac.macosforge.org/projects/macports/changeset/33790
>> Author:   brett at macports.org
>> Date:     2008-02-05 05:40:09 -0800 (Tue, 05 Feb 2008)
>>
>> Log Message:
>> -----------
>> linted
>>
>> Modified Paths:
>> --------------
>>    trunk/dports/devel/cut/Portfile
>>
>> Modified: trunk/dports/devel/cut/Portfile
>> ===================================================================
>> --- trunk/dports/devel/cut/Portfile	2008-02-05 13:28:15 UTC (rev  
>> 33789)
>> +++ trunk/dports/devel/cut/Portfile	2008-02-05 13:40:09 UTC (rev  
>> 33790)
>> @@ -1,12 +1,11 @@
>> -# -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode:  
>> nil; \
>> -# c-basic-offset: 4 -*-  
>> vim:fenc=utf-8:filetype=tcl:et:sw=4:ts=4:sts=4
>> # $Id$
>
> [snip]
>
> Actually, we are trying to *add* modelines to portfiles, not remove  
> them, though I wish we would standardize on a single modeline and  
> not just whatever someone feels like using.


	Modelines are required for base source files (because the belong to  
all of us) but only suggested for Portfiles (because, in a way, they  
belong more to the corresponding port maintainers). But that put  
aside, there's definitely a standard for modelines, documented in base/ 
HACKING. More specifically, the modeline for Tcl files reads:


-*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; c- 
basic-offset: 4 -*- vim:fenc=utf-8:filetype=tcl:et:sw=4:ts=4:sts=4

	Which encompasses both [x]emacs and vi[m]. If that does not suit in  
any way a particular maintainer, I think its fair to allow him/her to  
choose the modeline that best fits his/her needs (with an inclination  
to the proposed standard for openmaintainer and no maintainer ports,  
of course)

	Regards,...


-jmpp


>
>
>
> "port lint" in MacPorts 1.6.0 is broken in its check for modelines.  
> This was fixed in trunk already:
>
> http://trac.macosforge.org/projects/macports/ticket/13496
>
>
>
>
> _______________________________________________
> macports-dev mailing list
> macports-dev at lists.macosforge.org
> http://lists.macosforge.org/mailman/listinfo/macports-dev