[50905] trunk/dports/science

[Ryan Schmidt]

On May 12, 2009, at 10:55, ram at macports.org wrote:

> Revision: 50905
>           http://trac.macports.org/changeset/50905
> Author:   ram at macports.org
> Date:     2009-05-12 08:55:36 -0700 (Tue, 12 May 2009)
> Log Message:
> -----------
> science/lscsoft-deps: add new lscsoft dependencies meta-port
>
> Added Paths:
> -----------
>     trunk/dports/science/lscsoft-deps/
>     trunk/dports/science/lscsoft-deps/Portfile
>
> Added: trunk/dports/science/lscsoft-deps/Portfile
> ===================================================================
> --- trunk/dports/science/lscsoft-deps/ 
> Portfile	                        (rev 0)
> +++ trunk/dports/science/lscsoft-deps/Portfile	2009-05-12 15:55:36  
> UTC (rev 50905)
> @@ -0,0 +1,51 @@
> +# $Id$
> +
> +PortSystem    1.0
> +
> +name          lscsoft-deps
> +version       20090512
> +categories    science
> +maintainers   ram
> +platforms     darwin
> +
> +description   LSCSoft dependencies meta-port
> +long_description  This is a meta-port that depends on all the ports \
> +  needed for developing LSC Software including glue, lal, and  
> lalapps.
> +
> +homepage      http://www.lsc-group.phys.uwm.edu/daswg
> +master_sites  ${homepage}
> +
> +depends_run   port:python25 \
> +              port:python_select \
> +              port:py25-numpy \
> +              port:py25-pyrxp \
> +              port:py25-socket-ssl \
> +              port:py25-m2crypto \
> +              port:pkgconfig \

This port doesn't appear to make any use of pkgconfig so I would  
delete this dependency.

If you think you must keep it, then it should be a build dependency  
and not a runtime dependency.


> +              port:fftw-3 \
> +              port:fftw-3-single \
> +              port:libframe \
> +              port:metaio \
> +              port:libxml2 \
> +              port:cfitsio \
> +              port:git-core \
> +              port:autoconf \
> +              port:automake

autoconf and automake are also likely to be build dependencies only,  
and not of this port, so I would remove these here as well.

Or maybe I don't understand the purpose of this port...


> +fetch {}
> +checksum {}

You shouldn't override the fetch or checksum phases. Instead, set  
distfiles to the empty string (write "distfiles" by itself on a line  
of the portfile).

> +build {}
> +destroot {
> +  xinstall -d ${destroot}${prefix}/share/doc/${name}-${version}
> +  system "echo ${long_description} > ${destroot}${prefix}/share/ 
> doc/${name}-${version}/README.txt"
> +}
> +
> +post-activate {
> +  ui_msg "\nTo complete the installation and prepare your system  
> for use, please run:
> +\n\tsudo python_select python25\n"
> +}
> +
> +use_configure no
> +universal_variant no
> +
> +livecheck.check none