Do we have Md5 for fetch?

"Ryan Schmidt" <[email protected]> wrote:

>On Apr 13, 2010, at 10:39, [email protected] wrote:
>
>> Revision: 66441
>>          http://trac.macports.org/changeset/66441
>> Author:   [email protected]
>> Date:     2010-04-13 08:39:15 -0700 (Tue, 13 Apr 2010)
>> Log Message:
>> -----------
>> Added port for NERD Tree Vim plugin
>
>> +fetch {
>> +    xinstall -d ${prefix}/var/macports/distfiles/${name}/
>> +    set dl 
>> ${prefix}/var/macports/distfiles/${name}/${distname}${extract.suffix}
>> +    foreach site ${master_sites} {
>> +        if {[file exists $dl]} {
>> +            if {[file size $dl] == 0} {
>> +                system "curl ${site} > ${dl}"
>> +            }
>> +        } else {
>> +            system "curl ${site} > ${dl}"
>> +        }
>> +    }
>> +}
>
>Why override the fetch phase?
>
>
>> +    # rebuild help tags
>> +    system "${prefix}/bin/vim -c \":helptags 
>> /opt/local/share/vim/vim72/doc\" -c \":q\" "
>
>You should use ${prefix}, not hardcode /opt/local. Also it appears that here 
>inside the destroot phase, you're doing operations outside of the destroot 
>directory.
>
>_______________________________________________
>macports-dev mailing list
>[email protected]
>http://lists.macosforge.org/mailman/listinfo.cgi/macports-dev
>
_______________________________________________
macports-dev mailing list
[email protected]
http://lists.macosforge.org/mailman/listinfo.cgi/macports-dev

Reply via email to